Skip to content

chore: verify sha256sum of kubectl#134

Merged
cpanato merged 1 commit intohelm:mainfrom
felix-kaestner:main
Apr 11, 2025
Merged

chore: verify sha256sum of kubectl#134
cpanato merged 1 commit intohelm:mainfrom
felix-kaestner:main

Conversation

@felix-kaestner
Copy link
Copy Markdown
Contributor

This patch extends the installation of the kubectl binary to validate the checksum of the downloaded file, similar to what is already done for kind. This approach is also described as an optional step on https://kubernetes.io/docs/tasks/tools/install-kubectl-linux/#install-kubectl-binary-with-curl-on-linux.

@cpanato
Copy link
Copy Markdown
Member

cpanato commented Mar 5, 2025

please sign the dco

@felix-kaestner
Copy link
Copy Markdown
Contributor Author

please sign the dco

@cpanato Done ✔️

@felix-kaestner
Copy link
Copy Markdown
Contributor Author

Hey @cpanato, are there any other blockers for proceeding with this change?

@cpanato
Copy link
Copy Markdown
Member

cpanato commented Apr 10, 2025

apolagizes just saw the notification now

seems the job did not trigger can you try an amend and push again?

This patch extends the installation of the kubectl binary to validate
the checksum of the downloaded file, similar to what is already done for
kind. This approach is also described as an optional step on https://kubernetes.io/docs/tasks/tools/install-kubectl-linux/#install-kubectl-binary-with-curl-on-linux.

Signed-off-by: Felix Kästner <[email protected]>
@felix-kaestner
Copy link
Copy Markdown
Contributor Author

@cpanato I think you need to manually approve the workflow run.

Copy link
Copy Markdown
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

thanks

@cpanato cpanato merged commit a6dfd81 into helm:main Apr 11, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants