linux-system-roles / network

An ansible role to configure networking
https://linux-system-roles.github.io/network/
BSD 3-Clause "New" or "Revised" License
238 stars 106 forks source link

ci: fix and improve integration container testing #721

Closed richm closed 3 months ago

richm commented 3 months ago

Use the vault for centos 7 and centos stream 8

Exclude tests/tests_team_plugin_installation_nm.yml since tests/tests_team_nm.yml is excluded.

Use grouping to group log lines for better readability. https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/workflow-commands-for-github-actions#grouping-log-lines

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

codecov[bot] commented 3 months ago

Codecov Report

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

Project coverage is 45.12%. Comparing base (34d1f2d) to head (4a572e5). Report is 27 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #721 +/- ## =========================================== + Coverage 20.50% 45.12% +24.62% =========================================== Files 10 12 +2 Lines 1478 3109 +1631 Branches 433 436 +3 =========================================== + Hits 303 1403 +1100 - Misses 1174 1705 +531 Partials 1 1 ``` | [Flag](https://app.codecov.io/gh/linux-system-roles/network/pull/721/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/network/pull/721/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=linux-system-roles) | `20.40% <ø> (-0.10%)` | :arrow_down: | 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.

richm commented 3 months ago

I have to merge this to trigger a rebuild of the containers