Skip to content
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

Remove istioctl collateral bash/zsh #34092

Closed
ericvn opened this issue Jul 15, 2021 · 3 comments
Closed

Remove istioctl collateral bash/zsh #34092

ericvn opened this issue Jul 15, 2021 · 3 comments
Labels
area/user experience kind/enhancement lifecycle/automatically-closed Indicates a PR or issue that has been closed automatically. lifecycle/stale Indicates a PR or issue hasn't been manipulated by an Istio team member for a while

Comments

@ericvn
Copy link
Contributor

ericvn commented Jul 15, 2021

Describe the feature request

The way to create the completion files has changes to istioctl completion zsh/bash per an earlier PR (#33787). I have created a PR to use the new commands as part of the make (#34091). We should deprecate/remove the collateral zsh/bash from istioctl.

Describe alternatives you've considered

[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure

Additional context

@esnible
Copy link
Contributor

esnible commented Aug 13, 2021

Can this be closed?

@ericvn
Copy link
Contributor Author

ericvn commented Aug 13, 2021

No. Running ./out/darwin_amd64/istioctl collateral completion --zsh still generates a file which is different than ./out/darwin_amd64/istioctl completion zsh > _istioctl_2 and the later should be correct per #33787.

completion is shown as the available command. I suspect we just want to remove/deprecate as it is now wrong collateral --bash and collateral --zsh or point them to the correct new command.

@istio-policy-bot istio-policy-bot added the lifecycle/stale Indicates a PR or issue hasn't been manipulated by an Istio team member for a while label Feb 10, 2022
@istio-policy-bot
Copy link

🚧 This issue or pull request has been closed due to not having had activity from an Istio team member since 2021-08-13. If you feel this issue or pull request deserves attention, please reopen the issue. Please see this wiki page for more information. Thank you for your contributions.

Created by the issue and PR lifecycle manager.

@istio-policy-bot istio-policy-bot added the lifecycle/automatically-closed Indicates a PR or issue that has been closed automatically. label Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/user experience kind/enhancement lifecycle/automatically-closed Indicates a PR or issue that has been closed automatically. lifecycle/stale Indicates a PR or issue hasn't been manipulated by an Istio team member for a while
Projects
None yet
Development

No branches or pull requests

3 participants