openstack/kolla
Code Issues Proposed changes

Fix if $user is root at extend_start

Browse Source 
In ubuntu is not evaluating correctly
if $user == root check at extend_start.

Changing to "$(whoami)" == 'root' fixes the issue

Change-Id: I296a1f98764cdfa09650c483bc7beba53e6bdba4
Closes-Bug: #1690360
This commit is contained in:
egonzalez90 2017-05-12 12:56:13 +01:00 committed by Eduardo Gonzalez
parent 2e43891838
commit 45c54d7959
4 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
docker/cinder/cinder-api/extend_start.sh
View File

@ -9,7 +9,7 @@ if [[ "${!KOLLA_BOOTSTRAP[@]}" ]]; then
fi
# Assume the service runs on top of Apache when user is root
if [[ $USER == 'root' ]]; then
if [[ "$(whoami)" == 'root' ]]; then
# NOTE(pbourke): httpd will not clean up after itself in some cases which
# results in the container not being able to restart. (bug #1489676, 1557036)
if [[ "${KOLLA_BASE_DISTRO}" =~ debian|ubuntu ]]; then

2
docker/heat/heat-base/extend_start.sh
View File

@ -8,7 +8,7 @@ if [[ $(stat -c %a /var/log/kolla/heat) != "755" ]]; then
fi
# Assume the service runs on top of Apache when user is root
if [[ $USER == 'root' ]]; then
if [[ "$(whoami)" == 'root' ]]; then
# NOTE(pbourke): httpd will not clean up after itself in some cases which
# results in the container not being able to restart. (bug #1489676, 1557036)
if [[ "${KOLLA_BASE_DISTRO}" =~ debian|ubuntu ]]; then

2
docker/nova/nova-api/extend_start.sh
View File

@ -10,7 +10,7 @@ if [[ "${!KOLLA_BOOTSTRAP[@]}" ]]; then
fi
# Assume the service runs on top of Apache when user is root
if [[ $USER == 'root' ]]; then
if [[ "$(whoami)" == 'root' ]]; then
# NOTE(pbourke): httpd will not clean up after itself in some cases which
# results in the container not being able to restart. (bug #1489676, 1557036)
if [[ "${KOLLA_BASE_DISTRO}" =~ debian|ubuntu ]]; then

2
docker/zaqar/extend_start.sh
View File

@ -8,7 +8,7 @@ if [[ $(stat -c %a /var/log/kolla/zaqar) != "755" ]]; then
fi
# Assume the service runs on top of Apache when user is root
if [[ $USER == 'root' ]]; then
if [[ "$(whoami)" == 'root' ]]; then
# NOTE(pbourke): httpd will not clean up after itself in some cases which
# results in the container not being able to restart. (bug #1489676, 1557036)
if [[ "${KOLLA_BASE_DISTRO}" =~ debian|ubuntu ]]; then