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

Create sha256 checksums for all binaries #786

Merged
merged 4 commits into from
Jun 15, 2023

Conversation

Jaanki
Copy link
Contributor

@Jaanki Jaanki commented Jun 12, 2023

Add a make target to create checksums for all subctl binaries created under dist/ directory. Add this new target to build-cross target so that the checksums are generated during the automated release process.

Epic: submariner-io/enhancements#182

@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr786/Jaanki/sha256_checks
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@Jaanki Jaanki self-assigned this Jun 12, 2023
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Add a make target to create checksums for all subctl binaries created
under dist/ directory. Add this new target to `build-cross` target so
that the checksums are generated during the automated release process.

Epic: submariner-io/enhancements#182

Signed-off-by: Janki Chhatbar <[email protected]>
@Jaanki Jaanki requested a review from skitt June 12, 2023 13:10
@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label Jun 14, 2023
@Jaanki Jaanki enabled auto-merge (rebase) June 14, 2023 07:53
@Jaanki Jaanki merged commit 46b7ceb into submariner-io:devel Jun 15, 2023
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr786/Jaanki/sha256_checks]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-test When a PR is ready for full E2E testing
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants