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

ContainerRuntimeConfig controller should limit the volume of preserved conditions

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Normal Normal
    • None
    • 4.16.z
    • Node / CRI-O
    • None
    • False
    • Hide

      None

      Show
      None
    • None
    • Important
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Description of problem:

      The container_runtime_config_controller.go currently preserves all status conditions, leading 

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

      All current releases

      How reproducible:

      Always

      Actual results:

      • All ContainerRuntimeConfig status conditions are preserved, eventually leading to ResourceExhausted error3

       

      $ oc get containerruntimeconfig worker -o json | jq -r '.status.conditions | .[] | .type' | sort | uniq -c
         3313 Failure
         3313 Success 
      
      W1216 08:55:50.750828       1 container_runtime_config_controller.go:557] error updating container runtime config status: rpc error: code = ResourceExhausted desc = trying to send message larger than max (2526542 vs. 2097152)

       

      Expected results:

      • Only for a limited volume of status conditions to be preserved.

      [0] https://github.com/openshift/machine-config-operator/blame/main/pkg/controller/container-runtime-config/container_runtime_config_controller.go#L564-L568

      [1] https://github.com/openshift/machine-config-operator/commit/409081bc72c3a1dd91a72748e699c044aa8791a0 

              qiwan233 Qi Wang
              rhn-support-rsandu Robert Sandu
              None
              None
              Min Li Min Li
              None
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated: