Skip to content

fix(cd): stop sending provenance attestation to GCOM#815

Open
xnyo wants to merge 5 commits into
mainfrom
giuseppe/remove-artifacts-attestation
Open

fix(cd): stop sending provenance attestation to GCOM#815
xnyo wants to merge 5 commits into
mainfrom
giuseppe/remove-artifacts-attestation

Conversation

@xnyo

@xnyo xnyo commented Jun 23, 2026

Copy link
Copy Markdown
Member

Fixes #814

GCOM no longer consumes the provenanceAttestation field (see grafana/grafana-com#18942), so this PR stops including it in the plugin publish payload. The input and provenance calculation steps are kept in case we decide to add it back. It's still possible to generate the attestation for the plugin in case it needs to be used for other reason (rather than POST-ing it to GCOM).

@xnyo xnyo self-assigned this Jun 23, 2026
@xnyo xnyo moved this from 📬 Triage to 🧑‍💻 In development in Grafana Catalog Team Jun 23, 2026
GCOM no longer consumes the "provenanceAttestation" field
(see grafana/grafana-com#18942), so it is no longer included in the
plugin publish payload.

The attestation machinery is intentionally kept in place: the
`attestation` input, the `attestations: write` permission, the
`build-attestation` job, the action's `provenance-attestation` input
and the `--provenance-attestation` flag are all still present. Only the
`provenanceAttestation` JSON field is omitted from the GCOM API call,
and it is commented (in publish.sh) so it can be wired back in easily if
needed. Docstrings are updated accordingly.

Fixes #814
@xnyo
xnyo force-pushed the giuseppe/remove-artifacts-attestation branch from 29b0fee to 4ccb259 Compare July 2, 2026 12:50
@xnyo xnyo changed the title feat(cd)!: remove provenance attestation fix(cd): stop sending provenance attestation to GCOM Jul 2, 2026
@xnyo
xnyo marked this pull request as ready for review July 2, 2026 13:07
@xnyo
xnyo requested a review from a team as a code owner July 2, 2026 13:07
@xnyo
xnyo requested review from oshirohugo, toddtreece and wbrowne July 2, 2026 13:07
@xnyo xnyo moved this from 🧑‍💻 In development to 🔬 In review in Grafana Catalog Team Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🔬 In review

Development

Successfully merging this pull request may close these issues.

Remove artifact attestation

1 participant