Uploaded image for project: 'OpenShift Bugs'
  1. OpenShift Bugs
  2. OCPBUGS-28231

Guard mint-mode GCP 4.14 to 4.15 on sufficient creds

XMLWordPrintable

      Description of problem

      IR-407 and other similar 4.15 work around moving to GCP custom credentials means 4.15 mint mode GCP clusters will need additional roles associated with their root (minting) credential. Clusters which lack those roles may stick in 4.14-to-4.15 updates with failures like:

      $ oc -n openshift-cloud-credential-operator get -o json credentialsrequests | jq -r '.items[] | select(tostring | contains("InfrastructureMismatch") | not) | .metadata.name as $n | .status.conditions // [{type: "NoCon
      ditions"}] | .[] | .type + "=" + .status + " " + $n + " " + .reason + ": " + .message' | sort
      ...
      CredentialsProvisionFailure=True openshift-ingress-gcp CredentialsProvisionFailure: failed to grant creds: error syncing creds in mint-mode: error creating custom role: rpc error: code = PermissionDenied desc = You don't have permission to create a role in projects/...
      ...
      

      The 4.14 cloud-cred opeator should grow a guard to inform any exposed customers before they launch the update into 4.15, so they can calmly plan their changes ahead of tim, and not be surprised mid-update.

      Version-Release number of selected component

      4.14-to-4.15 updates are exposed. The 4.14.z releases need a patch to encode this new guard.

      How reproducible

      Every time.

      Steps to Reproduce

      1. Install a 4.14 GCP cluster.
      2. Set the root secret to one with 4.14's minimal permissions.
      3. Launch an update to 4.15.

      Actual results

      The update sticks on a Degraded=True CCO ClusterOperator, which is in turn because the minting cred lacks permission to provision the incoming 4.15 ingress credential.

      Expected results

      GCP mint mode admins on 4.14 are informed of the need to adjust their minting credential, and the CVO does not allow unforced updates to begin until there is som sign that has happened.

      Additional info

              jstuever@redhat.com Jeremiah Stuever
              trking W. Trevor King
              Jianping Shu Jianping Shu
              Jeana Routh Jeana Routh
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

                Created:
                Updated:
                Resolved: