Skip to content

Add Healthy status condition#262

Merged
stefanprodan merged 1 commit intomainfrom
healthy-condition
Feb 2, 2021
Merged

Add Healthy status condition#262
stefanprodan merged 1 commit intomainfrom
healthy-condition

Conversation

@stefanprodan
Copy link
Copy Markdown
Member

@stefanprodan stefanprodan commented Feb 2, 2021

@stefanprodan stefanprodan added enhancement New feature or request area/kstatus Health checking related issues and pull requests labels Feb 2, 2021
@stefanprodan stefanprodan requested a review from hiddeco February 2, 2021 11:17
hiddeco
hiddeco reviewed Feb 2, 2021
View reviewed changes
Comment thread api/v1beta1/kustomization_types.go Outdated
@stefanprodan stefanprodan force-pushed the healthy-condition branch 2 times, most recently from 9fd9291 to adca92c Compare February 2, 2021 12:58
@stefanprodan
Add Healthy status condition
5e17dd4 
- record the last health assessment result in a dedicated status condition
- use the condition status when issuing events to prevent notifications spam

Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
@stefanprodan stefanprodan merged commit d7eeea3 into main Feb 2, 2021
@stefanprodan stefanprodan deleted the healthy-condition branch February 2, 2021 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hiddeco hiddeco hiddeco approved these changes

Assignees

No one assigned

Labels

area/kstatus Health checking related issues and pull requests enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Health check event is emitted in every reconciliation cycle

2 participants

@stefanprodan @hiddeco
⚡