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

Sign release artefacts using cosign #302

Merged
merged 4 commits into from
Mar 14, 2022

Conversation

cpanato
Copy link
Member

@cpanato cpanato commented Jan 29, 2022

What type of PR is this?

/kind feature

Any specific area of the project related to this PR?

/area build

What this PR does / why we need it:

Sign the release artefacts using cosign and also generate the SBOM using syft for now

the secret key is already set in circleci

/assign @leogr @Issif
cc @danpopnyc

Which issue(s) this PR fixes:

Fixes #300

Special notes for your reviewer:

Copy link
Member

@leogr leogr left a comment

Choose a reason for hiding this comment

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

SGTM, but I can't try it.

@Issif @cpanato do we want to merge it directly and see what happens?

@cpanato
Copy link
Member Author

cpanato commented Feb 2, 2022

@leogr I've run in a sample repo the commands and that worked :)

let's try, what we can do is when we cut the release, I will be watching to check and if fail can act right away

@Issif

@Issif
Copy link
Member

Issif commented Feb 2, 2022

I think I will work on the new release this month, I would to manage source and tags in payload first, this is important because of plugins and talon, but this is a big work as we have many outputs

@Issif Issif added this to the 2.25.0 milestone Mar 2, 2022
@Issif
Copy link
Member

Issif commented Mar 10, 2022

@cpanato I created the PR for dealing with source and tags in the events, feel free to rebase this PR on it after

@leogr
Copy link
Member

leogr commented Mar 11, 2022

@cpanato I created the PR for dealing with source and tags in the events, feel free to rebase this PR on it after

There are no conflicts. I guess we can approve this PR. @Issif is that ok for you?

@cpanato
Copy link
Member Author

cpanato commented Mar 11, 2022

i can rebase just to make sure, one sec

Signed-off-by: Carlos Panato <[email protected]>
Signed-off-by: Carlos Panato <[email protected]>
Signed-off-by: Carlos Panato <[email protected]>
@cpanato
Copy link
Member Author

cpanato commented Mar 11, 2022

@leogr @Issif rebased

Copy link
Member

@Issif Issif left a comment

Choose a reason for hiding this comment

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

❤️

@poiana poiana added the lgtm label Mar 14, 2022
@poiana
Copy link

poiana commented Mar 14, 2022

LGTM label has been added.

Git tree hash: 6fb279971631c5d5763edabc1feb874be85e92ee

@poiana
Copy link

poiana commented Mar 14, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cpanato, Issif

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@poiana poiana merged commit 323fca8 into falcosecurity:master Mar 14, 2022
@cpanato cpanato deleted the release-sign branch March 14, 2022 13:34
@cpanato
Copy link
Member Author

cpanato commented Mar 14, 2022

@Issif when you cut let me know in case we need to tweak anything :)

@Issif
Copy link
Member

Issif commented Mar 14, 2022

@Issif when you cut let me know in case we need to tweak anything :)

I'll ask you to review the PR for the next release anyway 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Sign release artifacts using cosign
4 participants