forked from argoproj/argo-cd
-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix(health): only consider non-empty health checks (argoproj#20232)
* fix(health): only consider non-empty health checks For wildcard health checks, only consider wildcards with a non-empty health check. Fixes argoproj#16905 (at least partially). Signed-off-by: Blake Pettersson <[email protected]> * test: renaming test case for clarity Signed-off-by: Blake Pettersson <[email protected]> * refactor: add clarity as to what the function is supposed to do Signed-off-by: Blake Pettersson <[email protected]> * Update docs/operator-manual/health.md Co-authored-by: Michael Crenshaw <[email protected]> Signed-off-by: Blake Pettersson <[email protected]> * test: add test case for `*/*` override with empty healthcheck Signed-off-by: Blake Pettersson <[email protected]> --------- Signed-off-by: Blake Pettersson <[email protected]> Co-authored-by: Michael Crenshaw <[email protected]>
- Loading branch information
1 parent
2f6b6ad
commit 5ba2405
Showing
3 changed files
with
63 additions
and
18 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters