Skip to content

fix(healthcheck): adopt upstream ArgoCD NNCP health check - #31

Merged
aharivel merged 1 commit into
validatedpatterns-sandbox:mainfrom
oliashish:fix/nncp-healthcheck-upstream
Aug 18, 2026
Merged

fix(healthcheck): adopt upstream ArgoCD NNCP health check#31
aharivel merged 1 commit into
validatedpatterns-sandbox:mainfrom
oliashish:fix/nncp-healthcheck-upstream

Conversation

@oliashish

Copy link
Copy Markdown
Collaborator

Replace custom NNCP healthcheck with the ArgoCD v3.4+ built-in version, adding Ignored->Suspended mapping and condition.reason in messages.

Jira: OSPRH-33119

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com
Signed-by: oliashish3@gmail.com

Replace custom NNCP healthcheck with the ArgoCD v3.4+ built-in version, adding Ignored->Suspended mapping and condition.reason in messages.

Jira: OSPRH-33119

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-by: oliashish3@gmail.com
@oliashish
oliashish requested review from aharivel and cjeanner August 14, 2026 14:55

@aharivel aharivel left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, clean adoption of upstream Argocd

@aharivel
aharivel merged commit 6c034ec into validatedpatterns-sandbox:main Aug 18, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants