Skip to content

Conversation

@rickeylev
Copy link
Collaborator

My original change to use the bazel-contrib BCR publishing workflow was horribly broken.
This fixes a variety of issues.

  • Fix the workflow call secrets variable name
  • Allow getting the BCR publishing token from the workflow arg (the release
    workflow calls it)
  • Fix the overall syntax of the workflows. It was just entirely invalid in
    several regards.
  • Fix the yaml -> yml file name reference. NGL, it took me longer than I'd
    like to admit to see that, eh.

Along the way, some cleanups and improvements

  • Allow specifying a particular tag to release, while using workflow state from
    a different commit. This allows us to make fixes on main, and then use it to
    manually re-trigger a tag to be released.
  • Add descriptions for workflow inputs
  • Allow bcr token to be passed to release. This will allow e.g. automatically
    scheduled releases.
  • Quote shell variables because its good practice.

@rickeylev rickeylev requested a review from aignas as a code owner October 17, 2025 04:28
@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@rickeylev rickeylev mentioned this pull request Oct 17, 2025
11 tasks
@rickeylev rickeylev added this pull request to the merge queue Oct 19, 2025
Merged via the queue into bazel-contrib:main with commit 6f80d34 Oct 19, 2025
4 checks passed
@rickeylev rickeylev deleted the fix.release.workflow branch October 19, 2025 23:16
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