Skip to content

Make rabbitmq user and vhost name shorter in kuttl tests#1121

Open
amoralej wants to merge 1 commit into
openstack-k8s-operators:mainfrom
amoralej:fix-rmquser-kuttl
Open

Make rabbitmq user and vhost name shorter in kuttl tests#1121
amoralej wants to merge 1 commit into
openstack-k8s-operators:mainfrom
amoralej:fix-rmquser-kuttl

Conversation

@amoralej
Copy link
Copy Markdown

Some recent change (maybe [1a)], the string used in infra-operator for rabbitmquser creation has changed from
nova-kuttl-notification-transport-nova-notifications-user to rabbitmq-notifications-nova-notifications-user-nova-notifications which is longer that 63 char openshift limit, leading to error:

RabbitMQ user error occurred Secret "rabbitmq-user-rabbitmq-notifications-nova-notifications-user-nova-notifications" is invalid: metadata.labels: Invalid value: "rabbitmq-notifications-nova-notifications-user-nova-notifications": must be no more than 63 characters

While this may be understood as a bug, this patch is making vhost and user names shorter to unblock the kuttl test.

[1] openstack-k8s-operators/infra-operator@fcc8556#diff-627c3e6f14aec9dd16405a764cf2e60199490e15a399bd3df182df26f00e999d

@openshift-ci openshift-ci Bot requested review from SeanMooney and jamepark4 May 21, 2026 08:02
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 21, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: amoralej
Once this PR has been reviewed and has the lgtm label, please assign dprince 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

@centosinfra-prod-github-app
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/3b69658868794702a9c0d179edd78f60

openstack-meta-content-provider FAILURE in 13m 01s
⚠️ nova-operator-kuttl SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode-ceph SKIPPED Skipped due to failed job openstack-meta-content-provider

@amoralej amoralej force-pushed the fix-rmquser-kuttl branch from be8016a to 1be93f7 Compare May 21, 2026 08:22
@centosinfra-prod-github-app
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/8a6c646c573d46d0a85ed24454f10d61

openstack-meta-content-provider FAILURE in 11m 06s
⚠️ nova-operator-kuttl SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode-ceph SKIPPED Skipped due to failed job openstack-meta-content-provider

Some recent change (maybe [1a)], the string used in infra-operator for
rabbitmquser creation has changed from
`nova-kuttl-notification-transport-nova-notifications-user`
to `rabbitmq-notifications-nova-notifications-user-nova-notifications`
which is longer that 63 char openshift limit, leading to error:

RabbitMQ user error occurred Secret "rabbitmq-user-rabbitmq-notifications-nova-notifications-user-nova-notifications" is invalid: metadata.labels: Invalid value: "rabbitmq-notifications-nova-notifications-user-nova-notifications": must be no more than 63 characters

While this may be understood as a bug, this patch is making vhost and user names
shorter to unblock the kuttl test.

[1] openstack-k8s-operators/infra-operator@fcc8556#diff-627c3e6f14aec9dd16405a764cf2e60199490e15a399bd3df182df26f00e999d

Signed-off-by: Alfredo Moralejo <amoralej@redhat.com>
@amoralej amoralej force-pushed the fix-rmquser-kuttl branch from 1be93f7 to 45265a3 Compare May 21, 2026 09:12
@centosinfra-prod-github-app
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/446d42043b2b40abbdf2860f1181754d

✔️ openstack-meta-content-provider SUCCESS in 3h 34m 24s
nova-operator-kuttl FAILURE in 1h 05m 08s
✔️ nova-operator-tempest-multinode SUCCESS in 2h 29m 09s
✔️ nova-operator-tempest-multinode-ceph SUCCESS in 3h 10m 48s

@amoralej
Copy link
Copy Markdown
Author

fixed in infra-operator openstack-k8s-operators/infra-operator#583

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.

1 participant