openstack/senlin
Code Issues Proposed changes

Do not for force-reinstall when upgrading packages

Browse Source 
Using fore-reinstall with upgrade will cause all
dependencies to always be reinstalled. This can
cause issues when other components install Senlin
as a dependency.

In this case we are causing issues because we are
re-installing certify, a file that is modified by
devstack after installation to make sure that the
system has the appropriate ssl certificates.

See the function fix_system_ca_bundle_path in devstack
for more information.

Change-Id: Ibb6578f45f9277f97ca61b60714c8ce2b4eacb61
Closes-Bug: #1769541
This commit is contained in:
Erik Olof Gunnar Andersson 2018-05-06 17:44:28 -07:00
parent a7fea71b9e
commit 4d298c6f2e
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
devstack/lib/senlin
View File

@ -192,7 +192,7 @@ function install_senlinclient {
git_clone $SENLINCLIENT_REPO $SENLINCLIENT_DIR $SENLINCLIENT_BRANCH
setup_develop $SENLINCLIENT_DIR
else
pip_install --upgrade --force-reinstall python-senlinclient
pip_install --upgrade python-senlinclient
fi
}
@ -209,7 +209,7 @@ function install_senlin_dashboard {
setup_develop $SENLIN_DASHBOARD_DIR
mv $SENLIN_DASHBOARD_DIR/_test-requirements.txt $SENLIN_DASHBOARD_DIR/test-requirements.txt
else
pip_install --upgrade --force-reinstall senlin-dashboard
pip_install --upgrade senlin-dashboard
fi
}