• Icon: Task Task
    • Resolution: Unresolved
    • Icon: Minor Minor
    • None
    • None
    • 8
    • False
    • None
    • False
    • 8

      User Story:
      As an OpenShift Engineer I want introduce govc 0.4x.x so we can deploy new features and fixes.

      Description:
      govc 0.4x.x features updates to the json schema used in our CI steps; most notably, `ci-operator/step-registry/ipi/conf/vsphere/check/vcm/ipi-conf-vsphere-check-vcm-commands.sh`. 

      Acceptance Criteria:

      • Update steps to support either the new or the old schema. We'll have to support both for a time.
      • Bump the UPI installer to use the desired version of govc.

       

      Other Information:

      tested patch on v0.45.1:

      diff --git a/ci-operator/step-registry/ipi/conf/vsphere/check/vcm/ipi-conf-vsphere-check-vcm-commands.sh b/ci-operator/step-registry/ipi/conf/vsphere/check/vcm/ipi-conf-vsphere-check-vcm-commands.sh
      index 0e794eea809..55448d07a9c 100755
      --- a/ci-operator/step-registry/ipi/conf/vsphere/check/vcm/ipi-conf-vsphere-check-vcm-commands.sh
      +++ b/ci-operator/step-registry/ipi/conf/vsphere/check/vcm/ipi-conf-vsphere-check-vcm-commands.sh
      @@ -96,17 +96,17 @@ function getTypeInHeirarchy() {
       
         FOUND=0
         while [[ $FOUND == 0 ]]; do
      -    PARENTREF=$(jq --compact-output -r '.[] | select (.Name=="parent").Val' <<< "$LEVEL")
      +    PARENTREF=$(jq --compact-output -r '.[] | select (.name=="parent").val' <<< "$LEVEL")
           echo "$PARENTREF"
           if [[ ${PARENTREF} == "null" ]]; then
             log "unable to find ${DATATYPE}"
             return 1
           fi
      -    TEMPTYPE=$(jq -r .Type <<< "${PARENTREF}")
      +    TEMPTYPE=$(jq -r .type <<< "${PARENTREF}")
           log "type ${TEMPTYPE}"
      -    LEVEL=$(govc object.collect -json "${TEMPTYPE}":"$(jq --compact-output -r .Value <<< "${PARENTREF}")")
      +    LEVEL=$(govc object.collect -json "${TEMPTYPE}":"$(jq --compact-output -r .value <<< "${PARENTREF}")")
           if [[ ${TEMPTYPE} == "${DATATYPE}" ]]; then
      -      LEVEL_NAME=$(jq -r '.[] | select(.Name == "name") | .Val' <<< "${LEVEL}")
      +      LEVEL_NAME=$(jq -r '.[] | select(.name == "name") | .val' <<< "${LEVEL}")
             FOUND=1
           fi
         done
      @@ -134,11 +134,14 @@ function getDVSInfo() {
         while [[ $DVS_idx -lt ${elements} ]]; do
           log "DVS: ${DVS_idx}"
       
      -    parentDVS=$(jq --compact-output -r .elements[${DVS_idx}].Object.Config.DistributedVirtualSwitch /tmp/dvs.json)
      -    UUID_FORMATTED=$(govc object.collect -json "$(jq -r .Type <<< "${parentDVS}")":"$(jq -r .Value <<< "${parentDVS}")" | jq -r '.[] | select(.Name=="uuid") | .Val')
      +    parentDVS=$(jq --compact-output -r .elements[${DVS_idx}].Object.config.distributedVirtualSwitch /tmp/dvs.json)
      +    log "Parent DVS ${parentDVS}" 
      +
      +    UUID_FORMATTED=$(govc object.collect -json "$(jq -r .type <<< "${parentDVS}")":"$(jq -r .value <<< "${parentDVS}")" | jq -r '.[] | select(.name=="uuid") | .val')
      +    log "UUID: ${UUID_FORMATTED}"
       
           # determine the cluster where this dvs instance resides
      -    HOSTS=$(jq -r '.elements['${DVS_idx}'].Object.Host | .[].Value' /tmp/dvs.json)
      +    HOSTS=$(jq -r '.elements['${DVS_idx}'].Object.host | .[].value' /tmp/dvs.json)
           for HOST in ${HOSTS}; do
             _HOST=$(govc object.collect -json HostSystem:"${HOST}")
       // code placeholder
      

       

       

      issue created by splat-bot

      created from thread: https://redhat-internal.slack.com/archives/C06TSNLHU3W/p1730834850401889

              Unassigned Unassigned
              rhn-ocp-splat-service-account OpenShift SPLAT Service Account
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated: