Merge "Remove duplicate code in nova extend_start.sh"

docker/nova/nova-api/Dockerfile.j2

@@ -17,8 +17,8 @@ RUN apt-get install -y --no-install-recommends \
     {% endif %}
 {% endif %}
 
-COPY extend_start.sh /usr/local/bin/kolla_extend_start
-RUN chmod 755 /usr/local/bin/kolla_extend_start
+COPY extend_start.sh /usr/local/bin/kolla_nova_extend_start
+RUN chmod 755 /usr/local/bin/kolla_nova_extend_start
 
 {{ include_footer }}
 

docker/nova/nova-api/extend_start.sh

 #!/bin/bash
 
-if [[ ! -d "/var/log/kolla/nova" ]]; then
-    mkdir -p /var/log/kolla/nova
-fi
-if [[ $(stat -c %a /var/log/kolla/nova) != "755" ]]; then
-    chmod 755 /var/log/kolla/nova
-fi
-
 # Bootstrap and exit if KOLLA_BOOTSTRAP variable is set. This catches all cases
 # of the KOLLA_BOOTSTRAP variable being set, including empty.
 if [[ "${!KOLLA_BOOTSTRAP[@]}" ]]; then

docker/nova/nova-base/Dockerfile.j2

@@ -56,6 +56,8 @@ RUN chmod 750 /etc/sudoers.d \
 {% endif %}
 
 COPY extend_start.sh /usr/local/bin/kolla_extend_start
-RUN chmod 755 /usr/local/bin/kolla_extend_start
+
+RUN touch /usr/local/bin/kolla_nova_extend_start \
+    && chmod 755 /usr/local/bin/kolla_extend_start /usr/local/bin/kolla_nova_extend_start
 
 RUN usermod -a -G kolla nova

docker/nova/nova-base/extend_start.sh

@@ -6,3 +6,5 @@ fi
 if [[ $(stat -c %a /var/log/kolla/nova) != "755" ]]; then
     chmod 755 /var/log/kolla/nova
 fi
+
+source /usr/local/bin/kolla_nova_extend_start

docker/nova/nova-compute/Dockerfile.j2

@@ -52,8 +52,8 @@ RUN /var/lib/kolla/venv/bin/pip --no-cache-dir install --upgrade -c requirements
 {% endif %}
 
 COPY nova_compute_sudoers /etc/sudoers.d/nova_compute_sudoers
-COPY extend_start.sh /usr/local/bin/kolla_extend_start
-RUN chmod 755 /usr/local/bin/kolla_extend_start \
+COPY extend_start.sh /usr/local/bin/kolla_nova_extend_start
+RUN chmod 755 /usr/local/bin/kolla_nova_extend_start \
     && chmod 750 /etc/sudoers.d \
     && chmod 440 /etc/sudoers.d/nova_compute_sudoers \
     && rm -f /etc/machine-id

docker/nova/nova-compute/extend_start.sh

 #!/bin/bash
 
-
-if [[ ! -d "/var/log/kolla/nova" ]]; then
-    mkdir -p /var/log/kolla/nova
-fi
-if [[ $(stat -c %a /var/log/kolla/nova) != "755" ]]; then
-    chmod 755 /var/log/kolla/nova
-fi
-
 # Bootstrap and exit if KOLLA_BOOTSTRAP variable is set. This catches all cases
 # of the KOLLA_BOOTSTRAP variable being set, including empty.
 if [[ "${!KOLLA_BOOTSTRAP[@]}" ]]; then

docker/nova/nova-libvirt/Dockerfile.j2

@@ -30,8 +30,8 @@ RUN apt-get install -y --no-install-recommends \
 
 {% endif %}
 
-COPY extend_start.sh /usr/local/bin/kolla_extend_start
-RUN chmod 755 /usr/local/bin/kolla_extend_start \
+COPY extend_start.sh /usr/local/bin/kolla_nova_extend_start
+RUN chmod 755 /usr/local/bin/kolla_nova_extend_start \
     && useradd --user-group nova
 
-{{ include_footer }}
+{{ include_footer }}
\ No newline at end of file

docker/nova/nova-libvirt/extend_start.sh

@@ -12,10 +12,3 @@ fi
 if [[ $(lsmod | grep xenfs) ]]; then
     mount -t xenfs xenfs /proc/xen
 fi
-
-if [[ ! -d "/var/log/kolla/nova" ]]; then
-    mkdir -p /var/log/kolla/nova
-fi
-if [[ $(stat -c %a /var/log/kolla/nova) != "755" ]]; then
-    chmod 755 /var/log/kolla/nova
-fi