Skip to content
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.

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

tests: Use EPEL-7 from archive #719

Merged
merged 1 commit into from
Aug 16, 2024
Merged

Conversation

jharuda
Copy link
Contributor

@jharuda jharuda commented Aug 14, 2024

Enhancement:
Fix tests on EL-7

Reason:
EL-7 test were failing due to missing EPEL-7 packages/repository

Result:
EL-7 test should pass

Issue Tracker Tickets (Jira or BZ if any):
#718

Copy link

codecov bot commented Aug 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.29%. Comparing base (89d7148) to head (8c22594).
Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #719       +/-   ##
===========================================
+ Coverage   20.40%   43.29%   +22.88%     
===========================================
  Files          10       12        +2     
  Lines        1485     3109     +1624     
  Branches      436        0      -436     
===========================================
+ Hits          303     1346     +1043     
- Misses       1181     1763      +582     
+ Partials        1        0        -1     
Flag Coverage Δ
sanity ?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@jharuda jharuda changed the title Tests: Use EPEL-7 from archive Tests Use EPEL-7 from archive Aug 14, 2024
@jharuda jharuda changed the title Tests Use EPEL-7 from archive tests: Use EPEL-7 from archive Aug 14, 2024
tests/tasks/enable_epel.yml Outdated Show resolved Hide resolved
tests/tasks/enable_epel.yml Outdated Show resolved Hide resolved
tests/tasks/enable_epel.yml Outdated Show resolved Hide resolved
tests/tasks/enable_epel.yml Outdated Show resolved Hide resolved
tests/tasks/enable_epel.yml Outdated Show resolved Hide resolved
tests/tasks/enable_epel.yml Outdated Show resolved Hide resolved
tests/tasks/enable_epel.yml Outdated Show resolved Hide resolved
@richm
Copy link
Contributor

richm commented Aug 14, 2024

[citest]

@spetrosi
Copy link
Collaborator

[citest]

@richm
Copy link
Contributor

richm commented Aug 15, 2024

I merged the fix for the failing fedora-39 and fedora-40 tests. @jharuda please rebase to the latest main branch. Also, the network role requires PRs to be signed with git commit -s - once you do the rebase, do a git commit -s --amend to add the signature, then git push -f ... your commit.

@richm
Copy link
Contributor

richm commented Aug 16, 2024

I also merged the fix for the integration / integration tests

@jharuda jharuda force-pushed the epel7_archive branch 2 times, most recently from d32259c to 4c58911 Compare August 16, 2024 12:29
@richm richm merged commit 3841192 into linux-system-roles:main Aug 16, 2024
24 checks passed
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.

3 participants