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

Change to HA-Compatible Status Schemas #159

Merged
merged 2 commits into from Jun 28, 2019

Conversation

Projects
None yet
2 participants
@maxsmythe
Copy link
Contributor

commented Jun 28, 2019

Currently the status schemas used for our resources would
not work if an HA deployment as only one Pod could report
its status at a time. This PR adds a mechanism for multiple
pods to broadcast their handling of a resource.

This fixes #158

Signed-off-by: Max Smythe smythe@google.com

Change to HA-Compatible Status Schemas
Currently the status schemas used for our resources would
not work if an HA deployment as only one Pod could report
its status at a time. This PR adds a mechanism for multiple
pods to broadcast their handling of a resource.

This fixes #158

Signed-off-by: Max Smythe <smythe@google.com>

@maxsmythe maxsmythe requested review from ritazh and ctab Jun 28, 2019

Fix broken CT unit tests
Signed-off-by: Max Smythe <smythe@google.com>
@ctab

ctab approved these changes Jun 28, 2019

Copy link
Member

left a comment

LGTM

@maxsmythe maxsmythe merged commit 1f95f8c into open-policy-agent:master Jun 28, 2019

3 checks passed

DCO DCO
Details
cla/linuxfoundation maxsmythe authorized
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@maxsmythe maxsmythe deleted the maxsmythe:ha_compatible_status branch Jul 9, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.