Skip to content

[CI] CI: set release publish repository context - #5204

Merged
gyohuangxin merged 1 commit into
mainfrom
ci/fix-release-publish-gh-repo
Sep 2, 2026
Merged

gyohuangxin merged 1 commit into
mainfrom
ci/fix-release-publish-gh-repo

Conversation

@gyohuangxin

Copy link
Copy Markdown
Member

Summary

  • Set GH_REPO in the release automation publish job so gh release commands have repository context even without a checkout.
  • Fixes the final publish failure from run 33586350146 where gh release view failed with fatal: not a git repository.

Test plan

  • python3 -c "import yaml; yaml.safe_load(open('.github/workflows/release-automation.yaml')); print('yaml ok')"
  • git diff --check

@gyohuangxin
gyohuangxin requested review from a team and a lite review from Copilot September 2, 2026 05:43
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

🏷️ CI Guide

Runs automatically on every PR:

  • ✅ Pre-checks (submodule verification, code formatting)
  • ✅ Aiter op tests (gfx942 + gfx950)
  • ✅ Triton tests on MI35X (only when aiter/ops/triton/** or related paths are changed)

Extended tests (opt-in via labels):

Label Tests
ci:gfx1250-ffm-triton Run the five-shard gfx1250 FFM Triton test suite
ci:triton-300x Run an additional Triton test job on MI300X in PRs; main branch always runs both MI35X and MI300X
multigpu Aiter multi-GPU tests on the 8-GPU runner
ci:sglang SGLang integration tests: DeepSeek-R1-MXFP4 accuracy, Qwen 3.5 accuracy
ci:atom ATOM benchmark: DeepSeek-R1-0528, GPT-OSS-120B
ci:atom_full ATOM accuracy suite for PR and main models from ATOM models_accuracy.json
ci:vllm vLLM benchmark: GPT-OSS-120B, DeepSeek-R1-0528, Kimi-K2.5
ci:all All standard extended tests (excludes ci:atom_full)

Only add ci:atom_full for FlyDSL or Triton upgrades.
Add labels via the sidebar or gh pr edit 5204 --add-label <label>

PR title tags & labels:
Component tags ([Triton/Gluon], [HIP], [CK], [ASM], ...) are added to the PR title and as PR labels automatically from the changed files and re-synced on every push — change-type tags like [fix]/[Perf], op tags like [MLA], and human labels (ci:*) are left untouched. Add the no-auto-title label to opt this PR out.

@github-actions github-actions Bot changed the title CI: set release publish repository context [CI] CI: set release publish repository context Sep 2, 2026
@github-actions github-actions Bot added the CI label Sep 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.

🟢 Approval recommended

The change is minimal, correctly scoped to the failing job, and aligns with GitHub CLI’s expected environment-based repository context.

Pull request overview

This PR fixes the release publish job in .github/workflows/release-automation.yaml by ensuring the GitHub CLI has explicit repository context even when the job does not check out the repo, preventing failures like fatal: not a git repository during gh release view.

Changes:

  • Set GH_REPO to ${{ github.repository }} in the publish job environment so gh release * commands can resolve the target repository without relying on a local .git checkout.
File summaries
File Description
.github/workflows/release-automation.yaml Adds GH_REPO to the publish job env so gh release view/edit/create/upload works without a checkout.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@gyohuangxin
gyohuangxin merged commit 62f9a50 into main Sep 2, 2026
34 checks passed
@gyohuangxin
gyohuangxin deleted the ci/fix-release-publish-gh-repo branch September 2, 2026 05:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants