-
Notifications
You must be signed in to change notification settings - Fork 61
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
Publish olmv1 kubectl plugin #1852
Comments
The entire release and krew index update can heppen automatically: https://github.com/operator-framework/kubectl-operator/blob/main/.github/workflows/release.yml We need to decide as a community what we want the new name of the plugin to be in the krew index. I expect that to be a semi-contentious topic. |
Also, we should consider the versioning implications of putting this binary in this repository because this repository is now at major version 1. I know that we have said in our public API docs that the CLIs here are not part of the public API, but the reason that mostly works is because users aren't really expected to interact with the CLIs. In the case of the kubectl plugin, users are expected to interact with the CLI, which leads to:
|
I wasn't aware we have a workflow in place for publishing to krew, we should hold off on that at least until we finalize what the cli looks like. is there also a pipeline that publishes kubectl-operator to opensuse somehow? Apart from that, I don't find too many places directly using kubectl-operator that we may need to get in touch with; just console using it to list operands (which we don't have support for in the olmv1 cli as of now) and a few demos from There's also the extension_developer_e2e tests like @joelanford mentioned in the demo/docs issue where it makes sense to use the new cli operator-controller/test/extension-developer-e2e/extension_developer_test.go Lines 40 to 199 in a6de9f9
|
Publish a new release of the olmv1 kubectl plugin to krew from the operator-controller repo by:
See https://krew.sigs.k8s.io/docs/developer-guide/distributing-with-krew/
The text was updated successfully, but these errors were encountered: