Skip to content

Added auto-detect feature for controller version #29

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 2, 2020
Merged

Added auto-detect feature for controller version #29

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 14 additions & 8 deletions recipes/get_node_extrainfo.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@
################################################################################
# Gets extra node info from an applications node list
#
# NB; review SOURCE_API_NODEINFO setup, for version of controller
#
# Useful output lines for debug;
# echo "lines: ${#app_list_array[@]} | entries: ${#app_array[@]}"
# for key in "${!app_array[@]}"; do echo "key : $key"; echo "value : ${app_array[$key]}"; done
Expand Down Expand Up @@ -42,12 +40,6 @@ usage () {
################################################################################
# Setup Variables, etc
################################################################################
# RESTUI API call
# TODO: Find exact version api calls changed
# NB; this api call is valid on > mid-4.5 controller...
SOURCE_API_NODEINFO="/controller/restui/v1/nodes/list/health/ids"
# ...before which it was
#SOURCE_API_NODEINFO="/controller/restui/nodes/list/health/ids"

# Set default input params
LEVEL="0" #This means -A is mandatory unless -L is changed
Expand Down Expand Up @@ -129,6 +121,19 @@ runChecks () {
if [[ ${LEVEL} == "0" && -z ${APP_ID} ]]; then echo "No -A param passed for -L=0."; usage; exit 1; fi
}

# Gets the controllers version, to switch the restui API format
# TODO: Find exact version api calls changed, assumed 4.5.15 at present
getControllerVer () {
VER=$(../act.sh -E ${ENVIRONMENT} controller version)
if [[ -z ${VER} ]]; then echo "Unable to get controllers version."; exit 1; fi
# If controller version is less than 4.5.15, then use old restui API format
if [[ ${VER} =~ ^4.5.([0-9]*) && ${BASH_REMATCH[1]} -lt 15 ]]; then
SOURCE_API_NODEINFO="/controller/restui/nodes/list/health/ids"
else
SOURCE_API_NODEINFO="/controller/restui/v1/nodes/list/health/ids"
fi
}

# Setup our data sources
setupSources () {
# Filenames for input, if SOURCE=1
Expand Down Expand Up @@ -348,6 +353,7 @@ getNodeInfo () {

# Lets get ready
runChecks
getControllerVer
setupSources

# Output header line
Expand Down