Skip to content

Add support for DaemonSet restarts to the Restarter #235

Open

Description

@xeniape

Use Case:
The Restarter already has the ability to restart StatefulSet workloads based on a label. This functionality should also be implemented for DaemonSet workloads, since some workloads, like the OpaCluster, are running as a DaemonSet but might need a restart when the configuration provided by a ConfigMap changes. The need for this extension surfaced during the refinement for stackabletech/opa-operator#422
There exists a current workaround of manually restarting the Pods, so it's not very urgent.

Activity

  1. xeniape commented on Jun 3, 2024

    @xeniape
    MemberAuthor

    This functionality isn't needed anymore for the use case of stackabletech/opa-operator#422 and its original problem with the restart of OpaCluster after changes to decision logging configuration. The current implementation updates the decision log log levels in the opa start command, which automatically restarts the pods. But it might be needed later on if this behaviour changes or maybe for other use cases.

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

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions