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

Request to add Argo resource health checks for Ionos resources #199

Closed
leroyshirto opened this issue Apr 2, 2024 · 2 comments · Fixed by #229
Closed

Request to add Argo resource health checks for Ionos resources #199

leroyshirto opened this issue Apr 2, 2024 · 2 comments · Fixed by #229
Assignees
Labels
enhancement New feature or request

Comments

@leroyshirto
Copy link

Please may you look into contributing resource health checks to the ArgoCD project for the resources provided by this provider?
At present if you use Argo to manage your crossplane resources, they will all show as healthy even if they are currently failing. Argo provides a mechanism for third parties to submit custom resource health checks, see https://argo-cd.readthedocs.io/en/stable/operator-manual/health/#way-2-contribute-a-custom-health-check

The Argo repo contains examples of health checks for other crossplane providers see https://github.com/argoproj/argo-cd/tree/master/resource_customizations

Thanks

@leroyshirto leroyshirto added the enhancement New feature or request label Apr 2, 2024
@cbeti-ionos cbeti-ionos self-assigned this Apr 24, 2024
@cbeti-ionos
Copy link
Member

Hello, this is now currently in the works, however if you'd like to have health checks for the ionoscloud resources available in Argocd in the meantime, the argo-cd ConfigMap can be edited as shown here.

I've also created this PR for ArgoCD usage examples where I add the health checks as a value for the argocd helm chart. The ionoscloud resources created here all have tracking and health checks enabled.

@cbeti-ionos cbeti-ionos linked a pull request May 14, 2024 that will close this issue
7 tasks
@cbeti-ionos
Copy link
Member

It seems bundling health checks for managed resources is not something other providers do. One of the reasons for this is that Claims or Composite resources don't inherit their health state.
Managed Resources are also fully covered by the script provided in the Crossplane docs so it's probably better if users include these as settings of their Argocd installation.

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

Successfully merging a pull request may close this issue.

2 participants