MnrGreg / kubectl-node-restart

Krew plugin to restart Kubernetes Nodes sequentially and gracefully
Other
104 stars 11 forks source link

feat: Specify context in order to be able to manage several cluster in the same time #11

Closed cyrinux closed 1 year ago

cyrinux commented 1 year ago

Hi @MnrGreg,

Thanks for the toy.

A tiny modification to be able to play on different clusters in the same time in a safe way. - I understand we actually have to enable the good context with kubectx for example, but if I switch the context while the script is running in the background, this can be dangerous. I let the parameter optional.

If you accept those modifications, can you do a release then ?

Thanks 🙏

MnrGreg commented 1 year ago

thanks for contributing @cyrinux