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

Inconsistent Health of cert-manger.io/Certificate #16523

Closed
3 tasks done
chr15murray opened this issue Dec 4, 2023 · 2 comments
Closed
3 tasks done

Inconsistent Health of cert-manger.io/Certificate #16523

chr15murray opened this issue Dec 4, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@chr15murray
Copy link
Contributor

Checklist:

  • I've searched in the docs and FAQ for my answer: https://bit.ly/argocd-faq.
  • I've included steps to reproduce the bug.
  • I've pasted the output of argocd version.

Describe the bug

The health of a cert-manager.io/certificate is not always consistent within ArgoCD when Issuing and Ready states both exist. This occurs because the Ready: False state may come first or second in the status conditions.

To Reproduce

Create a certificate manifest for cert-manager to handle. Most of the time it will show 'progressing' in ArgoCD but occasionally it will show a degraded state.

Expected behavior

When a status for 'Issuing' exists I expect the health to show 'progressing' instead of degraded. This can occur if the 'Ready' status appears first in the statuses.

Example Degraded

status:
conditions:
- lastTransitionTime: '2023-12-01T11:15:28Z'
message: Issuing certificate as Secret does not exist
observedGeneration: 1
reason: DoesNotExist
status: 'False'
type: Ready
- lastTransitionTime: '2023-12-01T11:15:28Z'
message: Issuing certificate as Secret does not exist
observedGeneration: 1
reason: DoesNotExist
status: 'True'
type: Issuing
nextPrivateKeySecretName: certtest-81-vg977
Example Issuing

status:
conditions:
- lastTransitionTime: '2023-12-01T11:15:28Z'
message: Issuing certificate as Secret does not exist
observedGeneration: 1
reason: DoesNotExist
status: 'True'
type: Issuing
- lastTransitionTime: '2023-12-01T11:15:28Z'
message: Issuing certificate as Secret does not exist
observedGeneration: 1
reason: DoesNotExist
status: 'False'
type: Ready
nextPrivateKeySecretName: certtest-82-f4r27

Screenshots

Version

Paste the output from `argocd version` here.

I do not have api access to ArgCD to run this. Version is v2.9.2+c5ea5c4
Logs

Paste any relevant application logs here.
@chr15murray chr15murray added the bug Something isn't working label Dec 4, 2023
@chr15murray
Copy link
Contributor Author

PR to resolve this here: #16520

ishitasequeira pushed a commit that referenced this issue Dec 4, 2023
…ssue #16523) (#16520)

* Update cert-manager.opcertificate health.lua

Signed-off-by: Chris Murray <[email protected]>

* adding test case for cert issuing

Signed-off-by: Chris Murray <[email protected]>

* fixing typo

Signed-off-by: Chris Murray <[email protected]>

---------

Signed-off-by: Chris Murray <[email protected]>
vladfr pushed a commit to vladfr/argo-cd that referenced this issue Dec 13, 2023
…ssue argoproj#16523) (argoproj#16520)

* Update cert-manager.opcertificate health.lua

Signed-off-by: Chris Murray <[email protected]>

* adding test case for cert issuing

Signed-off-by: Chris Murray <[email protected]>

* fixing typo

Signed-off-by: Chris Murray <[email protected]>

---------

Signed-off-by: Chris Murray <[email protected]>
tesla59 pushed a commit to tesla59/argo-cd that referenced this issue Dec 16, 2023
…ssue argoproj#16523) (argoproj#16520)

* Update cert-manager.opcertificate health.lua

Signed-off-by: Chris Murray <[email protected]>

* adding test case for cert issuing

Signed-off-by: Chris Murray <[email protected]>

* fixing typo

Signed-off-by: Chris Murray <[email protected]>

---------

Signed-off-by: Chris Murray <[email protected]>
@blakepettersson
Copy link
Member

Resolved with #16520

JulienFuix pushed a commit to JulienFuix/argo-cd that referenced this issue Feb 6, 2024
…ssue argoproj#16523) (argoproj#16520)

* Update cert-manager.opcertificate health.lua

Signed-off-by: Chris Murray <[email protected]>

* adding test case for cert issuing

Signed-off-by: Chris Murray <[email protected]>

* fixing typo

Signed-off-by: Chris Murray <[email protected]>

---------

Signed-off-by: Chris Murray <[email protected]>
lyda pushed a commit to lyda/argo-cd that referenced this issue Mar 28, 2024
…ssue argoproj#16523) (argoproj#16520)

* Update cert-manager.opcertificate health.lua

Signed-off-by: Chris Murray <[email protected]>

* adding test case for cert issuing

Signed-off-by: Chris Murray <[email protected]>

* fixing typo

Signed-off-by: Chris Murray <[email protected]>

---------

Signed-off-by: Chris Murray <[email protected]>
Signed-off-by: Kevin Lyda <[email protected]>
Hariharasuthan99 pushed a commit to AmadeusITGroup/argo-cd that referenced this issue Jun 16, 2024
…ssue argoproj#16523) (argoproj#16520)

* Update cert-manager.opcertificate health.lua

Signed-off-by: Chris Murray <[email protected]>

* adding test case for cert issuing

Signed-off-by: Chris Murray <[email protected]>

* fixing typo

Signed-off-by: Chris Murray <[email protected]>

---------

Signed-off-by: Chris Murray <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants