-
Notifications
You must be signed in to change notification settings - Fork 140
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
Add todo for health state checks #354
Conversation
The following ISSUES will be created once got merged. If you want me to skip creating the issue, you can use Details1. Check health state of CronJob resourceThis was created by todo plugin since "TODO:" was found in 285525c when #354 was merged. cc: @nakabonne.2. Check health state of PersistentVolume resourceThis was created by todo plugin since "TODO:" was found in 285525c when #354 was merged. cc: @nakabonne. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -286,6 +290,12 @@ func determineReplicaSetHealth(obj *unstructured.Unstructured) (status model.Kub | |||
return | |||
} | |||
|
|||
// TODO: Check health state of CronJob resource | |||
func determineCronJobHealth(obj *unstructured.Unstructured) (status model.KubernetesResourceState_HealthStatus, desc string) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
obj
is unused in determineCronJobHealth
@@ -434,6 +444,12 @@ func determineSecretHealth(obj *unstructured.Unstructured) (status model.Kuberne | |||
return | |||
} | |||
|
|||
// TODO: Check health state of PersistentVolume resource | |||
func determinePersistentVolumeHealth(obj *unstructured.Unstructured) (status model.KubernetesResourceState_HealthStatus, desc string) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
obj
is unused in determinePersistentVolumeHealth
/approve |
Code coverage for golang is
|
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: