Skip to content

[heat] Update containers to exclude unneeded requirements - #150

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
elfiesmelfie:heat_exclude_requirements
Aug 31, 2026
Merged

[heat] Update containers to exclude unneeded requirements#150
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
elfiesmelfie:heat_exclude_requirements

Conversation

@elfiesmelfie

Copy link
Copy Markdown
Contributor

Remove the requirements that were previously excluded by the rpm package

Assisted-by: Claude Sonnet

Remove the requirements that were previously excluded by the rpm package
Assisted-by: Claude Sonnet

@yazug yazug left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm /approve

python-blazarclient
python-zunclient
python-magnumclient
python-mistralclient

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Though this is aligining with what we used to do in rdo packaging, it's bit weird that we're including the resource plugins that depend on these clients. When a user would do resource type show those types would show up but won't work in a template.

We should exclude those resource plugins folders like [1] as well in a followup.
[1] https://github.com/openstack/heat/tree/master/heat/engine/resources/openstack/mistral

@rebtoor rebtoor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve
/lgtm

@openshift-ci

openshift-ci Bot commented Aug 31, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rebtoor

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

The pull request process is described 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

@openshift-merge-bot
openshift-merge-bot Bot merged commit 5380147 into openstack-k8s-operators:main Aug 31, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants