Skip to content

chore(actions): Add doc how to verify GitHub Attestations with GitHub cli and verify release artifacts with Cosign#2846

Merged
stevehipwell merged 29 commits intointegrations:mainfrom
ViacheslavKudinov:maintenance/add-attestation
Feb 2, 2026
Merged

chore(actions): Add doc how to verify GitHub Attestations with GitHub cli and verify release artifacts with Cosign#2846
stevehipwell merged 29 commits intointegrations:mainfrom
ViacheslavKudinov:maintenance/add-attestation

Conversation

@ViacheslavKudinov
Copy link
Copy Markdown
Contributor

@ViacheslavKudinov ViacheslavKudinov commented Oct 29, 2025

Resolves #NaN


Before the change?

  • There is no doc how to verify attestations

After the change?

Pull request checklist

  • Schema migrations have been created if needed (example)
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@ViacheslavKudinov ViacheslavKudinov force-pushed the maintenance/add-attestation branch from 10dbe42 to cc98a12 Compare October 29, 2025 23:50
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov ViacheslavKudinov force-pushed the maintenance/add-attestation branch from cc98a12 to 4d2ad53 Compare October 29, 2025 23:50
@ViacheslavKudinov ViacheslavKudinov marked this pull request as ready for review October 29, 2025 23:52
@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

ViacheslavKudinov commented Oct 30, 2025

Any known thing why we may regret by adding attestation?

Just loudly thinking if anything we need to consider

@ViacheslavKudinov ViacheslavKudinov changed the title chore(Actions): Add GH attestation on release process chore(actions): Add GH attestation on release process Nov 1, 2025
@nickfloyd nickfloyd added this to the v7 Next milestone Nov 20, 2025
@nickfloyd nickfloyd moved this from Backlog to On Deck in Terraform Provider Nov 20, 2025
@github-actions github-actions bot added the Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR label Nov 23, 2025
@ViacheslavKudinov ViacheslavKudinov changed the title chore(actions): Add GH attestation on release process chore(actions): Add doc how to verify GH attestations Dec 8, 2025
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

@nickfloyd @stevehipwell I've updated PR to resolve conflicts after workflows were updated.
Now it includes only doc how to do verification of attestations.

Please, feel free to suggest any updates.

Copy link
Copy Markdown
Collaborator

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

Would it be possible to add the cosign equivalents of the gh commands? We also ought to provide the command to verify the SHA256SUMS file signature.

@ViacheslavKudinov ViacheslavKudinov force-pushed the maintenance/add-attestation branch from c992315 to 56c8a2e Compare December 10, 2025 21:47
@ViacheslavKudinov ViacheslavKudinov changed the title chore(actions): Add doc how to verify GH attestations chore(actions): Add doc how to verify GH attestations with GitHub cli and verify release artifacts with Cosign Dec 10, 2025
@ViacheslavKudinov ViacheslavKudinov force-pushed the maintenance/add-attestation branch from 56c8a2e to 17126dc Compare December 10, 2025 21:54
@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

@stevehipwell i've updated doc.
It is not exactly equivalents as with Cosign is only signed checksum file,
but anyway i tried to cover what we can do with current settings to verify the SHA256SUMS file signature and then with checksum the artifact(s).

Please, let me know if something else was expected or i've missed.

Copy link
Copy Markdown
Collaborator

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

I've added some comments, note that you can also verify the attestations with cosign.

ViacheslavKudinov and others added 2 commits December 12, 2025 17:11
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
@ViacheslavKudinov ViacheslavKudinov changed the title chore(actions): Add doc how to verify GH Attestations with GitHub cli and verify release artifacts with Cosign chore(actions): Add doc how to verify GitHub Attestations with GitHub cli and verify release artifacts with Cosign Jan 16, 2026
ViacheslavKudinov and others added 3 commits January 16, 2026 18:36
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

@ViacheslavKudinov this is looking really good. I've just added a couple of simple suggestions (note the headings one applies to all heading not just the one I highlighted).

@stevehipwell thanks for the suggestions. I had a chance to make the changes.

Could you check how it looks now ?
Anything else would be good to update?

Thank you in advance.

Copy link
Copy Markdown
Collaborator

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

I think the headings still need to be capitalized, I've mad some changes in the spirit of being concise but the real point is to get the capitalization right.

ViacheslavKudinov and others added 9 commits January 16, 2026 20:47
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

I think the headings still need to be capitalized, I've mad some changes in the spirit of being concise but the real point is to get the capitalization right.

Oh, sorry i didn't get it correctly.

Is it some "known rule" how capitalization should happen in "heading" ?

I'm not aware that there is something in place, but doesn't mean it is not there.

Copy link
Copy Markdown
Collaborator

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

Thanks for your patience @ViacheslavKudinov.

LGTM

@stevehipwell stevehipwell added the Type: Documentation Improvements or additions to documentation label Jan 20, 2026
@stevehipwell stevehipwell modified the milestones: v7 Next, v6.11.0 Release Jan 20, 2026
@stevehipwell stevehipwell enabled auto-merge (squash) February 2, 2026 17:11
@stevehipwell stevehipwell disabled auto-merge February 2, 2026 17:11
@stevehipwell
Copy link
Copy Markdown
Collaborator

@ViacheslavKudinov could you please rebase this?

@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

@ViacheslavKudinov could you please rebase this?

@stevehipwell merged "main" into this branch via UI

@stevehipwell stevehipwell enabled auto-merge (squash) February 2, 2026 18:35
@stevehipwell stevehipwell merged commit bbf7abb into integrations:main Feb 2, 2026
7 checks passed
@github-project-automation github-project-automation bot moved this from On Deck to Done in Terraform Provider Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Documentation Improvements or additions to documentation Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR

Projects

Development

Successfully merging this pull request may close these issues.

4 participants