Skip to content

Commit 978a286

Browse files
seunghun1eemnasiadka
authored andcommitted
Fix symlinking of openstack-base source
Original symlinking works fine with a tarball source for the requirements repo. However, when user supplies git type source of it, we cannot guarantee that the source direcotry name contains ``requirements``. This patch fixes it by moving plugins directory outside of openstack-base directory, which ultimately allow us to use simpler symlinking as openstack-base-archive would be the only directory inside. Closes-Bug: #2074075 Change-Id: I5a9162a30b23cfabb64dabe5c12c226fa8627abd
1 parent 082d90c commit 978a286

File tree

2 files changed

+11
-6
lines changed

2 files changed

+11
-6
lines changed

docker/openstack-base/Dockerfile.j2

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
135135
'pika',
136136
'prettytable',
137137
'psutil',
138-
'/pycadf',
138+
'/plugins/pycadf*',
139139
'pyinotify',
140140
'pymysql',
141141
'pyngus',
@@ -182,13 +182,12 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
182182
%}
183183

184184
ADD openstack-base-archive /openstack-base-source
185-
ADD plugins-archive /openstack-base-source
185+
ADD plugins-archive /
186186

187-
RUN ln -s openstack-base-source/plugins/* /pycadf \
188-
&& mkdir -p /etc/pycadf \
189-
&& cp /pycadf/etc/pycadf/* /etc/pycadf/
187+
RUN mkdir -p /etc/pycadf \
188+
&& cp /plugins/pycadf*/etc/pycadf/* /etc/pycadf/
190189

191-
RUN ln -s openstack-base-source/*requirements* /requirements \
190+
RUN ln -s openstack-base-source/* /requirements \
192191

193192
{# NOTE(mnasiadka): Remove ovs from upper-constraints.txt because python3-openvswitch
194193
is usually newer than UC entry and older version would get installed
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
---
2+
fixes:
3+
- |
4+
Fixed a bug preventing image builds when requirements are provided from
5+
git type source.
6+
`LP#2074075 <https://bugs.launchpad.net/kolla/+bug/2074075>`__

0 commit comments

Comments
 (0)