• Icon: Story Story
    • Resolution: Done
    • Icon: Normal Normal
    • None
    • None
    • None

      Enable readOnlyRootFilesystem on all of the cloud-credential-operator pods. This will require reverting prior changes that caused the tls-ca-bundler.pem to be mounted in a temporary location and then moved to the default location as part of the cloud-credential-operator pod's command.

            [CCO-647] Enable readOnlyRootFilesystem on all pods

            mihuang@redhat.com  OCP-80542 lgtm

            Jeremiah Stuever added a comment - mihuang@redhat.com  OCP-80542 lgtm

            mihuang@redhat.com  OCP-80542 mostly looks good, with exception of the step(s) to ensure the CA certificates are correctly mounted. As it is currently written, it only checks that the file exists. Because this file exists in the images themselves, it always exists. In order to accomplish this task, you would need to somehow show that the contents of the file match the contents of the file that was mounted.

            Jeremiah Stuever added a comment - mihuang@redhat.com  OCP-80542 mostly looks good, with exception of the step(s) to ensure the CA certificates are correctly mounted. As it is currently written, it only checks that the file exists. Because this file exists in the images themselves, it always exists. In order to accomplish this task, you would need to somehow show that the contents of the file match the contents of the file that was mounted.

            My concerns about removing the CA bundle workaround have been resolved. In looking at the original bugs where it was introduced, it was said that the workaround could only be tested in a 4.1 to 4.2 upgrade (as opposed to a 4.2 to 4.3 upgrade). This indicates the workaround was specifically to resolve an issue with that particular upgrade.

            Jeremiah Stuever added a comment - My concerns about removing the CA bundle workaround have been resolved. In looking at the original bugs where it was introduced, it was said that the workaround could only be tested in a 4.1 to 4.2 upgrade (as opposed to a 4.2 to 4.3 upgrade). This indicates the workaround was specifically to resolve an issue with that particular upgrade.

              jstuever@redhat.com Jeremiah Stuever
              jstuever@redhat.com Jeremiah Stuever
              Mingxia Huang Mingxia Huang
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: