Skip to content

Storage: fix resize check fail issue #6540

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Storage: fix resize check fail issue #6540

wants to merge 1 commit into from

Conversation

meinaLi
Copy link
Contributor

@meinaLi meinaLi commented Sep 5, 2025

In image mode, the volume size of netfs pool will be affected by the metadata size. So it has size change and this change is within a reasonable range. So cover this size change in code.

@meinaLi
Storage: fix resize check fail issue
c8684f2 
In image mode, the volume size of netfs pool will be affected by the
metadata size. So it has size change and this change is within a
reasonable range. So cover this size change in code.

Signed-off-by: meinaLi <[email protected]>
@meinaLi
Copy link
Contributor Author

meinaLi commented Sep 5, 2025 

# avocado run --vt-type libvirt --vt-omit-data-loss --vt-machine-type q35 virsh.vol_resize.positive_test.non_encrypt..netfs
JOB ID     : 8367b74faad3a02b78585f6e3e7abf4b7f77cdfe
JOB LOG    : /var/log/avocado/job-results/job-2025-09-04T23.03-8367b74/job.log
 (1/8) type_specific.io-github-autotest-libvirt.virsh.vol_resize.positive_test.non_encrypt.non_acl.sparse_capacity.netfs: STARTED
 (1/8) type_specific.io-github-autotest-libvirt.virsh.vol_resize.positive_test.non_encrypt.non_acl.sparse_capacity.netfs: PASS (10.12 s)
......
 (8/8) type_specific.io-github-autotest-libvirt.virsh.vol_resize.positive_test.non_encrypt.acl_test.delta_allocate_capacity.netfs: STARTED
 (8/8) type_specific.io-github-autotest-libvirt.virsh.vol_resize.positive_test.non_encrypt.acl_test.delta_allocate_capacity.netfs: PASS (19.85 s)
RESULTS    : PASS 8 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML   : /var/log/avocado/job-results/job-2025-09-04T23.03-8367b74/results.html
JOB TIME   : 138.56 s

@meinaLi
Copy link
Contributor Author

meinaLi commented Sep 5, 2025

This PR depends on avocado-framework/avocado-vt#4215.

@chloerh chloerh added the depend on The PR has dependency on other PRs label Sep 5, 2025
@smitterl
Copy link
Contributor

This PR depends on avocado-framework/avocado-vt#4215.

Dependency has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chloerh chloerh chloerh approved these changes

Assignees
No one assigned
Labels
depend on The PR has dependency on other PRs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@meinaLi @smitterl @chloerh