Skip to content

Commit

Permalink
ipa-cert-fix man page: add note about certmonger renewal
Browse files Browse the repository at this point in the history
ipa-cert-fix man page needs to explain that certmonger may
trigger a renewal right after ipa-cert-fix completes because
certmonger does not notice the updated certificates.

Also add a similar note at the end of ipa-cert-fix.

Fixes: https://pagure.io/freeipa/issue/8702
Signed-off-by: Florence Blanc-Renaud <flo@redhat.com>
  • Loading branch information
flo-renaud committed Jun 10, 2021
1 parent bcb8d4d commit 9d64c96
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 0 deletions.
7 changes: 7 additions & 0 deletions install/tools/man/ipa-cert-fix.1
Expand Up @@ -39,6 +39,13 @@ for shared certificates via \fIgetcert-resubmit(1)\fR (on the other
CA server). This is to avoid unnecessary renewal of shared
certificates.

Important note: the \fIcertmonger\fR daemon does not immediately notice
the updated certificates and may trigger a renewal after \fIipa-cert-fix\fR
completes. As a consequence, \fIgetcert list\fR output may display
that a renewal is in progress even if \fIipa-cert-fix\fR just
finished. It is recommended to monitor the certmonger-initiated
renewal and wait for its completion before any other administrative task.

.SH "OPTIONS"
.TP
\fB\-\-version\fR
Expand Down
7 changes: 7 additions & 0 deletions ipaserver/install/ipa_cert_fix.py
Expand Up @@ -57,6 +57,12 @@
"""

renewal_note = """
Note: Monitor the certmonger-initiated renewal of
certificates after ipa-cert-fix and wait for its completion before
any other administrative task.
"""

RENEWED_CERT_PATH_TEMPLATE = "/etc/pki/pki-tomcat/certs/{}-renewed.crt"

logger = logging.getLogger(__name__)
Expand Down Expand Up @@ -175,6 +181,7 @@ def run(self):
print("Restarting IPA")
ipautil.run(['ipactl', 'restart'], raiseonerr=True)

print(renewal_note)
return 0


Expand Down

0 comments on commit 9d64c96

Please sign in to comment.