Skip to content

Bump manila and fix functional tests#1450

Merged
openshift-merge-bot[bot] merged 2 commits into
openstack-k8s-operators:mainfrom
stuggi:bump_manila
May 22, 2025
Merged

Bump manila and fix functional tests#1450
openshift-merge-bot[bot] merged 2 commits into
openstack-k8s-operators:mainfrom
stuggi:bump_manila

Conversation

@stuggi

@stuggi stuggi commented May 22, 2025

Copy link
Copy Markdown
Contributor
  • manila introduced validation webhooks check in [1] for required fields. This adds thise fields in the manila template.
  • fixes heat password selector from AuthEncryptionKey -> authEncryptionKey
  • openstackclient does not (yet) have an Enabled parameter

[1] openstack-k8s-operators/manila-operator#418

@openshift-ci openshift-ci Bot requested review from frenzyfriday and rabi May 22, 2025 08:38
@stuggi stuggi requested review from abays, dprince and fmount and removed request for frenzyfriday and rabi May 22, 2025 08:39
stuggi added 2 commits May 22, 2025 10:53
@stuggi
bump manila-operator reference
ae032e1
@stuggi
Fix functional tests
38173d7 
* manila introduced validation webhooks check in [1] for required
fields. This adds thise fields in the manila template.
* fixes heat password selector from AuthEncryptionKey -> authEncryptionKey
* openstackclient does not (yet) have an Enabled parameter

[1] openstack-k8s-operators/manila-operator#418

Signed-off-by: Martin Schuppert <mschuppert@redhat.com>
fmount
fmount approved these changes May 22, 2025
View reviewed changes

@fmount fmount 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

@openshift-ci openshift-ci Bot added the lgtm label May 22, 2025
@openshift-ci

openshift-ci Bot commented May 22, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fmount, stuggi

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

@stuggi stuggi mentioned this pull request May 22, 2025
Merged
@fmount

fmount commented May 22, 2025

Copy link
Copy Markdown
Contributor

/test openstack-operator-build-deploy-kuttl

@stuggi

stuggi commented May 22, 2025

Copy link
Copy Markdown
Contributor Author

only must-gather failed in kuttl job at the end. overriding it.

@stuggi

stuggi commented May 22, 2025

Copy link
Copy Markdown
Contributor Author

/override ci/prow/openstack-operator-build-deploy-kuttl

@openshift-ci

openshift-ci Bot commented May 22, 2025

Copy link
Copy Markdown
Contributor

@stuggi: Overrode contexts on behalf of stuggi: ci/prow/openstack-operator-build-deploy-kuttl

Details

In response to this:

/override ci/prow/openstack-operator-build-deploy-kuttl

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-merge-bot openshift-merge-bot Bot merged commit fe46370 into openstack-k8s-operators:main May 22, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fmount fmount fmount approved these changes

@abays abays Awaiting requested review from abays

@dprince dprince Awaiting requested review from dprince

Assignees

@fmount fmount

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@stuggi @fmount