Uploaded image for project: 'Project Quay'
  1. Project Quay
  2. PROJQUAY-5683

Quay operator should recycle the existing mirror pods after setting mirror as unmanaged

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Won't Do
    • Icon: Normal Normal
    • None
    • quay-v3.8.9, quay-v3.9.0
    • quay-operator
    • False
    • Hide

      None

      Show
      None
    • False

      Description of problem:

      Deploy quay with managed mirror at first.  When all pods are running successfully,  Update quayregistry to set mirror as unmanaged.  The existing mirror pods won't be stopped by quay operator. 

       

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

      quay-operator-bundle-container-v3.9.0-122
       
      
      $ oc get $(oc get pod -l name=quay-operator-alm-owned -o name -n quay-enterprise) -o jsonpath='{.spec.containers[0].image}' -n quay-enterprise
      registry.redhat.io/quay/quay-operator-rhel8@sha256:6fbe3d927a6b609d8de2fde811c56ce10863d54a25f00aa68516c570141af877
      
      ------------------------------
      $ oc get $(oc get pod -l app=quay -o NAME -n quay-enterprise|head -n 1) -o jsonpath='{.spec.containers[0].image}' -n quay-enterprise
      registry.redhat.io/quay/quay-rhel8@sha256:87cd280aa8e5448065f168be5c8e3105d6cc3fa44623f075196850574194226f

       

      How reproducible:

      always

      Steps to Reproduce:
      1.  deploy quay by below setting

      $ cat quayregistry.yaml 
      apiVersion: quay.redhat.com/v1
      kind: QuayRegistry
      metadata:
        name: quayregistry
        namespace: quay-enterprise 
      spec:
        configBundleSecret: config-bundle-secret 
        components:
          - kind: clair
            managed: true 
          - kind: postgres
            managed: true 
          - kind: objectstorage
            managed: false 
          - kind: redis
            managed: true 
          - kind: horizontalpodautoscaler
            managed: false 
          - kind: route
            managed: true 
          - kind: mirror
            managed: true 
          - kind: monitoring
            managed: false 
          - kind: quay
            managed: true
            overrides:
              env:    
              - name: DEBUGLOG
                value: "true" 

       

      2.  update quayregistry, set mirror as unmanaged by "oc edit quayregistry quayregistry"

          - kind: mirror       
            managed: false

      3. check pods status 

      $ oc get pod
      NAME                                               READY   STATUS      RESTARTS        AGE
      quay-operator.v3.9.0-7bb99f8d89-54pd4              1/1     Running     0               3h10m
      quayregistry-clair-app-64c799494d-pb7mh            1/1     Running     0               15m
      quayregistry-clair-app-64c799494d-tlj4c            1/1     Running     0               15m
      quayregistry-clair-postgres-f756f9d7c-lnfrs        1/1     Running     1 (3h10m ago)   3h10m
      quayregistry-quay-app-68cb6df984-ltjmv             1/1     Running     0               15m
      quayregistry-quay-app-68cb6df984-pvvd6             1/1     Running     0               14m
      quayregistry-quay-app-upgrade-h8klz                0/1     Completed   0               3h10m
      quayregistry-quay-config-editor-7f658fd69d-jhlpb   1/1     Running     0               15m
      quayregistry-quay-database-54f959fccd-v4mgr        1/1     Running     0               3h10m
      quayregistry-quay-mirror-75bcf5f4cd-d29p9          1/1     Running     0               46m
      quayregistry-quay-mirror-75bcf5f4cd-jnnkx          1/1     Running     0               46m
      quayregistry-quay-redis-58d84bd6dc-bjfhl           1/1     Running     0               3h10m 

      Actual results:

      The existing mirror pods won't be stopped by quay operator. 

      Expected results:

      The existing mirror pods should be stopped by quay operator. 

       

      Additional info:

      This is not Quay 3.9.0 regression issue, also existed in previous quay operator version.

       

              Unassigned Unassigned
              rhwhu Weihua Hu
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: