Skip to content

Commit 9860876

Browse files
author
Amrith Kumar
committed
perform install_infra sooner in stack.sh
The install_infra() call needs to be done earlier since pip_install_gr() depends on it. Also the fact that python module names are supposed to be lower case but some use camel case is a problem (for example with XenAPI). Change-Id: I7012d77134fa0d9c746d87e837934e7dcb337e5c Closes-Bug: #1441820
1 parent 60996b1 commit 9860876

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

inc/python

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ function pip_install {
137137
# get_from_global_requirements <package>
138138
function get_from_global_requirements {
139139
local package=$1
140-
local required_pkg=$(grep -h ^${package} $REQUIREMENTS_DIR/global-requirements.txt | cut -d\# -f1)
140+
local required_pkg=$(grep -i -h ^${package} $REQUIREMENTS_DIR/global-requirements.txt | cut -d\# -f1)
141141
if [[ $required_pkg == "" ]]; then
142142
die $LINENO "Can't find package $package in requirements"
143143
fi

stack.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -714,6 +714,9 @@ fi
714714
# Extras Pre-install
715715
# ------------------
716716

717+
# Install required infra support libraries
718+
install_infra
719+
717720
# Phase: pre-install
718721
run_phase stack pre-install
719722

@@ -733,9 +736,6 @@ fi
733736

734737
echo_summary "Installing OpenStack project source"
735738

736-
# Install required infra support libraries
737-
install_infra
738-
739739
# Install Oslo libraries
740740
install_oslo
741741

0 commit comments

Comments
 (0)