Skip to content

Conversation

@richm
Copy link
Contributor

@richm richm commented Apr 14, 2025

Some systems do not use the ethN interface naming scheme, and
use ensN instead. The test wants to use eth1 as the second
interface. If this does not exist, try ens4 instead.

Some systems do not use the `ethN` interface naming scheme, and
use `ensN` instead.  The test wants to use `eth1` as the second
interface.  If this does not exist, try `ens4` instead.
@richm richm requested a review from liangwen12year as a code owner April 14, 2025 16:40
@codecov
Copy link

codecov bot commented Apr 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.25%. Comparing base (1b57520) to head (2b23a83).
Report is 12 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #779      +/-   ##
==========================================
+ Coverage   43.11%   43.25%   +0.13%     
==========================================
  Files          12       12              
  Lines        3124     3121       -3     
==========================================
+ Hits         1347     1350       +3     
+ Misses       1777     1771       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@richm
Copy link
Contributor Author

richm commented Apr 15, 2025

fedora 42 tests are hanging on tests_mac_address_match - can reproduce locally - debugging

@richm
Copy link
Contributor Author

richm commented Apr 15, 2025

Will fix one of the f42 issues in a separate commit.

@richm richm merged commit 7c73334 into linux-system-roles:main Apr 15, 2025
22 of 32 checks passed
@richm richm deleted the test-find-second-interface branch April 15, 2025 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant