[WIP] Add nightly GitHub CI workflow to fetch main artifact by Copilot · Pull Request #1532 · kvcache-ai/Mooncake

Copilot · 2026-02-10T11:54:36Z

Create a nightly GitHub CI workflow (nightly.yml) that:
- Runs on a schedule (daily at 08:00 UTC) and supports manual dispatch
- Fetches the latest artifacts from the most recent successful CI workflow run on main
- Downloads wheels from ci.yml (CUDA 12.8) and ci_cu13.yml (CUDA 13) workflow runs
- Creates a nightly pre-release on GitHub with the fetched wheel artifacts
- Uses nightly-YYYYMMDD tag naming convention
- Handles cleanup of existing same-day nightly releases
- Validates that at least some artifacts were downloaded before proceeding
Validate workflow YAML syntax
Address code review feedback (fixed glob check, added download validation)
Run security checks

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…GitHub release Co-authored-by: stmatengss <11641725+stmatengss@users.noreply.github.com>

Co-authored-by: stmatengss <11641725+stmatengss@users.noreply.github.com>

stmatengss · 2026-02-10T12:01:28Z

/gemini review

Copilot

Pull request overview

Adds a scheduled GitHub Actions workflow to automatically publish a nightly pre-release by downloading wheel artifacts from the latest successful CI runs on main.

Changes:

Introduces .github/workflows/nightly.yml scheduled + manual workflow to fetch artifacts from ci.yml and ci_cu13.yml.
Extracts downloaded artifacts and creates/overwrites a same-day nightly-YYYYMMDD GitHub pre-release with wheel assets.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-10T12:14:17Z

.github/workflows/nightly.yml

+                console.log(`Downloading artifact: ${artifact.name}`);
+                const download = await github.rest.actions.downloadArtifact({
+                  owner: context.repo.owner,
+                  repo: context.repo.repo,
+                  artifact_id: artifact.id,
+                  archive_format: 'zip',
+                });
+
+                const artifactDir = path.join('nightly-wheels', artifact.name);
+                fs.mkdirSync(artifactDir, { recursive: true });
+                const zipPath = path.join(artifactDir, `${artifact.name}.zip`);
+                fs.writeFileSync(zipPath, Buffer.from(download.data));
+                console.log(`Saved to ${zipPath}`);


The artifact download loop doesn’t handle per-artifact download failures. If a single artifact download fails (expired artifact, transient API issue, etc.), the whole step will throw and the workflow will abort without a clear summary. Consider wrapping downloadArtifact/writeFileSync in a try/catch, logging the failure, and continuing while still enforcing the final downloaded === 0 guard.

Copilot · 2026-02-10T12:14:17Z

.github/workflows/nightly.yml

+            try {
+              const release = await github.rest.repos.getReleaseByTag({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                tag: tag,
+              });
+              await github.rest.repos.deleteRelease({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                release_id: release.data.id,
+              });
+              console.log(`Deleted existing release for tag ${tag}`);
+            } catch (e) {
+              console.log(`No existing release found for tag ${tag}`);
+            }


Both deletion try/catch blocks treat any error as “not found”. This can mask real failures (permissions, rate limiting, API outage) and lead to confusing downstream errors when creating the new release/tag. Please only swallow 404/Not Found and rethrow (or core.setFailed) for other statuses.

Copilot · 2026-02-10T12:14:18Z

.github/workflows/nightly.yml

+            try {
+              await github.rest.git.deleteRef({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                ref: `tags/${tag}`,
+              });
+              console.log(`Deleted existing tag ${tag}`);
+            } catch (e) {
+              console.log(`No existing tag found: ${tag}`);
+            }


Same as above for tag deletion: catching all exceptions and logging “No existing tag found” will hide non-404 errors. Please check the error status/code and only ignore the expected not-found case.

Copilot · 2026-02-10T12:14:18Z

.github/workflows/nightly.yml

+            }
+
+      - name: Upload wheels to GitHub Nightly Release
+        uses: softprops/action-gh-release@v1


For supply-chain security, consider pinning third-party actions (e.g., softprops/action-gh-release) to a specific commit SHA (or at least a full semver tag) instead of a moving major tag. This reduces the risk of a compromised upstream release affecting your release automation.

Suggested change

uses: softprops/action-gh-release@v1

uses: softprops/action-gh-release@v1.10.0

Copilot · 2026-02-10T12:14:18Z

.github/workflows/nightly.yml

+          echo "Found wheel files:"
+          ls -la nightly-release/*.whl
+


The artifacts uploaded from CI preserve their relative paths (e.g., mooncake-wheel/dist-py…/*.whl), so after unzip the wheels will typically be under subdirectories of nightly-release/, not directly in nightly-release/. This ls nightly-release/*.whl will fail even when wheels exist (your preceding find check would pass). Consider either flattening wheels into a single directory (like the release workflows do with find … -exec cp) or updating the listing command to use find.

Copilot · 2026-02-10T12:14:18Z

.github/workflows/nightly.yml

+            Generated at: ${{ steps.tag.outputs.nightly_tag }}
+            This is an automated nightly release and may be unstable.
+          prerelease: true
+          files: nightly-release/*.whl


softprops/action-gh-release is configured to upload nightly-release/*.whl, but after unzipping CI artifacts the wheels are likely nested under subdirectories (because CI uploads from mooncake-wheel/dist-py…/*.whl). As written, the release can be created with zero uploaded assets. Suggest flattening wheels into a dedicated folder (and upload from there) or using a recursive glob that the action supports.

Copilot · 2026-02-10T12:14:19Z

.github/workflows/nightly.yml

+jobs:
+  nightly-release:
+    runs-on: ubuntu-22.04
+    permissions:
+      contents: write
+      actions: read


Because this workflow deletes and recreates a shared tag/release (nightly-YYYYMMDD), concurrent runs (e.g., a manual dispatch overlapping the scheduled run) can race and interfere with each other. Consider adding a concurrency group (and possibly cancel-in-progress: true) to ensure only one nightly-release job runs at a time.

Copilot · 2026-02-10T12:14:19Z

.github/workflows/nightly.yml

+
+on:
+  schedule:
+    # Run at 08:00 UTC every day (midnight PST)


The schedule comment is inaccurate during daylight saving time: 08:00 UTC is midnight PST only in winter, but 01:00 PDT in summer. Please update the comment to a DST-agnostic phrasing (e.g., “00:00 Pacific Time (08:00 UTC)” or just “08:00 UTC”).

Suggested change

# Run at 08:00 UTC every day (midnight PST)

# Run at 08:00 UTC every day (00:00 Pacific Time)

codecov-commenter · 2026-02-10T12:20:47Z

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Initial plan

b3fcf1a

Copilot AI assigned Copilot and stmatengss Feb 10, 2026

Copilot started work on behalf of stmatengss February 10, 2026 11:57 View session

Copilot AI and others added 2 commits February 10, 2026 11:59

Add nightly CI workflow to fetch latest main artifacts and upload to …

2fcff6c

…GitHub release Co-authored-by: stmatengss <11641725+stmatengss@users.noreply.github.com>

Address code review: fix glob check and add artifact download validation

ce124d4

Co-authored-by: stmatengss <11641725+stmatengss@users.noreply.github.com>

stmatengss marked this pull request as ready for review February 10, 2026 12:00

stmatengss requested review from Ann-1024, stmatengss and ykwd as code owners February 10, 2026 12:00

Copilot AI requested review from Ann-1024, stmatengss and ykwd as code owners February 10, 2026 12:00

stmatengss closed this Feb 10, 2026

github-actions bot added the run-ci label Feb 10, 2026

stmatengss reopened this Feb 10, 2026

Copilot stopped work on behalf of stmatengss due to an error February 10, 2026 12:00
The session was cancelled by the user.

stmatengss requested a review from Copilot February 10, 2026 12:03

Copilot started reviewing on behalf of stmatengss February 10, 2026 12:11 View session

Copilot AI reviewed Feb 10, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

[WIP] Add nightly GitHub CI workflow to fetch main artifact#1532

[WIP] Add nightly GitHub CI workflow to fetch main artifact#1532
Copilot wants to merge 3 commits intomainfrom
copilot/add-nightly-ci-workflow

Copilot AI commented Feb 10, 2026 •

edited

Loading

Uh oh!

stmatengss commented Feb 10, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 10, 2026

Uh oh!

Copilot AI Feb 10, 2026

Uh oh!

Copilot AI Feb 10, 2026

Uh oh!

Copilot AI Feb 10, 2026

Uh oh!

Copilot AI Feb 10, 2026

Uh oh!

Copilot AI Feb 10, 2026

Uh oh!

Copilot AI Feb 10, 2026

Uh oh!

Copilot AI Feb 10, 2026

Uh oh!

codecov-commenter commented Feb 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	uses: softprops/action-gh-release@v1
	uses: softprops/action-gh-release@v1.10.0

	# Run at 08:00 UTC every day (midnight PST)
	# Run at 08:00 UTC every day (00:00 Pacific Time)

Comments

Conversation

Copilot AI commented Feb 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

stmatengss commented Feb 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

Uh oh!

codecov-commenter commented Feb 10, 2026

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Feb 10, 2026 •

edited

Loading

stmatengss commented Feb 10, 2026 •

edited

Loading