Skip to content

Commit

Permalink
Fix symlinking of openstack-base source
Browse files Browse the repository at this point in the history
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
  • Loading branch information
seunghun1ee authored and mnasiadka committed Aug 29, 2024
1 parent 082d90c commit 978a286
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 6 deletions.
11 changes: 5 additions & 6 deletions docker/openstack-base/Dockerfile.j2
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
'pika',
'prettytable',
'psutil',
'/pycadf',
'/plugins/pycadf*',
'pyinotify',
'pymysql',
'pyngus',
Expand Down Expand Up @@ -182,13 +182,12 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
%}

ADD openstack-base-archive /openstack-base-source
ADD plugins-archive /openstack-base-source
ADD plugins-archive /

RUN ln -s openstack-base-source/plugins/* /pycadf \
&& mkdir -p /etc/pycadf \
&& cp /pycadf/etc/pycadf/* /etc/pycadf/
RUN mkdir -p /etc/pycadf \
&& cp /plugins/pycadf*/etc/pycadf/* /etc/pycadf/

RUN ln -s openstack-base-source/*requirements* /requirements \
RUN ln -s openstack-base-source/* /requirements \

{# NOTE(mnasiadka): Remove ovs from upper-constraints.txt because python3-openvswitch
is usually newer than UC entry and older version would get installed
Expand Down
6 changes: 6 additions & 0 deletions releasenotes/notes/fix-git-build-23c8fe6c4b0d92c4.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
fixes:
- |
Fixed a bug preventing image builds when requirements are provided from
git type source.
`LP#2074075 <https://bugs.launchpad.net/kolla/+bug/2074075>`__

0 comments on commit 978a286

Please sign in to comment.