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

ipatests: test_epn: Fix package installation #4915

Closed
wants to merge 1 commit into from

Conversation

miskopo
Copy link
Member

@miskopo miskopo commented Jul 14, 2020

EPN functionality is provided as separate package
client-epn, but it is not installed during setup. This resolves this
behaviour.

Signed-off-by: Michal Polovka mpolovka@redhat.com

@miskopo miskopo requested a review from rcritten July 14, 2020 14:16
@miskopo miskopo added the needs review Pull Request is waiting for a review label Jul 14, 2020
@rcritten rcritten self-assigned this Jul 14, 2020
@rcritten
Copy link
Contributor

Looks good, let's see what CI thinks about it. I'm not sure if *ipa-client-epn is in the template already or what but this looks like an obviously necessary change.

@Tiboris
Copy link
Member

Tiboris commented Jul 14, 2020

I hope it does not break.

EPN functionality is provided as separate package
freeipa-client-epn, but it is not installed during setup. This resolves
this behaviour.

Signed-off-by: Michal Polovka <mpolovka@redhat.com>
@miskopo
Copy link
Member Author

miskopo commented Jul 14, 2020

Temp commit run available at: AwwWwS

@rcritten
Copy link
Contributor

Please remove the temp commit and set any backport flags needed. I assume just ipa-4-8

@rcritten rcritten removed the needs review Pull Request is waiting for a review label Jul 14, 2020
@miskopo miskopo added the ipa-4-8 Mark for backport to ipa 4.8 label Jul 14, 2020
@miskopo miskopo added prioritized Pull Request has higher priority for PR-CI re-run Trigger a new run of PR-CI labels Jul 14, 2020
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Jul 14, 2020
@rcritten rcritten added the ack Pull Request approved, can be merged label Jul 14, 2020
@rcritten
Copy link
Contributor

master:

  • 3c18f94 ipatests: test_epn: Fix package installation

@rcritten rcritten added the pushed Pull Request has already been pushed label Jul 14, 2020
@rcritten rcritten closed this Jul 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged ipa-4-8 Mark for backport to ipa 4.8 prioritized Pull Request has higher priority for PR-CI pushed Pull Request has already been pushed
Projects
None yet
4 participants