In [1], two OpenStack clients were moved to the `DIST_PACKAGES` section
of the `stx-openstackclients` container image build recipe due to the
need to use packages patched exclusively for StarlingX:
Although this change served its purpose, it had a side-effect that
caused the main CLI, `openstack`, to stop recognizing other clients
installed via `PIP_PACKAGES`.
This occurred because once installed via `DIST_PACKAGES`, both
`cinderclient` and `openstackclient` started to reside in the
dist-packages/ directory, while all other clients continued to reside in
site-packages/. As `openstackclient` is the one responsible for the main
CLI, it stopped finding the other clients because its search directory
changed to dist-packages/.
Therefore, in this change, we chose to move all OpenStack clients to
DIST_PACKAGES. This way, it is guaranteed that the main CLI will
correctly identify the other OpenStack clients.
Test Plan:
PASS - Build stx-debian base image
PASS - Build stx-openstackclients container image
PASS - Upload/apply stx-openstack
PASS - Upload built image to the system's local registry and perform
a helm-override to use it in the OpenStack clients container
PASS - Run commands that had disappeared since the merge of [1], e.g.,
the ones provided by `heatclient`:
* openstack stack list
* openstack stack create
...
PASS - Remove/delete stx-openstack
Closes-Bug: 2039096
Co-authored-by: Lucas de Ataides <email address hidden>
Co-authored-by: Romulo Leite <email address hidden>
Change-Id: I61d653ceef07f99568a21b4ad6f6dceffed22ca1
Signed-off-by: Luan Nunes Utimura <email address hidden>
(cherry picked from commit 5bcbab406aacffeb89468d4f37624cb9cc403b10)
Reviewed: https:/ /review. opendev. org/c/starlingx /openstack- armada- app/+/898461 /opendev. org/starlingx/ openstack- armada- app/commit/ 3793f3ca4be75aa 103b06b85ecc59a 29b83b9faa
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit 3793f3ca4be75aa 103b06b85ecc59a 29b83b9faa
Author: Luan Nunes Utimura <email address hidden>
Date: Wed Oct 11 15:56:51 2023 -0300
stx- openstackclient s: Move all clients to DIST
In [1], two OpenStack clients were moved to the `DIST_PACKAGES` section lients` container image build recipe due to the
of the `stx-openstackc
need to use packages patched exclusively for StarlingX:
* python3- cinderclient; openstackclient .
* python3-
Although this change served its purpose, it had a side-effect that
caused the main CLI, `openstack`, to stop recognizing other clients
installed via `PIP_PACKAGES`.
This occurred because once installed via `DIST_PACKAGES`, both
`cinderclient` and `openstackclient` started to reside in the
dist-packages/ directory, while all other clients continued to reside in
site-packages/. As `openstackclient` is the one responsible for the main
CLI, it stopped finding the other clients because its search directory
changed to dist-packages/.
Therefore, in this change, we chose to move all OpenStack clients to
DIST_PACKAGES. This way, it is guaranteed that the main CLI will
correctly identify the other OpenStack clients.
[1] https:/ /review. opendev. org/c/starlingx /openstack- armada- app/+/896129
Test Plan: ients container image
PASS - Build stx-debian base image
PASS - Build stx-openstackcl
PASS - Upload/apply stx-openstack
PASS - Upload built image to the system's local registry and perform
a helm-override to use it in the OpenStack clients container
PASS - Run commands that had disappeared since the merge of [1], e.g.,
the ones provided by `heatclient`:
* openstack stack list
* openstack stack create
...
PASS - Remove/delete stx-openstack
Closes-Bug: 2039096
Co-authored-by: Lucas de Ataides <email address hidden> 9568a21b4ad6f6d ceffed22ca1 b89468d4f37624c b9cc403b10)
Co-authored-by: Romulo Leite <email address hidden>
Change-Id: I61d653ceef07f9
Signed-off-by: Luan Nunes Utimura <email address hidden>
(cherry picked from commit 5bcbab406aacffe