Repository navigation
Adopt health checks in sharedmain #1048
Copy link
Copy link
Closed
knative/serving
#13563Labels
kind/featurelifecycle/frozenIndicates that an issue or PR should not be auto-closed due to staleness.Indicates that an issue or PR should not be auto-closed due to staleness.
Description
Activity
This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with/reopen. Mark the issue as
fresh by adding the comment/remove-lifecycle stale.- addedlifecycle/staleDenotes an issue or PR has remained open with no activity and has become stale.Denotes an issue or PR has remained open with no activity and has become stale.
on Aug 24, 2020 /open
/lifecycle frozen
- addedlifecycle/frozenIndicates that an issue or PR should not be auto-closed due to staleness.Indicates that an issue or PR should not be auto-closed due to staleness.and removedlifecycle/staleDenotes an issue or PR has remained open with no activity and has become stale.Denotes an issue or PR has remained open with no activity and has become stale.
on Oct 20, 2020 We should use the shared
drainerlogic, but probably have something that does this over HTTP (no need for TLS)Reacted by Andrea Frittoli/assign
I'll take a shot at this.
Unassigned so someone can resolve this faster than I can.
Metadata
Metadata
Assignees
Labels
kind/featurelifecycle/frozenIndicates that an issue or PR should not be auto-closed due to staleness.Indicates that an issue or PR should not be auto-closed due to staleness.
Currently, we do not use health checks in our controllers descended from
sharedmain. Adopting health checks will allow us to begin reporting controllers as unhealthy so that they are restarted by the kubelet. For example, we will be able to use the leaderelection health checker to begin reporting unhealthy after the leader election timeout elapsed when renewing the leader election lease./kind feature
Expected Behavior
Controllers can report themselves as unhealthy so that they can be restarted.
Actual Behavior
That doesn't happen.