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

Skip update if old package can't be installed #37

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

jpopelka
Copy link
Member

https://pagure.io/fedora-ci/general/issue/382
https://gitlab.com/testing-farm/oculus/-/issues/18

Ideally, the prepare phase of the downgrade test would do the same, i.e.
first remove package, then try to install the old one and if that fails, skip the test, otherwise update to the new package.
But that'd be 3 operations (remove, install, update) more than what we do now for every package just for a very rare use-case (old package can't be installed).
Now the prep-downgrade does nothing because the previous (update) test leaves the new package installed.

Also, have separate exit code for skipped tests so that we can distinguish them in the viewer.

https://pagure.io/fedora-ci/general/issue/382
https://gitlab.com/testing-farm/oculus/-/issues/18

Ideally, the prepare phase of downgrade test would do the same, i.e.
first remove package, then try to install the old one and if that
fails, skip the test, otherwise update to the new package.
But that'd be 3 operations (remove, install, update) more than
what we do now for every package just for a very rare use-case
(old package can't be installed).
Now the prep-downgrade does nothing because
the previous (update) test leaves the new package installed.
This way we can distinguish such tests in the viewer
@jpopelka jpopelka merged commit 4116938 into fedora-ci:main Jan 19, 2024
9 checks passed
@jpopelka jpopelka deleted the skipping branch January 19, 2024 09:01
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