Skip to content

Add release build output canaries - #2400

Draft
msarahan wants to merge 2 commits into
mainfrom
codex/release-build-output-manifests
Draft

Add release build output canaries#2400
msarahan wants to merge 2 commits into
mainfrom
codex/release-build-output-manifests

Conversation

@msarahan

@msarahan msarahan commented Aug 3, 2026

Copy link
Copy Markdown
Member

Posted by Codex (GPT-5) on behalf of @msarahan. This PR description is LLM-generated; readers should treat its content accordingly.

Summary

  • point the two Conda and two wheel producer jobs at rapidsai/shared-workflows@codex/release-build-output-manifests and opt them into companion generation
  • opt the matrixed libcuvs_c and Java custom-job producers into the same producer-side dispatch action
  • remove the separate post-build manifest jobs; companions are now generated from the original local files in the producer-owned matrix
  • emit exact build-time package JSON sidecars for both custom bundles and select only the CUDA-classified release JAR
  • apply the rollout consistently in both build.yaml and pr.yaml

Expected canary coverage

  • four Conda C++ companions
  • four Conda Python companions
  • four libcuvs wheel companions
  • four cuvs wheel companions
  • four standalone libcuvs_c archive companions
  • two Java companions

The previous implementation produced and independently validated all 22 source/companion pairs. This revision verifies that the shared-actions dispatch refactor preserves the same contract without an extra runner or artifact download.

Evidence semantics

These producers do not supply dependency SBOMs. Their companions therefore contain generated-identity SPDX envelopes, which identify each primary artifact and checksum but must not be reported as dependency coverage.

Tracks rapidsai/build-infra#381, rapidsai/shared-workflows#609, and rapidsai/shared-actions#134. Neither shared development branch must merge to main for this canary.

Validation

  • full pre-commit completed with only the unrelated locally installed Cargo 1.82 edition2024 incompatibility
  • pre-commit run --files .github/workflows/build.yaml .github/workflows/pr.yaml
  • git diff --check

This remains a draft with DO NOT MERGE until the dispatch-based branch-run artifacts are inspected and recorded in the rollout issue.

@copy-pr-bot

copy-pr-bot Bot commented Aug 3, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@msarahan msarahan added DO NOT MERGE feature request New feature or request non-breaking Introduces a non-breaking change ci labels Aug 3, 2026
@msarahan

msarahan commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

/ok to test bdeac96

@msarahan

msarahan commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

/ok to test 8f3d55e

@codecov-commenter

codecov-commenter commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.93%. Comparing base (be8ab31) to head (f0015ac).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2400   +/-   ##
=======================================
  Coverage   86.93%   86.93%           
=======================================
  Files          33       33           
  Lines         176      176           
=======================================
  Hits          153      153           
  Misses         23       23           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@msarahan
msarahan force-pushed the codex/release-build-output-manifests branch from 8f3d55e to 19b1073 Compare August 4, 2026 14:53
@msarahan

msarahan commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

/ok to test

1 similar comment
@msarahan

msarahan commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

/ok to test

@msarahan

msarahan commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

/ok to test 19b1073

@msarahan
msarahan force-pushed the codex/release-build-output-manifests branch from 19b1073 to f0015ac Compare August 4, 2026 15:05
@msarahan

msarahan commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

/ok to test f0015ac

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci DO NOT MERGE feature request New feature or request non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants