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

Update HEALTH_CHECKS.md #53

Merged
merged 1 commit into from
Oct 18, 2024
Merged

Conversation

eburhansjah
Copy link

@eburhansjah eburhansjah commented Oct 18, 2024

Summary

  • What changes are proposed in this pull request?
    Updated Health_Checks.md: Included details on formatting of error labels + real life example

Scope and Impact

  • API Changes? No
  • Should any users or specific teams be notified of breaking changes?

GitHub Issue

How was this Pull-Request Tested and Validated?

  • Steps used to test and validate the changes. Commands and additional content is welcomed.
  • If not applicable mark as N/A.

Pull-Request Reminders

  • Does the Autopilot Readme require updates?

    • Yes or No -- if yes, were they made? No
  • Are there any new software dependencies introduced to this Pull-Request?

    • Yes or No -- if yes, what are they? No

Include details on formatting of error labels + real life example

Signed-off-by: Ellen Burhansjah <[email protected]>
@cmisale cmisale self-requested a review October 18, 2024 15:57
Copy link
Collaborator

@cmisale cmisale left a comment

Choose a reason for hiding this comment

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

That's great, thanks a lot for helping improving the documentation!

@cmisale cmisale merged commit 6d45e81 into IBM:helmupgrade Oct 18, 2024
1 check 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.

2 participants