Skip to content

feat: add simple smoke test after draft release - #223

Merged
maxday merged 2 commits into
masterfrom
maxday/add-release-smoke-test
Aug 14, 2026
Merged

feat: add simple smoke test after draft release#223
maxday merged 2 commits into
masterfrom
maxday/add-release-smoke-test

Conversation

@maxday

@maxday maxday commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Add a post-release smoke-test job that downloads the published static library, verifies its checksum, and invokes a dummy handler through RIE in the provided:al2023 base image on both x86_64 and aarch64, gating the draft release on the artifact actually being consumable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

runs-on: ${{ matrix.runner }}
timeout-minutes: 15
permissions:
contents: write

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do we need write permission for running smoke tests?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Draft releases are only visible to tokens with push access, so the download step needs contents: write.

@maxday
maxday merged commit 8ee0d4d into master Aug 14, 2026
77 of 79 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.

2 participants