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-6] certupdate: only add LWCA tracking requests on CA servers #4902

Closed

Conversation

frasertweedale
Copy link
Contributor

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

ipa-certupdate throws an exception when executed on a non-CA server
in a CA-ful deployment with lightweight sub-CAs (LWCAs).  Check that
we are on a CA server before attempting to create Certmonger
tracking requests for LWCAs.

HOW TO TEST

1. Install first server (with CA)
2. Install replica without CA
3. Create sub-CA (`ipa ca-add`)
4. Run `ipa-certupdate` on replica.  Observe that no stack trace is
   produced.

Fixes: https://pagure.io/freeipa/issue/8399
@frasertweedale frasertweedale added the ack Pull Request approved, can be merged label Jul 9, 2020
@frasertweedale
Copy link
Contributor Author

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

@abbra abbra added the pushed Pull Request has already been pushed label Jul 9, 2020
@abbra
Copy link
Contributor

abbra commented Jul 9, 2020

ipa-4-6:

  • 31c5104 certupdate: only add LWCA tracking requests on CA servers

@abbra abbra closed this Jul 9, 2020
@frasertweedale frasertweedale deleted the backport_pr4896_ipa-4-6 branch July 13, 2020 00:34
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
2 participants