Skip to content

ci(release): gate RC creation on strict-mode build of the tagged commit#4857

Merged
bobbai00 merged 2 commits into
apache:mainfrom
bobbai00:ci/4856-rc-strict-build
May 3, 2026
Merged

ci(release): gate RC creation on strict-mode build of the tagged commit#4857
bobbai00 merged 2 commits into
apache:mainfrom
bobbai00:ci/4856-rc-strict-build

Conversation

@bobbai00

@bobbai00 bobbai00 commented May 3, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this PR?

Add a `build` job at the top of `create-release-candidate.yml` that calls the reusable `build.yml` with:

`create-rc` now depends on `build` via `needs: build`. If the tag doesn't compile or has license-binary drift, no RC tarball is produced and nothing is signed/uploaded to ASF SVN.

Any related issues, documentation, discussions?

Closes #4856.

How was this PR tested?

Was this PR authored or co-authored using generative AI tooling?

Generated-by: Claude Code (claude-opus-4-7)

Add a build job at the top of create-release-candidate.yml that calls the
reusable build.yml with checkout_ref=<input.tag> and mode=release. create-rc
now depends on it via needs: build.

Closes the gap where an RC could be cut from a tag that doesn't compile
or has license-binary drift only the nightly cron would catch.

Closes apache#4856

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added the ci changes related to CI label May 3, 2026
@bobbai00 bobbai00 requested a review from aglinxinyuan May 3, 2026 09:41
@bobbai00 bobbai00 added the release/v1.1.0-incubating back porting to release/v1.1.0-incubating label May 3, 2026
@codecov-commenter

codecov-commenter commented May 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.40%. Comparing base (37d1341) to head (eaae718).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #4857      +/-   ##
============================================
+ Coverage     43.34%   43.40%   +0.05%     
- Complexity     2035     2108      +73     
============================================
  Files           957      957              
  Lines         34077    34946     +869     
  Branches       3753     3893     +140     
============================================
+ Hits          14772    15167     +395     
- Misses        18512    18975     +463     
- Partials        793      804      +11     
Flag Coverage Δ
access-control-service 28.12% <ø> (ø)
agent-service 33.49% <ø> (-0.24%) ⬇️
amber 41.95% <ø> (+0.41%) ⬆️
computing-unit-managing-service 0.00% <ø> (ø)
config-service 0.00% <ø> (ø)
file-service 32.40% <ø> (-0.85%) ⬇️
frontend 34.97% <ø> (-0.31%) ⬇️
python 85.21% <ø> (-0.14%) ⬇️
workflow-compiling-service 47.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

@bobbai00 bobbai00 added the emergency Pull requests that need to be merged ASAP label May 3, 2026
@bobbai00 bobbai00 enabled auto-merge (squash) May 3, 2026 10:04
@bobbai00 bobbai00 merged commit 31adc70 into apache:main May 3, 2026
37 checks passed
Yicong-Huang pushed a commit that referenced this pull request May 3, 2026
…it (#4857)

### What changes were proposed in this PR?

Add a \`build\` job at the top of \`create-release-candidate.yml\` that
calls the reusable \`build.yml\` with:

- \`checkout_ref: \${{ github.event.inputs.tag }}\` — build the tagged
commit (\`build.yml\` already accepts this input).
- \`mode: release\` — strict license-binary check, no
\`--ignore-transitive-version\` (the third mode added in #4734 alongside
\`PR\` and \`nightly\`).

\`create-rc\` now depends on \`build\` via \`needs: build\`. If the tag
doesn't compile or has license-binary drift, no RC tarball is produced
and nothing is signed/uploaded to ASF SVN.

### Any related issues, documentation, discussions?

Closes #4856.

### How was this PR tested?

### Was this PR authored or co-authored using generative AI tooling?

Generated-by: Claude Code (claude-opus-4-7)

(backported from commit 31adc70)

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented May 3, 2026

Copy link
Copy Markdown
Contributor

Backport to release/v1.1.0-incubating succeeded as a244f2a. Run

yangzhang75 pushed a commit to yangzhang75/texera that referenced this pull request Jun 22, 2026
…it (apache#4857)

### What changes were proposed in this PR?

Add a \`build\` job at the top of \`create-release-candidate.yml\` that
calls the reusable \`build.yml\` with:

- \`checkout_ref: \${{ github.event.inputs.tag }}\` — build the tagged
commit (\`build.yml\` already accepts this input).
- \`mode: release\` — strict license-binary check, no
\`--ignore-transitive-version\` (the third mode added in apache#4734 alongside
\`PR\` and \`nightly\`).

\`create-rc\` now depends on \`build\` via \`needs: build\`. If the tag
doesn't compile or has license-binary drift, no RC tarball is produced
and nothing is signed/uploaded to ASF SVN.


### Any related issues, documentation, discussions?

Closes apache#4856.


### How was this PR tested?


### Was this PR authored or co-authored using generative AI tooling?

Generated-by: Claude Code (claude-opus-4-7)

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci changes related to CI emergency Pull requests that need to be merged ASAP release/v1.1.0-incubating back porting to release/v1.1.0-incubating

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Gate RC creation on a strict-mode build of the tagged commit

3 participants