Skip to content

Commit

Permalink
Restore redfish and mixed mode for BMC's (openshift-metal3#1077)
Browse files Browse the repository at this point in the history
This restores redfish as the default for all users, and mixed mode as
the default for CI.
  • Loading branch information
stbenjam authored Jul 31, 2020
1 parent 4d3ab5a commit ddc7140
Showing 1 changed file with 7 additions and 11 deletions.
18 changes: 7 additions & 11 deletions common.sh
Original file line number Diff line number Diff line change
Expand Up @@ -229,19 +229,15 @@ fi

export OPENSHIFT_RELEASE_TAG=$(echo $OPENSHIFT_RELEASE_IMAGE | sed -E 's/[[:alnum:]\/.-]*release.*://')

# FIXME(stbenjam): Temporarily use ipmi as sushy tools currently has the
# wrong version in CI builds of ironic images.
export BMC_DRIVER=${BMC_DRIVER:-ipmi}

# Use "ipmi" for 4.3 as it didn't support redfish, for other versions
# use "redfish", unless its CI where we use "mixed"
#if [[ "$OPENSHIFT_VERSION" == "4.3" ]]; then
# export BMC_DRIVER=${BMC_DRIVER:-ipmi}
#elif [[ -z "$OPENSHIFT_CI" ]]; then
# export BMC_DRIVER=${BMC_DRIVER:-redfish}
#else
# export BMC_DRIVER=${BMC_DRIVER:-mixed}
#fi
if [[ "$OPENSHIFT_VERSION" == "4.3" ]]; then
export BMC_DRIVER=${BMC_DRIVER:-ipmi}
elif [[ -z "$OPENSHIFT_CI" ]]; then
export BMC_DRIVER=${BMC_DRIVER:-redfish}
else
export BMC_DRIVER=${BMC_DRIVER:-mixed}
fi

# Both utils.sh and 04_setup_ironic.sh use this log file, so set the
# name one time. Users should not override this.
Expand Down

0 comments on commit ddc7140

Please sign in to comment.