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 only a basic config observer #242
Add only a basic config observer #242
Conversation
opening now just to see which tests fail while I observe whether the rebased commits are reasonable |
/cc @p0lyn0mial |
b682e66
to
2aa6d95
Compare
2aa6d95
to
4cc7704
Compare
Removed some unnecessary comments in the latest revision |
/retest |
4cc7704
to
bdda8f4
Compare
@sttts I updated the PR to use the CORS observer from lib-go |
/retest |
pkg/operator2/configobservation/configobservercontroller/observe_config_controller.go
Outdated
Show resolved
Hide resolved
pkg/operator2/configobservation/configobservercontroller/observe_config_controller.go
Outdated
Show resolved
Hide resolved
e9a8bc6
to
eb32c85
Compare
pkg/operator2/configobservation/configobservercontroller/observe_config_controller.go
Outdated
Show resolved
Hide resolved
eb32c85
to
d9437c4
Compare
3efb1c8
to
f57c4ce
Compare
f57c4ce
to
a817a7b
Compare
Addressed comments + added a unit test to each of the observers that should make sure we don't spam update events as seen elsewhere. |
/hold cancel |
/retest |
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: p0lyn0mial, stlaz, sttts The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test e2e-aws-operator |
/retest Please review the full test history for this PR and help us cut down flakes. |
6 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
This config observer PR leaves out the tricky
oauth/cluster
bits of #222 to get this merged faster