Skip to content

DEVOPS-77: fix legacy-14 publish tag prefix and STS checkout - #127

Merged
blumamir merged 3 commits into
odigos-io:mainfrom
blumamir:devops-77-fix-legacy-node-14-publish-workflow-git-tag-prefix-and-sts
Aug 29, 2026
Merged

DEVOPS-77: fix legacy-14 publish tag prefix and STS checkout#127
blumamir merged 3 commits into
odigos-io:mainfrom
blumamir:devops-77-fix-legacy-node-14-publish-workflow-git-tag-prefix-and-sts

Conversation

@blumamir

Copy link
Copy Markdown
Collaborator

What this PR does / why we need it:

Fixes the Publish Legacy 14 workflow so it bumps/creates git tags under the nodejs-community-14/ prefix (avoiding collisions with mainline v0.0.x), and uses the tag-releaser STS token with output-git-config for checkout + push. Docker/AGENT_VERSION tags remain bare v0.0.x.

Linear: https://linear.app/odigos/issue/DEVOPS-77/fix-legacy-node-14-publish-workflow-git-tag-prefix-and-sts-auth

Changelog entry: Does this PR introduce a user-facing bug fix, feature, dependency update, or breaking change??

NONE

Test plan

  • Dry-run / review workflow YAML: version calc lists nodejs-community-14/v0.0.*, git tag uses prefix, docker tags stay bare
  • Confirm STS runs before checkout with output-git-config: true and checkout uses steps.sts.outputs.GH_TOKEN
  • After merge, optionally dispatch Publish Legacy 14 once and verify the new git tag + ECR/Depot image tags

Made with Cursor

blumamir and others added 3 commits August 29, 2026 09:39
Use nodejs-community-14/v0.0.x git tags (Docker stays bare v0.0.x) and
authenticate checkout/push with the tag-releaser STS token + gitconfig.

Co-authored-by: Cursor <cursoragent@cursor.com>
Use bash -euo pipefail by default, reject empty versions/secrets, and
verify the git tag exists remotely after push.

Co-authored-by: Cursor <cursoragent@cursor.com>
Rely on Actions' default step failure; keep the scripts simple.

Co-authored-by: Cursor <cursoragent@cursor.com>
@blumamir
blumamir merged commit b85bfb5 into odigos-io:main Aug 29, 2026
7 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