openstack-k8s-operators / openstack-operator

Meta Operator for OpenStack
https://openstack-k8s-operators.github.io/openstack-operator/
Apache License 2.0
27 stars 76 forks source link

[validation] check glanceAPI names are valid #955

Closed stuggi closed 2 months ago

stuggi commented 2 months ago

The glanceAPI controller creates StatefulSet for glanceapi to run. This adds a StatefulSet pod's label "controller-revision-hash": "-" to the pod. The kubernetes label is restricted under 63 char and the revision hash is an int32, 10 chars + the hyphen. Which results in a default statefulset max len of 52 chars. The statefulset name also contain the glance name and the glanceAPI type + 2 hyphens. So the max len also need to be reduced bye the length of those.

Also the name of the created rabbitmq instance must match a lowercase RFC 1123.

Depends-On: openstack-k8s-operators/lib-common#532 Depends-On: https://github.com/openstack-k8s-operators/glance-operator/pull/587

Jira: https://issues.redhat.com/browse/OSPRH-8063

openshift-ci[bot] commented 2 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, gibizer, stuggi

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

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openstack-k8s-operators/openstack-operator/blob/main/OWNERS)~~ [abays,stuggi] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
softwarefactory-project-zuul[bot] commented 2 months ago

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://review.rdoproject.org/zuul/buildset/b9b15a1daf51423ab5338b343b263809

:heavy_check_mark: openstack-k8s-operators-content-provider SUCCESS in 3h 29m 09s :x: podified-multinode-edpm-deployment-crc FAILURE in 25m 22s :heavy_check_mark: cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 34m 42s :heavy_check_mark: cifmw-data-plane-adoption-osp-17-to-extracted-crc SUCCESS in 2h 32m 12s :x: openstack-operator-tempest-multinode FAILURE in 2h 29m 13s

openshift-ci[bot] commented 2 months ago

New changes are detected. LGTM label has been removed.

stuggi commented 2 months ago

rebased

openshift-ci[bot] commented 2 months ago

New changes are detected. LGTM label has been removed.

stuggi commented 2 months ago

once more a rebase

stuggi commented 2 months ago

/test openstack-operator-build-deploy-kuttl