kubernetes-sigs / cluster-api-operator

Home for Cluster API Operator, a subproject of sig-cluster-lifecycle
https://cluster-api-operator.sigs.k8s.io
Apache License 2.0
146 stars 60 forks source link

Capi-operator deployment needs proper ready signal for helm #537

Closed dtzar closed 3 weeks ago

dtzar commented 1 month ago

What steps did you take and what happened: Deployed CAPI-operator see #532 and we discovered that part of the reason why the provider was failing to install was due to capi-operator being started, but not fully ready for the provider and then the provider fails to install. This was resolved by manually putting in a sleep job in helm.

What did you expect to happen: The capi-operator pods/services have a proper health/readiness check so that providers don't run into failure cases.

Environment:

/kind bug

Related to #532

k8s-ci-robot commented 1 month ago

This issue is currently awaiting triage.

If CAPI Operator contributors determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
dtzar commented 3 weeks ago

Related to #406

dtzar commented 3 weeks ago

Closing as I think the --wait covers this