Skip to content

Post Honeycomb release marker on release#2114

Merged
edmorley merged 1 commit into
mainfrom
honeycomb-release-markers
Jul 6, 2026
Merged

Post Honeycomb release marker on release#2114
edmorley merged 1 commit into
mainfrom
honeycomb-release-markers

Conversation

@colincasey

@colincasey colincasey commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Passes the HONEYCOMB_API_KEY secret through to the shared _classic-buildpack-publish.yml@latest workflow as honeycomb_api_key, enabling a Honeycomb release marker on the builds dataset for each production publish of this classic buildpack.

The shared workflow support landed in heroku/languages-github-actions#369 (released in v1.11.0, which @latest now resolves to), and this repo's HONEYCOMB_API_KEY secret is configured — so once this PR merges, the marker step posts to the builds dataset on each production publish. The step is a safe no-op in any context where the secret is absent — it is gated on env.HONEYCOMB_API_KEY != '' && env.MODE == 'production' and declared required: false in the shared workflow.

What

  • Adds honeycomb_api_key: ${{ secrets.HONEYCOMB_API_KEY }} to the release job's secrets: block, forwarding the repo secret to the shared workflow's optional input.

The marker behavior (opt-in, production-only, idempotent, best-effort) lives in the shared workflow — see heroku/languages-github-actions#369 for details.

W-22846373

@colincasey colincasey self-assigned this Jul 6, 2026
@colincasey
colincasey marked this pull request as ready for review July 6, 2026 15:25
@colincasey
colincasey requested a review from edmorley as a code owner July 6, 2026 15:25
@colincasey
colincasey requested review from a team and edmorley and removed request for a team and edmorley July 6, 2026 15:25
@edmorley
edmorley merged commit 072ebb9 into main Jul 6, 2026
11 of 14 checks passed
@edmorley
edmorley deleted the honeycomb-release-markers branch July 6, 2026 16:06
@heroku-linguist heroku-linguist Bot mentioned this pull request Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants