Uploaded image for project: 'FlightPath'
  1. FlightPath
  2. FLPATH-2148

[m5] Kafka cluster installation script and doc have outdated instructions due to changes in orchestrator CR

XMLWordPrintable

    • False
    • Hide

      None

      Show
      None
    • False

      Description of the problem:
      In the Kafka cluster installation docs and in the installation script, the command for updating the orchestrator CR for m4 is:

      oc -n "${ORCHESTRATOR_NAMESPACE}" patch orchestrators.rhdh.redhat.com "${ORCHESTRATOR_NAME}" --type merge \
         -p '
      {
        "spec": {
          "orchestrator": {
            "sonataflowPlatform": {
              "eventing": {
                "broker": {
                  "name": "'"${BROKER_NAME}"'",
                  "namespace": "'"${BROKER_NAMESPACE}"'"
                }
              }
            }
          }
        }
      }' 

       In m5, however, the orchestrator CR was changed and the above no longer works. The command should  now be:

      oc -n "${ORCHESTRATOR_NAMESPACE}" patch orchestrators.rhdh.redhat.com "${ORCHESTRATOR_NAME}" --type merge    -p '
      {
        "spec": {
          "platform": {
              "eventing": {
                "broker": {
                  "name": "'"${BROKER_NAME}"'",
                  "namespace": "'"${BROKER_NAMESPACE}"'"
                }
              }
            }
         }
      }' 

      How reproducible: 100%

      Steps to reproduce:

      1. Deploy orchestrator operator 1.5.0-rc2

      2.  Deploy orchestrator

      3. Follow instructions [here|https://github.com/rhdhorchestrator/orchestrator-go-operator/tree/main/docs/main/eventing-communication to deploy a kafka cluster, either using the script or by following the manual instruction.

      Actual results: Patching the orchestrator resource fails:

      Warning: unknown field "spec.orchestrator"
      orchestrator.rhdh.redhat.com/orchestrator patched (no change) 

      Expected results: Docs and script should be updated to reflect the change in the CRD.

              rh-ee-eshalev Elai Shalev
              yfirst Yona First
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: