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

Unexpected featuregate "ExternalRouteCertificate" added in openshift/api

XMLWordPrintable

    • No
    • 2
    • NE Sprint 256, NE Sprint 257, NE Sprint 258
    • 3
    • Rejected
    • False
    • Hide

      None

      Show
      None

      Description of problem:

          As part of https://issues.redhat.com/browse/CFE-811, we added a featuregate "RouteExternalCertificate" to release the feature as TP, and all the code implementations were behind this gate.
      
      However, it seems https://github.com/openshift/api/pull/1731 inadvertently duplicated "ExternalRouteCertificate" as "RouteExternalCertificate".

      Version-Release number of selected component (if applicable):

          4.16

      How reproducible:

          100%

      Steps to Reproduce:

         $ oc get featuregates.config.openshift.io cluster -oyaml 
      <......>
      spec:
        featureSet: TechPreviewNoUpgrade
      status:
        featureGates:
          enabled:
          - name: ExternalRouteCertificate
          - name: RouteExternalCertificate
      <......>     

      Actual results:

          Both RouteExternalCertificate and ExternalRouteCertificate were added in the API

      Expected results:

      We should have only one featuregate "RouteExternalCertificate" and the same should be displayed in https://docs.openshift.com/container-platform/4.16/nodes/clusters/nodes-cluster-enabling-features.html

      Additional info:

       Git commits
      
      https://github.com/openshift/api/commit/11f491c2c64c3f47cea6c12cc58611301bac10b3
      
      https://github.com/openshift/api/commit/ff31f9c1a0e4553cb63c3e530e46a3e8d2e30930
      
      Slack thread: https://redhat-internal.slack.com/archives/C06EK9ZH3Q8/p1719867937186219

              rh-ee-ckyal Chirag Kyal
              rh-ee-ckyal Chirag Kyal
              Hongan Li Hongan Li
              Subhashini T K Subhashini T K
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Created:
                Updated: