Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an owner reference to the orchestrator pod for all created objects #20070

Merged
merged 1 commit into from
Apr 15, 2020

Conversation

carbonin
Copy link
Member

This will allow anything we create to be automatically removed when
the orchestrator pod is removed for any reason. If the pod is cleanly
scaled down we will continue to tell the workers to exit cleanly, but
if the orchestrator is abruptly removed this will ensure everything
is cleaned up properly.

Requires ManageIQ/manageiq-pods#440
Fixes ManageIQ/manageiq-pods#428

This will allow anything we create to be automatically removed when
the orchestrator pod is removed for any reason. If the pod is cleanly
scaled down we will continue to tell the workers to exit cleanly, but
if the orchestrator is abruptly removed this will ensure everything
is cleaned up properly.

Fixes ManageIQ/manageiq-pods#428
@miq-bot
Copy link
Member

miq-bot commented Apr 13, 2020

Checked commit carbonin@539a2cb with ruby 2.5.7, rubocop 0.69.0, haml-lint 0.28.0, and yamllint
1 file checked, 0 offenses detected
Everything looks fine. 👍

@Fryguy Fryguy merged commit 1f07002 into ManageIQ:master Apr 15, 2020
@carbonin carbonin deleted the orchestrator_add_owner_reference branch April 15, 2020 13:52
simaishi pushed a commit that referenced this pull request Apr 16, 2020
Add an owner reference to the orchestrator pod for all created objects

(cherry picked from commit 1f07002)
@simaishi
Copy link
Contributor

Jansa backport details:

$ git log -1
commit 79cf12f39444a169a5d24a9881ae9795e5374b84
Author: Jason Frey <fryguy9@gmail.com>
Date:   Wed Apr 15 09:52:13 2020 -0400

    Merge pull request #20070 from carbonin/orchestrator_add_owner_reference

    Add an owner reference to the orchestrator pod for all created objects

    (cherry picked from commit 1f070023fdaf08abf1f4dd8c0dc2ae9556f30477)

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

Successfully merging this pull request may close these issues.

Removing a manageiq CR leaves the worker containers
5 participants