-
Notifications
You must be signed in to change notification settings - Fork 2.2k
cmctl and kubectl-cert_manager to release targets #4523
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
cmctl and kubectl-cert_manager to release targets #4523
Conversation
runtime object names Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com>
command. Can be configured at build time. Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com>
registered. Gates the completion command which is disabled by defualt. Can be configured at build time. Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com>
command by the build name. Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com>
package name Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com>
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com>
binary Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com>
Signed-off-by: joshvanl <vleeuwenjoshua@gmail.com>
/kind feature |
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jakexks, JoshVanL 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 |
Branched from #4522 which should be merged first.
This PR switches the default of the build name and registered to
cmctl
and register the completion sub-command.Adds bazel to publish cmctl along with kubectl-cert_manager targets. These are built with the correct command names and whether they should have the completion subcommand.
/assign @jakexks