Bug 1895607 - certificates are regenerated only when crt is missing.
Summary: certificates are regenerated only when crt is missing.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Logging
Version: 4.5
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: 4.6.z
Assignee: Jeff Cantrill
QA Contact: Anping Li
URL:
Whiteboard: logging-core
Depends On: 1888958
Blocks: 1915840
TreeView+ depends on / blocked
 
Reported: 2020-11-07 17:52 UTC by Jeff Cantrill
Modified: 2024-06-13 23:22 UTC (History)
12 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Cause: Certificates are stored in a temp location and not kept across invocations of the operator. Additionally, multiple threads can access the same code paths leading to possible inconsistencies with the certificate generation. Consequence: It's possible the various nodes of the ES log store are restarted with different certificates. Additionally there are times where the collector may not be able to write to the logstore because there is a mismatch in certs. Fix: Provide additional log messages and checks to determine if certificates need to be regenerated as well as introduce a mutex to ensure single threaded execution. Additionally, all generated certs are stored and fetched from the master cert secret Result: Generated certificates are consistently fetched and written to/from the same source allowing them to be checked during reconciliation
Clone Of: 1888958
: 1915840 (view as bug list)
Environment:
Last Closed: 2021-01-22 15:28:30 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift cluster-logging-operator pull 775 0 None closed Bug 1895607: Store secrets in one place and utilize mutex 2021-02-15 14:17:44 UTC
Red Hat Knowledge Base (Solution) 5495561 0 None None None 2020-11-09 10:05:49 UTC

Comment 3 Anping Li 2020-12-02 06:12:08 UTC
Verified and pass on clusterlogging.4.6.0-202011261617.p0.

Comment 5 Oscar Casal Sanchez 2020-12-07 08:54:03 UTC
Hello,

I can see in the PR [1] that in the latest updated (12 days ago) it failed to apply on top of the branch "release-4.5". Is this going to be tried again?

Regards,
Oscar


Note You need to log in before you can comment on or make changes to this bug.