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

[Backport][ipa-4-9] Adapt redhat ipaplatform to RHEL9/ELN #5671

Closed
wants to merge 1 commit into from

Conversation

abbra
Copy link
Contributor

@abbra abbra commented Mar 29, 2021

This PR was opened automatically because PR #5669 was pushed to master and backport to ipa-4-9 is required.

On RHEL8, ipa is using named-pkcs11.service but RHEL9 is based on
fedora34 and uses named.service instead. There is already some support
for this distinction in ipaplatform, and the patch relies on the
specific settings that can be configured in ipaplatform/xx/services.py
and ipaplatform/xx/constants.py

On RHEL9 ipa also needs to define NAMED_OPENSSL_ENGINE for named
to use openssl's okcs11 engine.

Fixes: https://pagure.io/freeipa/issue/8753
@abbra abbra added the ack Pull Request approved, can be merged label Mar 29, 2021
@abbra
Copy link
Contributor Author

abbra commented Mar 29, 2021

PR was ACKed automatically because this is backport of PR #5669. Wait for CI to finish before pushing. In case of questions or problems contact @flo-renaud who is author of the original PR.

@abbra abbra added the re-run Trigger a new run of PR-CI label Mar 29, 2021
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Mar 29, 2021
@flo-renaud
Copy link
Contributor

ipa-4-9:

  • 8272da7 Adapt redhat ipaplatform to RHEL9/ELN

@flo-renaud flo-renaud added the pushed Pull Request has already been pushed label Mar 29, 2021
@flo-renaud flo-renaud closed this Mar 29, 2021
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 pushed Pull Request has already been pushed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants