Skip to content
Snippets Groups Projects

Ensure Pod Restarts upon Object Update

Merged Ensure Pod Restarts upon Object Update
(removed):wip/pvigo/pod-restart into collabora/staging
All threads resolved!
Merged Pablo Vigo Mas requested to merge (removed):wip/pvigo/pod-restart into collabora/staging
All threads resolved!

Ensure Pod restarts automatically whenever the ConfigMap or Secret object is updated. A hash of the object is now included in StatefulSet.

PODs only restart if the checksum of the ConfigMap or Secret is different, which means that the content of the object has been changed. PODs remain running if the content of the object is the same.

Phabricator task: T10031

Signed-off-by: Pablo Vigo pvigo@collabora.com

Merge request reports

Checking pipeline status.

Approved by
Ryan Gonzalez Emanuele Aina

Merged by Ryan GonzalezRyan Gonzalez 1 year ago (Jan 16, 2024 3:31pm UTC)

Merge details

  • Changes merged into collabora/staging with dbf93f54.
  • Deleted the source branch.

Pipeline #83240 passed

Stage: docker

    Pipeline passed for dbf93f54 on collabora/staging

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
  • Pablo Vigo Mas added 1 commit

    added 1 commit

    • 8293d3ae - Ensure Pod Restarts upon Object Update

    Compare with previous version

  • Pablo Vigo Mas resolved all threads

    resolved all threads

  • Emanuele Aina
  • Emanuele Aina
    Emanuele Aina @em ·
    Owner

    LGTM!

    I will leave the actual merging to @refi64 though, and my bikeshedding can safely be ignored.

    Edited by Emanuele Aina
  • Emanuele Aina approved this merge request

    approved this merge request

  • Pablo Vigo Mas resolved all threads

    resolved all threads

  • Pablo Vigo Mas added 1 commit

    added 1 commit

    • 27d94390 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Ryan Gonzalez approved this merge request

    approved this merge request

  • Ryan Gonzalez mentioned in commit dbf93f54

    mentioned in commit dbf93f54

  • Ryan Gonzalez merged

    merged

    • Please register or sign in to reply