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

context-rich and case-sensitive config validation for CMO

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done-Errata
    • Icon: Undefined Undefined
    • 4.18.0
    • 4.18
    • Monitoring
    • None
    • Quality / Stability / Reliability
    • False
    • Hide

      None

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

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

      Description of problem:

      
      https://issues.redhat.com//browse/OCPBUGS-42671 introduced strict validation.
      that should help identify and correct many "broken" config.
      
      Feedback said that the new "unknown field" and "duplicate field" checks don't provide enough context; `unknown field \"tolerations\"` r.g. isn't always enough, especially that `tolerations` can occur in many places.
      
      this is a limitation of Go "encoding/json" v1, tried to enrich the context in vain https://go-review.googlesource.com/c/go/+/648055/comments/aedcc220_a137107a?tab=comments
      
      https://github.com/kubernetes-sigs/json is an enhanced Go "encoding/json" v1 that provides more context and has also a case-sensitive check (which is another limitation of Go "encoding/json" v1 https://pkg.go.dev/encoding/json#Unmarshal)
      
      This case-sensitive unmarshalling will be helpful for the transition towards CRD based config, as CRD validation is itself case-sensitive.
      
      ---
      
      - [ ] Prepare for this in 4.18
      
      
      
          

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

      
          

      How reproducible:

      
          

      Steps to Reproduce:

          1.
          2.
          3.
          

      Actual results:

      
          

      Expected results:

      
          

      Additional info:

      
          

              rh-ee-amrini Ayoub Mrini
              openshift-crt-jira-prow OpenShift Prow Bot
              None
              None
              Junqi Zhao Junqi Zhao
              None
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Created:
                Updated:
                Resolved: