Skip to content

fix release workflow - #143

Merged
InftyAI-Agent merged 2 commits into
InftyAI:mainfrom
kerthcet:ci/release-existing-release
Nov 2, 2026
Merged

InftyAI-Agent merged 2 commits into
InftyAI:mainfrom
kerthcet:ci/release-existing-release

Conversation

@kerthcet

@kerthcet kerthcet commented Nov 2, 2026

Copy link
Copy Markdown
Member

What this PR does / why we need it

Which issue(s) this PR fixes

Fixes #

Special notes for your reviewer

Does this PR introduce a user-facing change?


Signed-off-by: kerthcet <kerthcet@gmail.com>
Signed-off-by: kerthcet <kerthcet@gmail.com>
Copilot AI balanced review requested due to automatic review settings November 2, 2026 23:39
@kerthcet

kerthcet commented Nov 2, 2026

Copy link
Copy Markdown
Member Author

/lgtm
/kind bug

@InftyAI-Agent InftyAI-Agent added needs-triage Indicates an issue or PR lacks a label and requires one. needs-priority Indicates a PR lacks a label and requires one. do-not-merge/needs-kind Indicates a PR lacks a label and requires one. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Nov 2, 2026
@InftyAI-Agent InftyAI-Agent added lgtm Looks good to me, indicates that a PR is ready to be merged. bug Categorizes issue or PR as related to a bug. and removed do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Nov 2, 2026

Copilot AI left a comment

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.

🟡 Changes recommended

Publishing a release in the GitHub UI does not emit the required tag push event, so removing manual dispatch leaves that release path without image builds, manifest upload, or a run to retry.

1 open issue
What changed in this PR

Updates the tag-driven release workflow to handle existing GitHub releases without overwriting their metadata.

Changes:

  • Removes manual dispatch and derives the version directly from the triggering tag.
  • Uploads or replaces the install manifest when a release already exists, preserving its title, notes, and pre-release status.
  • Updates comments to describe release triggers and retry behavior.
File Description
.github/​workflows/​release.yml Simplifies version selection and adds manifest uploads for existing releases.

Review effort: Balanced

Comment thread .github/workflows/release.yml Resolved

@InftyAI-Agent InftyAI-Agent left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved: PR has both lgtm and approved labels

@InftyAI-Agent InftyAI-Agent left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved: PR has both lgtm and approved labels

@InftyAI-Agent
InftyAI-Agent merged commit 5a309d1 into InftyAI:main Nov 2, 2026
44 of 46 checks passed
@kerthcet
kerthcet deleted the ci/release-existing-release branch November 3, 2026 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. bug Categorizes issue or PR as related to a bug. lgtm Looks good to me, indicates that a PR is ready to be merged. needs-priority Indicates a PR lacks a label and requires one. needs-triage Indicates an issue or PR lacks a label and requires one.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants