linux-system-roles / storage

Ansible role for linux storage management
https://linux-system-roles.github.io/storage/
MIT License
100 stars 59 forks source link

test: skip lvm pool shared test if not supported #426

Closed richm closed 7 months ago

richm commented 7 months ago

Our CI and local test frameworks do not currently support the shared lvm pool test. Skip it by default. If you want to run the test, set the environment variable SYSTEM_ROLES_STORAGE_SHARED_TEST=true in your ansible-playbook invocation environment e.g. SYSTEM_ROLES_STORAGE_SHARED_TEST=true ansible-playbook ... tests_lvm_pool_shared.yml

Signed-off-by: Rich Megginson rmeggins@redhat.com

codecov[bot] commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (533145b) 16.54% compared to head (efd90e9) 13.29%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #426 +/- ## ========================================== - Coverage 16.54% 13.29% -3.25% ========================================== Files 2 8 +6 Lines 284 1782 +1498 Branches 79 79 ========================================== + Hits 47 237 +190 - Misses 237 1545 +1308 ``` | [Flag](https://app.codecov.io/gh/linux-system-roles/storage/pull/426/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=linux-system-roles) | Coverage Δ | | |---|---|---| | [sanity](https://app.codecov.io/gh/linux-system-roles/storage/pull/426/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=linux-system-roles) | `16.54% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=linux-system-roles#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.