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

pkg/ansible/proxy: adding ability for dependent watches to be recovered #1067

Conversation

shawn-hurley
Copy link
Member

Description of the change:
This will allow ansible operator to re-watch dependent resources on an operator pod restart or on the loss of leader status.

Motivation for the change:
Operator pods are expected to be restarted or to lose leader status

@shawn-hurley shawn-hurley added kind/bug Categorizes issue or PR as related to a bug. language/ansible Issue is related to an Ansible operator project v0.4.x labels Feb 6, 2019
@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 6, 2019
@fabianvf fabianvf self-requested a review February 6, 2019 20:17
Copy link
Member

@fabianvf fabianvf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shawn-hurley shawn-hurley force-pushed the bug/recover-dependent-watches-0.4 branch from b0a1a00 to d663bd0 Compare February 7, 2019 13:11
@shawn-hurley shawn-hurley merged commit 071e020 into operator-framework:v0.4.x Feb 7, 2019
shawn-hurley pushed a commit to shawn-hurley/operator-sdk that referenced this pull request Apr 23, 2019
…ed (operator-framework#1067)

**Description of the change:**
This will allow ansible operator to re-watch dependent resources on an operator pod restart or on the loss of leader status.

**Motivation for the change:**
Operator pods are expected to be restarted or to lose leader status
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. language/ansible Issue is related to an Ansible operator project size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants