Skip to content

openstackclient: add novaclient and prometheus_client - #147

Open
lmiccini wants to merge 1 commit into
openstack-k8s-operators:mainfrom
lmiccini:openstackclient-add-novaclient-prometheus
Open

openstackclient: add novaclient and prometheus_client#147
lmiccini wants to merge 1 commit into
openstack-k8s-operators:mainfrom
lmiccini:openstackclient-add-novaclient-prometheus

Conversation

@lmiccini

Copy link
Copy Markdown
Contributor

The instanceha service (templates/instanceha/bin/instanceha.py in infra-operator) runs inside the openstackclient image and imports both novaclient and prometheus_client unconditionally at module load. In the tcib RPM-based image these were pulled in transitively, but the pip-based s2i image resolves strictly from the lockfile.

Add python-novaclient and prometheus_client to pythondeps.txt and regenerate requirements.lock.master. Versions (18.13.0 / 0.25.0) come from upper-constraints.txt.master; the resolution added no new transitive dependencies.

@openshift-ci

openshift-ci Bot commented Aug 27, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign elfiesmelfie for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

The instanceha service (templates/instanceha/bin/instanceha.py in
infra-operator) runs inside the openstackclient image and imports both
novaclient and prometheus_client unconditionally at module load. In the
tcib RPM-based image these were pulled in transitively, but the pip-based
s2i image resolves strictly from the lockfile.

Add python-novaclient and prometheus_client to pythondeps.txt and
regenerate requirements.lock.master. Versions (18.13.0 / 0.25.0) come
from upper-constraints.txt.master; the resolution added no new transitive
dependencies.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants