-
Notifications
You must be signed in to change notification settings - Fork 5
Closed
Description
Hey guys, was a pleasure talking with you on AWS summit TLV
Context
I stumbled across your upgrade to eks 1.22 tutorial and your guide to install kubectl-convert is specific to linux kernel
#installing kubectl convert
curl -LO "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/linux/amd64/kubectl-convert" && sudo install -o root -g root -m 0755 kubectl-convert /usr/local/bin/kubectl-convertIf you want to make it more agnostic I suggest adding OS mentioning like
#installing kubectl convert on linux
curl -LO "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/linux/amd64/kubectl-convert" && sudo install -o root -g root -m 0755 kubectl-convert /usr/local/bin/kubectl-convert
#installing kubectl convert on macOS
# Intel
curl -LO "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/darwin/amd64/kubectl-convert" && chmod +x ./kubectl-convert
# Apple Sillicon
curl -LO "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/darwin/arm64/kubectl-convert" && chmod +x ./kubectl-convert
sudo mv ./kubectl-convert /usr/local/bin/kubectl-convert && sudo chown root: /usr/local/bin/kubectl-convertor something like it, would do a PR but didn't find it, just my suggestion :)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels