Skip to content

Commit

Permalink
test/lvol: lower size of bdev in test case 700
Browse files Browse the repository at this point in the history
Using lower size of bdev might speed up test a little bit
and is enough for this case.

Change-Id: Iaba1b8a771aef58c7b26a880cd351c2ec33f27f2
Signed-off-by: Karol Latecki <[email protected]>
Reviewed-on: https://review.gerrithub.io/399655
Reviewed-by: Pawel Kaminski <[email protected]>
Tested-by: SPDK Automated Test System <[email protected]>
Reviewed-by: Jim Harris <[email protected]>
Reviewed-by: Daniel Verkamp <[email protected]>
Reviewed-by: Shuhei Matsumoto <[email protected]>
  • Loading branch information
karlatec authored and danielverkamp committed Feb 13, 2018
1 parent 261b5de commit ef4dba3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions test/lvol/test_cases.py
Original file line number Diff line number Diff line change
Expand Up @@ -956,9 +956,9 @@ def test_case700(self):
uuid_store,
self.cluster_size)

# size = approx 10% of total NVMe disk size
# size = approx 2% of total NVMe disk size
_ = self.c.get_lvol_stores()[0]
size = int(_["free_clusters"] / 10)
size = int(_["free_clusters"] / 50)

for i in range(5):
uuid_bdev = self.c.construct_lvol_bdev(uuid_store,
Expand Down
2 changes: 1 addition & 1 deletion test/lvol/test_plan.md
Original file line number Diff line number Diff line change
Expand Up @@ -675,7 +675,7 @@ Steps:
- construct lvol store on NVMe bdev
- using get_lvol_stores command verify lvol store was correctly created
- construct five lvol bdevs on previously created lvol store;
each lvol bdev size is approximately equal to 10% of total lvol store size
each lvol bdev size is approximately equal to 2% of total lvol store size
(approximately because of the lvol metadata which consumes some of the space)
- using get_bdevs command verify lvol bdevs were correctly created
- shutdown vhost application by sending SIGTERM signal
Expand Down

0 comments on commit ef4dba3

Please sign in to comment.