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

Cluster API should sort CredentialsRequest manifests after namespace

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done-Errata
    • Icon: Normal Normal
    • None
    • 4.17
    • None
    • Moderate
    • None
    • CLOUD Sprint 256, CLOUD Sprint 257
    • 2
    • False
    • Hide

      None

      Show
      None
    • N/A
    • Release Note Not Required
    • Done

      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
            trking W. Trevor King
            Milind Yadav Milind Yadav
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: