search

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

[ovndbs] For cert setup do not wait for dbs to be ready #793

Closed karelyatin closed 4 months ago

karelyatin commented 4 months ago

Currently when TLS is enabled OVNDBClusters are created first without Certs, wait for those to be ready, create certs and update ovndbclusters with tls enabled.

This causes issues as with this even with TLS enabled OVN raft is setup with local-address as non-ssl and later scale up do not work due to this. To fix this changing the conditions to create certs when TLS is enabled like other services.

Related-Issue: OSPRH-6899

openshift-ci[bot] commented 4 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: karelyatin, olliewalsh

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)~~ [olliewalsh] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
booxter commented 4 months ago

Thank you, I am late but this LGTM.