-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix: controller alias for multigroups (v3) #1498
fix: controller alias for multigroups (v3) #1498
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: camilamacedo86 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 |
This comment has been minimized.
This comment has been minimized.
a4482f9
to
d78c03b
Compare
d78c03b
to
625533e
Compare
This comment has been minimized.
This comment has been minimized.
625533e
to
ad313a5
Compare
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.
This can only be applied to v3 projects and onwards
ad313a5
to
399c53c
Compare
This comment has been minimized.
This comment has been minimized.
We now have a project version and a plugin |
5d678bd
to
50c1c6d
Compare
f1c2f37
to
99ed73b
Compare
99ed73b
to
9f2f08a
Compare
9f2f08a
to
f31b5ca
Compare
f31b5ca
to
7ae3d94
Compare
/hold cancel |
Client: mgr.GetClient(), | ||
Log: ctrl.Log.WithName("controllers").WithName("%s"), | ||
Log: ctrl.Log.WithName("controllers").WithName("%s").WithName("%s"), |
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.
Log: ctrl.Log.WithName("controllers").WithName("%s").WithName("%s"), | |
Log: ctrl.Log.WithName("controllers").WithName("%s"), |
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.
Disregard.
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.
Split as requested.
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.
Eh this can probably go in the same PR since it's related. I didn't see the TODO above this change.
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.
done already in another pr. You are right, the correct way is to have 1 pr for each change/motivation.
7ae3d94
to
1208e8c
Compare
/test pull-kubebuilder-e2e-k8s-1-14-1 |
/lgtm |
Description
Motivation
Closes: #1382