-
Notifications
You must be signed in to change notification settings - Fork 157
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
Switch operator to use protobuf client for non-CR resources #250
Switch operator to use protobuf client for non-CR resources #250
Conversation
Surprisingly fast green, do you have data before and after this change? |
fd624d5
to
599f468
Compare
Hrm, the openshift-config namespace seems to be missing... /retest |
63c539a
to
4f60bf8
Compare
pkg/operator/starter.go
Outdated
@@ -29,7 +29,7 @@ import ( | |||
) | |||
|
|||
func RunOperator(ctx *controllercmd.ControllerContext) error { | |||
kubeClient, err := kubernetes.NewForConfig(ctx.KubeConfig) | |||
protoKubeClient, err := kubernetes.NewForConfig(ctx.ProtoKubeConfig) |
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.
don't rename this
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.
fixed.
4f60bf8
to
d943b29
Compare
Fixed client rename, LGTM per Slack from @deads2k |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
d943b29
to
0fbf006
Compare
New changes are detected. LGTM label has been removed. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mfojtik 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 |
Re-applied LGTM as the only change was a panic fix in library-go. |
/retest Please review the full test history for this PR and help us cut down flakes. |
0fbf006
to
4866b05
Compare
New changes are detected. LGTM label has been removed. |
This builds on openshift/library-go#233 and switches the controller loops that deal with non-CR resources to client-go with protobuf enforced.
/cc @deads2k
/cc @smarterclayton
NOTE: Remove the artificial bump before merging.