Skip to content

ci: add missing xtensa-amd_acp_7_3 toolchain and enable provenance attestation#18

Merged
beriberikix merged 1 commit into
mainfrom
fix/workflow-warnings
Mar 15, 2026
Merged

ci: add missing xtensa-amd_acp_7_3 toolchain and enable provenance attestation#18
beriberikix merged 1 commit into
mainfrom
fix/workflow-warnings

Conversation

@beriberikix
Copy link
Copy Markdown
Contributor

Summary

Fix workflow-level warnings and metadata issues seen in GitHub Actions annotations for container builds.

Changes

W4 — Split collapsed xtensa-amd_acp entry into two toolchains

In the GNU matrix, one entry incorrectly combined:

  • arch: xtensa-amd_acp_7_0_adsp_zephyr-elf
  • nick: xtensa-amd_acp_7_3_adsp

This effectively dropped the 7.3 toolchain and mislabeled the 7.0 image tag. Replaced with two explicit entries:

  • xtensa-amd_acp_7_0_adsp_zephyr-elf / xtensa-amd_acp_7_0_adsp
  • xtensa-amd_acp_7_3_adsp_zephyr-elf / xtensa-amd_acp_7_3_adsp

W5 — Enable provenance attestations

For all four jobs (base, gnu, llvm, posix):

  • provenance: false -> provenance: true
  • Added required job permissions:
    • attestations: write
    • id-token: write

This resolves the build-push-action provenance warning while preserving existing build/push behavior.

Risk

Low. Changes are limited to workflow metadata and matrix entries; no Dockerfile or runtime logic changes.

@beriberikix beriberikix merged commit b642b61 into main Mar 15, 2026
36 checks passed
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.

1 participant