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

ci: use keyless signing for main and release branches #2783

Merged
merged 2 commits into from
May 17, 2023

Conversation

34fathombelow
Copy link
Member

@34fathombelow 34fathombelow commented May 16, 2023

This PR implements using keyless signing for cosign. It uses a reusable workflow (image-reuse.yaml) that will also be used in the release workflow to generate attestations.

Changes in docker-publish workflow:

  1. Concurrency added to reduce the amount of executors used on multiple pushes.
  2. Only the latest tag will be used, master tag has been removed on a push events of the master branch.
  3. image-reuse.yaml is the exact same reusable workflow we are using in Argo CD.

Successful testing of generating attestations for Argo Rollouts has been completed and will come in a separate PR along with all the documentation.
https://github.com/34fathombelow/rollouts-testing/actions/runs/4989497845
https://github.com/34fathombelow/rollouts-testing/releases/tag/v2.0.0-rc3

@github-actions
Copy link
Contributor

github-actions bot commented May 16, 2023

Go Published Test Results

1 952 tests   1 952 ✔️  2m 35s ⏱️
   118 suites         0 💤
       1 files           0

Results for commit 08ae012.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (497e69a) 81.64% compared to head (08ae012) 81.64%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2783   +/-   ##
=======================================
  Coverage   81.64%   81.64%           
=======================================
  Files         133      133           
  Lines       20178    20178           
=======================================
  Hits        16475    16475           
  Misses       2849     2849           
  Partials      854      854           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions
Copy link
Contributor

github-actions bot commented May 16, 2023

E2E Tests Published Test Results

    4 files      4 suites   3h 32m 2s ⏱️
  96 tests   81 ✔️   5 💤 10
400 runs  365 ✔️ 20 💤 15

For more details on these failures, see this check.

Results for commit 08ae012.

♻️ This comment has been updated with latest results.

Signed-off-by: Justin Marquis <[email protected]>
@sonarcloud
Copy link

sonarcloud bot commented May 16, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@zachaller zachaller merged commit 5435982 into argoproj:master May 17, 2023
@34fathombelow 34fathombelow deleted the keyless-signing branch May 17, 2023 21:30
zachaller pushed a commit that referenced this pull request May 24, 2023
* ci: use keyless signing for main and release branches

Signed-off-by: Justin Marquis <[email protected]>

* fix typo

Signed-off-by: Justin Marquis <[email protected]>

---------

Signed-off-by: Justin Marquis <[email protected]>
zachaller pushed a commit that referenced this pull request May 24, 2023
* ci: use keyless signing for main and release branches

Signed-off-by: Justin Marquis <[email protected]>

* fix typo

Signed-off-by: Justin Marquis <[email protected]>

---------

Signed-off-by: Justin Marquis <[email protected]>
zachaller pushed a commit that referenced this pull request May 24, 2023
* ci: use keyless signing for main and release branches

Signed-off-by: Justin Marquis <[email protected]>

* fix typo

Signed-off-by: Justin Marquis <[email protected]>

---------

Signed-off-by: Justin Marquis <[email protected]>
@zachaller zachaller added the cherry-pick-completed Used once we have cherry picked the PR to all requested releases label May 24, 2023
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