Skip to content

Conversation

@neilvana
Copy link
Collaborator

@neilvana neilvana commented Mar 3, 2022

What does this Pull Request accomplish?

Adds new reusable workflows and documentation to sign container images and helm charts published to Artifactory.

Why should this Pull Request be merged?

We need these features to enable delivering signed containers and helm charts to our customers. These workflows will enable adding signing to many different repos without having to reimplement the logic.

What testing has been done?

These reusable workflows have been tested from another repo. See the workflow run here:

https://github.com/neilvana/signing-dev/actions/runs/1925257319

@neilvana neilvana requested a review from irwand as a code owner March 3, 2022 00:01
@neilvana
Copy link
Collaborator Author

neilvana commented Mar 3, 2022

We should squash this pull request. It has a lot of noise.

env:
GNUPGHOME: ${{ runner.temp }}/gpg
- name: List keys
run: gpg --list-secret-keys
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is only for debugging so can be deleted?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You are correct. I'll remove that.

@neilvana neilvana merged commit 8ca12b2 into main Mar 3, 2022
@neilvana neilvana deleted the users/nstoddar/add-sign-container-workflow branch March 3, 2022 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants