Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release Ready status should be aggregation of readines of object deployed within a release #214

Open
pedjak opened this issue Mar 11, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@pedjak
Copy link

pedjak commented Mar 11, 2024

What problem are you facing?

Ready status of a Helm release becomes equal to true as soon as Helm creates objects that are a part of the given release. However, this does not necessary means that the workload is really ready, because images need to be pulled from registries, pods scheduled and started, etc.

In a complex compositions, quite often deployment of another chart or objects depends on readiness of their dependencies.
Semantically would make sense to mark a Helm release ready when the enclosed resources are really ready. Making the equivalence with being deployed is imprecise.

Even today, the initial readiness could be emulated by setting .spec.forProvider.wait to true. In that case, the provider is going to block and wait until Helm checks and returns back when all resources become ready.

Given that wait:true is blocking the reconcile loop, it has impact on other releases sitting in the queue and waiting to be reconciled. Also, once readiness is detected, we do not try to perform any future checks - a Helm release remains ready even if the workload crashes and does not recover at all.

How could Crossplane help solve your problem?

Helm release should become ready only when all deployed objects part of the release becomes ready. This check should be performed on every release reconciliation and ready condition should be updated accordingly.

@pedjak pedjak added the enhancement New feature or request label Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant