Skip to content

Fix release workflow to use workflow_dispatch for ccloud-private#109

Merged
linhcrl merged 1 commit into
cockroachdb:mainfrom
linhcrl:workflow-dispatch-release
May 18, 2026
Merged

Fix release workflow to use workflow_dispatch for ccloud-private#109
linhcrl merged 1 commit into
cockroachdb:mainfrom
linhcrl:workflow-dispatch-release

Conversation

@linhcrl
Copy link
Copy Markdown
Contributor

@linhcrl linhcrl commented May 18, 2026

The ccloud-private repo's sync-cli-with-sdk workflow uses workflow_dispatch, not repository_dispatch. This updates the release workflow to trigger it correctly using gh workflow run with the required sdk_version input.

Also fixes all references from tag-created to tag_created for consistency with the autotag-from-changelog action's output format.

@linhcrl linhcrl requested a review from fantapop May 18, 2026 17:38
runs-on: ubuntu-latest
outputs:
tag-created: ${{ steps.autotag.outputs.tag-created }}
tag_created: ${{ steps.autotag.outputs.tag_created }}
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.

the reusable workflow outputs tag_created in snake case, not kebab cases

The ccloud-private repo's sync-cli-with-sdk workflow uses
workflow_dispatch, not repository_dispatch. This updates the release
workflow to trigger it correctly using `gh workflow run` with the
required sdk_version input.

Also fixes all references from tag-created to tag_created for
consistency with the autotag-from-changelog action's output format.

Co-Authored-By: roachdev-claude <roachdev-claude-bot@cockroachlabs.com>
@linhcrl linhcrl force-pushed the workflow-dispatch-release branch from 038f1cd to f9b2c61 Compare May 18, 2026 17:46
Copy link
Copy Markdown
Contributor

@fantapop fantapop left a comment

Choose a reason for hiding this comment

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

LGTM

@linhcrl linhcrl merged commit 894ed62 into cockroachdb:main May 18, 2026
1 check 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