Skip to content

Adopt health checks in sharedmain #1048

Description

@pmorie

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.

Activity

  1. github-actions commented on Aug 24, 2020

    @github-actions
    Contributor

    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.

  2. added
    lifecycle/staleDenotes an issue or PR has remained open with no activity and has become stale.
    on Aug 24, 2020
  3. afrittoli commented on Oct 20, 2020

    @afrittoli
    Contributor

    /open

  4. afrittoli commented on Oct 20, 2020

    @afrittoli
    Contributor

    /lifecycle frozen

  5. added
    lifecycle/frozenIndicates that an issue or PR should not be auto-closed due to staleness.
    and removed
    lifecycle/staleDenotes an issue or PR has remained open with no activity and has become stale.
    on Oct 20, 2020
  6. mattmoor commented on Oct 20, 2020

    @mattmoor
    Member

    We should use the shared drainer logic, but probably have something that does this over HTTP (no need for TLS)

  7. mpetason commented on Oct 20, 2020

    @mpetason
    Contributor

    /assign

    I'll take a shot at this.

  8. removed their assignment
    on Jan 25, 2021
  9. mpetason commented on Jan 25, 2021

    @mpetason
    Contributor

    Unassigned so someone can resolve this faster than I can.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    kind/featurelifecycle/frozenIndicates that an issue or PR should not be auto-closed due to staleness.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions