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

Cluster API should sort CredentialsRequest manifests after namespace

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done-Errata
    • Icon: Normal Normal
    • 4.15.z
    • 4.17
    • None
    • Moderate
    • None
    • CLOUD Sprint 257
    • 1
    • False
    • Hide

      None

      Show
      None
    • Release Note Not Required
    • In Progress

      This is a clone of issue OCPBUGS-37441. The following is the description of the original issue:

      This is a clone of issue OCPBUGS-36296. The following is the description of the original issue:

      Description of problem

      Currently the manifests directory has:

      0000_30_cluster-api_00_credentials-request.yaml
      0000_30_cluster-api_00_namespace.yaml
      ...
      

      CredentialsRequests go into the openshift-cloud-credential-operator namespace, so they can come before or after the openshift-cluster-api namespace. But because they ask for Secrets in the openshift-cluster-api namespace, there would be less race and drama if the CredentialsRequest manifests were given a name that sorted them after the namespace. Like 0000_30_cluster-api_01_credentials-request.yaml.

      Version-Release number of selected component

      I haven't gone digging in history, it may have been like this since forever.

      How reproducible

      Every time.

      Steps to Reproduce

      With a release image pullspec like registry.ci.openshift.org/ocp/release:4.17.0-0.nightly-2024-06-27-184535:

      $ oc adm release extract --to manifests registry.ci.openshift.org/ocp/release:4.17.0-0.nightly-2024-06-27-184535
      $ ls manifests/0000_30_cluster-api_* | grep 'namespace\|credentials-request'
      

      Actual results

      $ ls manifests/0000_30_cluster-api_* | grep 'namespace\|credentials-request'
      manifests/0000_30_cluster-api_00_credentials-request.yaml
      manifests/0000_30_cluster-api_00_namespace.yaml
      

      Expected results

      $ ls manifests/0000_30_cluster-api_* | grep 'namespace\|credentials-request'
      manifests/0000_30_cluster-api_00_namespace.yaml
      manifests/0000_30_cluster-api_01_credentials-request.yaml
      

              ddonati@redhat.com Damiano Donati
              openshift-crt-jira-prow OpenShift Prow Bot
              Milind Yadav Milind Yadav
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Created:
                Updated:
                Resolved: