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

Signing Helm packages #1151

Merged
merged 5 commits into from
Feb 5, 2025
Merged

Signing Helm packages #1151

merged 5 commits into from
Feb 5, 2025

Conversation

didierofrivia
Copy link
Member

Part of the work needed for Kuadrant/helm-charts#18

This PR introduces the GPG signing of Helm chart packages upon creation. It also uploads its provenance file to the GH release page.

The job now requires to be passed an environment variable GPG_KEYRING_BASE64 which represents the GPG keyring base64 encoded, in order to be stored as a GH action variable.

The following PRs need to be merged first:

@didierofrivia didierofrivia added the kind/enhancement New feature or request label Jan 31, 2025
@didierofrivia didierofrivia self-assigned this Jan 31, 2025
@didierofrivia didierofrivia requested a review from a team as a code owner January 31, 2025 11:50
Copy link

codecov bot commented Jan 31, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.63%. Comparing base (d97e8dd) to head (9f79273).
Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1151      +/-   ##
==========================================
+ Coverage   83.45%   83.63%   +0.18%     
==========================================
  Files          81       81              
  Lines        6943     6943              
==========================================
+ Hits         5794     5807      +13     
+ Misses        925      913      -12     
+ Partials      224      223       -1     
Flag Coverage Δ
bare-k8s-integration 22.88% <ø> (+0.01%) ⬆️
controllers-integration 74.95% <ø> (+0.01%) ⬆️
envoygateway-integration 40.61% <ø> (+0.01%) ⬆️
gatewayapi-integration 19.47% <ø> (-0.09%) ⬇️
istio-integration 43.27% <ø> (+0.11%) ⬆️
unit 19.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
api/v1beta1 (u) 90.00% <ø> (ø)
api/v1beta2 (u) ∅ <ø> (∅)
pkg/common (u) ∅ <ø> (∅)
pkg/istio (u) 62.06% <ø> (ø)
pkg/log (u) 93.18% <ø> (ø)
pkg/reconcilers (u) 24.67% <ø> (ø)
pkg/rlptools (u) ∅ <ø> (∅)
controllers (i) 86.83% <ø> (+0.04%) ⬆️

see 7 files with indirect coverage changes

eguzki
eguzki previously approved these changes Feb 4, 2025
* Needs an extra step because svenstaro/upload-release-action@v2 doesn't
  support asset naming on file_glob

Signed-off-by: dd di cesare <[email protected]>
* the GnuPG v2 store the secret keyring using the format kbx and Helm
  works with the legacy gpg format.
* Enclosing within quotes to fix env interpolation

Signed-off-by: dd di cesare <[email protected]>
@didierofrivia didierofrivia added this pull request to the merge queue Feb 5, 2025
Merged via the queue into main with commit af40d9c Feb 5, 2025
41 checks passed
@didierofrivia didierofrivia deleted the helm-sign-packages branch February 5, 2025 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants