-
Notifications
You must be signed in to change notification settings - Fork 53
Conversation
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.
@jesuino can you please rebase and update the version again? Please don't forget to run pre-commit
to regenerate the README.md
.
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.
/lgtm
/approve
Btw, @jesuino do you think this should be rather a patch or feature release? Your PR title suggests it should be a feature release however your chart version change is a patch release. |
Hello @tumido I think a patch release is better suitable for this issue. Thanks for pointing this. I just fixed the conflicts with main branch |
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.
/lgtm
/approve
Description of the change
Adds the health check probes using the same value as the upstream commented probes:
https://github.com/backstage/charts/blob/main/charts/backstage/values.yaml#L162
This should be merged after janus-idp/backstage-showcase#567
Existing or Associated Issue(s)
janus-idp/backstage-showcase#565
Additional Information
This should fix the info dialog in Openshift, where it asks users to setup health checks
Checklist
Chart.yaml
according to semver.values.yaml
and added to the README.md. The pre-commit utility can be used to generate the necessary content. Usepre-commit run -a
to apply changes.pre-commit
hook.ct lint
command.