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.

Sign up for GitHub

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

Jump to bottom

"DNF4 Integration Tests" CI test does not use librepo built from a pull request #303

Open
ppisar opened this issue Mar 8, 2024 · 0 comments
Open

"DNF4 Integration Tests" CI test does not use librepo built from a pull request #303

ppisar opened this issue Mar 8, 2024 · 0 comments
Labels
Triaged Someone on the DNF 5 team has read the issue and determined the next steps to take

Comments

@ppisar
Copy link
Contributor

ppisar commented Mar 8, 2024

In PR #302 we noticed that "DNF4 Integration Tests" tests a wrong code on a pull request. Instead of installing librepo built from the pull request, it installs librepo from copr:copr.fedorainfracloud.org:rpmsoftwaremanagement:dnf-nightly repository. That repository contains a day old snapshot.

That means the "DNF4 Integration Tests" is never run for a pull request code and completely misses the point of CI.

The same applies to "DNF5 Integration Tests" task.
@jan-kolarik jan-kolarik added the Triaged Someone on the DNF 5 team has read the issue and determined the next steps to take label Mar 11, 2024
@m-blaha m-blaha mentioned this issue Oct 9, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Triaged Someone on the DNF 5 team has read the issue and determined the next steps to take
Projects
DNF team
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ppisar @jan-kolarik