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

Fix COPR and Python dependencies in Fedora Rawhide #468

Merged
merged 2 commits into from
Jun 29, 2022
Merged

Fix COPR and Python dependencies in Fedora Rawhide #468

merged 2 commits into from
Jun 29, 2022

Conversation

netoarmando
Copy link
Member

@netoarmando netoarmando commented Jun 28, 2022

Fix COPR repo address in mock when in rawhide boxes

When running rawhide Ansible's variable ansible_distribution_release is empty, it was expected to be Rawhide. So, instead of using that this change is adding a task to check the content of /etc/fedora-release file.

Install latest pytest-html

Old 1.22.x version was Python 2.7 compatible.

Issue: #467


This has been tested here: https://github.com/netoarmando/freeipa/pull/71

Previous 1.22.x version is Python 2.7 compatible.

Signed-off-by: Armando Neto <abiagion@redhat.com>
When running rawhide Ansible's variable `ansible_distribution_release`
is empty, it was expected to be `Rawhide`. So, instead of using that
this change is adding a task to check the content of
`/etc/fedora-release` file.

Signed-off-by: Armando Neto <abiagion@redhat.com>
Copy link
Contributor

@f-trivino f-trivino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. @netoarmando thanks for the patch, I have verified the rawhide copr URL is something like:

https://download.copr.fedorainfracloud.org/results/ftrivino/krb5-ftrivino/fedora-rawhide-x86_64/

and that the "grep -qvi rawhide /etc/fedora-release" command fails when running in rawhide.

@flo-renaud flo-renaud self-assigned this Jun 29, 2022
Copy link
Contributor

@flo-renaud flo-renaud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@netoarmando
Thanks for the PR, LGTM

@netoarmando netoarmando merged commit 9550018 into freeipa:master Jun 29, 2022
@netoarmando netoarmando deleted the rawhide-changes branch June 29, 2022 20:39
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