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

[v2] incomplete error message for invalid helm chart

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Normal Normal
    • None
    • 4.18, 4.19
    • oc-mirror
    • None
    • CLID Sprint 272
    • 1
    • False
    • Hide

      None

      Show
      None
    • Update the error messages for helm charts with more context.
    • Bug Fix
    • In Progress

      Description of problem:

          Running a m2d workflow with an ISC containing an invalid helm chart results in a unhelpful error message.

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

          4.18+

      How reproducible:

          always

      Steps to Reproduce:

         ❯ yq e helmisc.yaml 
      kind: ImageSetConfiguration
      apiVersion: mirror.openshift.io/v2alpha1
      mirror:
        helm:
          repositories:
            - url: helm.invalid.io/invalid
              name: invalid-helm-repository
              charts:
                - name: invalid-helm-chart

      Actual results:

      ❯ ~/prjs/github.com/openshift/oc-mirror-return-values/bin/oc-mirror --v2 --config helmisc.yaml file://data/foo-error --log-level debug
      [...]
      2025/04/01 16:09:47  [INFO]   : 🔍 collecting helm images...
      2025/04/01 16:09:47  [DEBUG]  : collection time     : 254.804µs
      2025/04/01 16:09:47  [INFO]   : mirror time     : 321.348µs
      2025/04/01 16:09:47  [INFO]   : 👋 Goodbye, thank you for using oc-mirror
      2025/04/01 16:09:47  [ERROR]  : collection error: could not find protocol handler for:  

      Expected results:

          More descriptive error message.

      Additional info:

          

              luzuccar@redhat.com Luigi Mario Zuccarelli
              rdossant Rafael Fonseca dos Santos
              Nidan Gavali Nidan Gavali
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated: