Closed vojtechtrefny closed 5 months ago
Attention: 1 lines
in your changes are missing coverage. Please review.
Comparison is base (
98cd81f
) 13.30% compared to head (030258d
) 13.29%.
Files | Patch % | Lines |
---|---|---|
library/blivet.py | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This breaks most of the tests. I've run tests on centos-9 and centos-8 and they fail with
TASK [linux-system-roles.storage : Manage the pools and volumes to match the specified state] ***
task path: /home/rmeggins/linux-system-roles/storage/tests/roles/linux-system-roles.storage/tasks/main-blivet.yml:84
...
MSG:
Failed to commit changes to disk: makes no sense to write a label when accepting default label
This breaks most of the tests. I've run tests on centos-9 and centos-8 and they fail with
Sorry, should be fixed now.
What is the relationship between fs_label
and fs_type
gfs2
?
What is the relationship between
fs_label
andfs_type
gfs2
?
I have added comment explaining this in test-verify-volume-fs.yml
. The problem is that blivet doesn't support specifying label when creating GFS2 so it has to be specified with the -t
option passed manually in fs_create_options
which means the verification will always fail -- label is set (we get that information from lsblk
), but we didn't specify label in fs_label
.
GFS2 is supported by blivet, but the support is disabled by default so we need to enable it.
Fixes: #417