Skip to content

Enable volume-based Octavia amphora on uni07eta - #786

Merged
centosinfra-prod-github-app[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
zahlabut:enable-uni07eta-volume-based-octavia-amphora
Jul 24, 2026
Merged

Enable volume-based Octavia amphora on uni07eta#786
centosinfra-prod-github-app[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
zahlabut:enable-uni07eta-volume-based-octavia-amphora

Conversation

@zahlabut

Copy link
Copy Markdown
Contributor

Enable Octavia volume_cinder_driver on uni07eta so amphorae boot from Cinder volumes.

@zahlabut
zahlabut requested a review from a team as a code owner July 16, 2026 19:10
@openshift-ci
openshift-ci Bot requested review from abays and fultonj July 16, 2026 19:10
@openshift-ci

openshift-ci Bot commented Jul 16, 2026

Copy link
Copy Markdown

Hi @zahlabut. Thanks for your PR.

I'm waiting for a openstack-k8s-operators member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

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.

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

Need ci-ops approval

@openshift-ci openshift-ci Bot added the lgtm label Jul 20, 2026
@fultonj fultonj added depends-on-ciops Tag to help ci-ops automation catch changes which might break unijobs and removed needs-ok-to-test labels Jul 20, 2026
@imatza-rh

Copy link
Copy Markdown

/lgtm

@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Unable to update github.com/openstack-k8s-operators/architecture

@imatza-rh

Copy link
Copy Markdown

recheck

@imatza-rh

Copy link
Copy Markdown

/lgtm
/approve

@openshift-ci openshift-ci Bot added the lgtm label Jul 23, 2026
@imatza-rh

Copy link
Copy Markdown

@zahlabut seems this change needs a rebase. can you check it?

@zahlabut
zahlabut force-pushed the enable-uni07eta-volume-based-octavia-amphora branch from 7ad525e to 818eb5b Compare July 23, 2026 12:42
@openshift-ci openshift-ci Bot removed the lgtm label Jul 23, 2026
@zahlabut

Copy link
Copy Markdown
Contributor Author

Hi @imatza-rh rebase is done.

@imatza-rh

Copy link
Copy Markdown

/approve
/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Jul 23, 2026

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

@openshift-ci

openshift-ci Bot commented Jul 24, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, imatza-rh, zahlabut

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

@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app
centosinfra-prod-github-app Bot merged commit bcf1f12 into openstack-k8s-operators:main Jul 24, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved depends-on-ciops Tag to help ci-ops automation catch changes which might break unijobs lgtm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants