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
Refactor metadata to separate controller (part 1) #292
Refactor metadata to separate controller (part 1) #292
Conversation
d05b6af
to
77bde3c
Compare
/hold stop retesting this. |
c90e1cc
to
ca6d33d
Compare
d6815ef
to
4513e5d
Compare
/hold cancel This is green and provide ground work. |
/retest |
1 similar comment
/retest |
storage tests flaking |
4ab9ca5
to
93b95f9
Compare
/retest infra registry flake |
93b95f9
to
40685a2
Compare
40685a2
to
384ed4f
Compare
@stlaz updated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Squash the last commit to the previous, move the condition that's not being set to stale and lets' get this merged
// These conditions are operated and defaulted by this controller. | ||
// Any new condition used by this controller sync() loop should be listed here. | ||
var knownConditionNames = sets.NewString( | ||
"WellKnownEndpointDegraded", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since you don't set it to true, move it to Stale conditions
d04c734
to
5ac5f48
Compare
5ac5f48
to
5981bd2
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mfojtik, stlaz 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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
11 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. |
/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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
5 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. |
This holds a ground work for splitting the main operator loop into more granular controllers based on controller factory.
To preserve order (temporarely), the operator progressing conditions are used. Note that this will eventually go away after all blocks are split (in this PR the first blocks is separated to its own controller).