Skip to content

This should? (not 100% sure) fix the krew plugin entitled debug-shell#15

Open
rshutt wants to merge 2 commits intodanisla:masterfrom
rshutt:master
Open

This should? (not 100% sure) fix the krew plugin entitled debug-shell#15
rshutt wants to merge 2 commits intodanisla:masterfrom
rshutt:master

Conversation

@rshutt
Copy link
Copy Markdown

@rshutt rshutt commented Sep 6, 2021

kubectl krew debug-shell seems to be non-functional in k8s 1.22.x. I'm sure it has been broken a while. I fixed it. Now I'm contributing back.

Please don't hesitate to reach out with any questions, comments, or concerns.

Regards,
--Randall

the debug-shell.sh script has been changed to fix the following:

* the post 1.12 extended plugin model no longer sets the ENV var that was previously expected for the namespace
* somewhere along the line, `--rm` is throwing errors if you do not pass `attach=true` even though per the manpage, 
  -i is supposed to set `attach=true`
Update debug-shell.sh for today's k8s (1.22 here)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant