Skip to content

storage: add comprehensive tests for lvmohba storage repositories#470

Open
glehmann wants to merge 1 commit intogln/skip-large-vdi-tests-vzolfrom
gln/storage-lvmohba-tests-rxnv
Open

storage: add comprehensive tests for lvmohba storage repositories#470
glehmann wants to merge 1 commit intogln/skip-large-vdi-tests-vzolfrom
gln/storage-lvmohba-tests-rxnv

Conversation

@glehmann
Copy link
Copy Markdown
Member

@glehmann glehmann commented Apr 13, 2026 •
edited
Loading

Signed-off-by: Gaëtan Lehmann gaetan.lehmann@vates.tech

This PR is part of a stack containing 18 PRs:

  1. master
  2. storage: avoid removing the xva/vdi before going in the debugger #436
  3. storage: test large volumes #437
  4. host: include mdadm RAID devices in disk detection #447
  5. Update randstream to 0.5.0 #446
  6. storage: Avoid writing the whole device in coalesce tests #449
  7. storage: avoid writing the whole device in migration tests #450
  8. xva/vdi: only write a small data amount in large volumes for faster tests #452
  9. storage: test full device write #453
  10. storage: test that we can't create a vdi over its max allowed size #454
  11. storage: add jobs for large volume tests #461
  12. skip large volume tests for zvol and nfsv4 #464
  13. "storage: add comprehensive tests for lvmohba storage repositories" (this PR)
  14. storage: free space for XVA import by destroying source VM first #471
  15. storage: limit data written per VDI with --write-volume-cap #481
  16. migration: create a xfs sr on the second host for intra/cross-pool migration #497
  17. Enhance block device management #498
  18. Add VHD_MAX and QCOW2_MAX symbolic size constants #500
  19. partially_populate_device: align spans to block size for better performance #509

@glehmann glehmann requested review from a team as code owners April 13, 2026 08:20
This was referenced Apr 13, 2026
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
@stormi
Copy link
Copy Markdown
Member

stormi commented Apr 14, 2026

This is an important commit. Let's give it more than just a commit title.

@glehmann glehmann force-pushed the gln/skip-large-vdi-tests-vzol branch from 03dabf3 to a11529a Compare April 15, 2026 16:18
@glehmann glehmann force-pushed the gln/storage-lvmohba-tests-rxnv branch from 604cee5 to 8427358 Compare April 15, 2026 16:18
klmp200
klmp200 approved these changes Apr 16, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@klmp200 klmp200 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good :)

@glehmann glehmann changed the title storage: add lvmohba tests storage: add comprehensive tests for lvmohba storage repositories Apr 16, 2026
@glehmann glehmann mentioned this pull request Apr 16, 2026
Open
@glehmann glehmann force-pushed the gln/skip-large-vdi-tests-vzol branch from a11529a to 2da0fe5 Compare April 16, 2026 18:00
@glehmann glehmann force-pushed the gln/storage-lvmohba-tests-rxnv branch 2 times, most recently from 1f54a73 to 75e6d68 Compare April 17, 2026 13:52
@glehmann glehmann force-pushed the gln/skip-large-vdi-tests-vzol branch 2 times, most recently from 34fd633 to 6a73db3 Compare April 21, 2026 06:08
@glehmann glehmann force-pushed the gln/storage-lvmohba-tests-rxnv branch from 75e6d68 to 0070121 Compare April 21, 2026 06:08
@glehmann glehmann force-pushed the gln/skip-large-vdi-tests-vzol branch from 6a73db3 to c8ab96f Compare April 21, 2026 12:47
@glehmann glehmann force-pushed the gln/storage-lvmohba-tests-rxnv branch 2 times, most recently from 0dabee2 to e5fefdf Compare April 21, 2026 13:24
@glehmann glehmann force-pushed the gln/skip-large-vdi-tests-vzol branch 2 times, most recently from 1043997 to c4f93a7 Compare April 21, 2026 14:03
@glehmann glehmann force-pushed the gln/storage-lvmohba-tests-rxnv branch 2 times, most recently from 75addb1 to 254b633 Compare April 24, 2026 14:55
@glehmann glehmann force-pushed the gln/skip-large-vdi-tests-vzol branch from c4f93a7 to 317f3ec Compare April 24, 2026 14:55
This was referenced Apr 24, 2026
Open
Open
Open
@glehmann
storage: add comprehensive tests for lvmohba storage repositories
848fda3 
Implement test suite for lvmohba SR operations including VDI management,
snapshot/clone operations, XVA export/import, and intra/cross-pool migrations.
Tests are mostly copied from lvmoiscsi with lvmohba-specific adjustments.

Signed-off-by: Gaëtan Lehmann <gaetan.lehmann@vates.tech>
@glehmann glehmann force-pushed the gln/skip-large-vdi-tests-vzol branch from 317f3ec to f424d59 Compare April 29, 2026 17:39
@glehmann glehmann force-pushed the gln/storage-lvmohba-tests-rxnv branch from 254b633 to 848fda3 Compare April 29, 2026 17:39
@glehmann glehmann mentioned this pull request Apr 29, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stormi stormi stormi approved these changes

@klmp200 klmp200 klmp200 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@glehmann @stormi @klmp200