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: add commentary about certmonger behaviour #2632

Closed

Conversation

flo-renaud
Copy link
Contributor

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

It is not obvious why we "renew" (reuse only) the IPA CA certificate
in ipa-certupdate.  Add some commentary to explain this behaviour.

Related: https://pagure.io/freeipa/issue/7751
See also: freeipa#2576 (comment)
@flo-renaud
Copy link
Contributor Author

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

@flo-renaud flo-renaud added the ack Pull Request approved, can be merged label Dec 3, 2018
@rcritten rcritten added the pushed Pull Request has already been pushed label Dec 4, 2018
@rcritten
Copy link
Contributor

rcritten commented Dec 4, 2018

ipa-4-6:

  • 35e9f64 certupdate: add commentary about certmonger behaviour

@rcritten rcritten closed this Dec 4, 2018
@flo-renaud flo-renaud deleted the backport_pr2622_ipa-4-6 branch December 5, 2018 10:56
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
3 participants