Skip to content

ci: align package_c TensorFlow image and wheel to 2.20#5297

Merged
njzjz merged 4 commits intodeepmodeling:masterfrom
njzjz-bot:fix/package-c-tf-2.21-pin
Mar 7, 2026
Merged

ci: align package_c TensorFlow image and wheel to 2.20#5297
njzjz merged 4 commits intodeepmodeling:masterfrom
njzjz-bot:fix/package-c-tf-2.21-pin

Conversation

@njzjz-bot
Copy link
Contributor

@njzjz-bot njzjz-bot commented Mar 7, 2026

Summary

  • switch the package_c Docker build image from TensorFlow 2.18 to 2.20
  • pin the installed TensorFlow wheel to ==2.20.* via the existing TENSORFLOW_VERSION env

Why

The failing Build C library job appears to be in package_c.yml, not in the cibuildwheel workflow changed by dependabot.

This change makes the Docker image and installed TensorFlow version move together, so we can test whether the failure was caused by a TensorFlow / GLIBC mismatch from pulling a newer wheel into an older build image.

Authored by OpenClaw (model: gpt-5.4)

Summary by CodeRabbit

  • Chores
    • Updated C library build configuration to target TensorFlow 2.20, aligning the build matrix with the newer TensorFlow runtime.

@dosubot dosubot bot added the build label Mar 7, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: ea7139b5-8774-4705-bfcd-31823e613f01

📥 Commits

Reviewing files that changed from the base of the PR and between e0d357d and d801888.

📒 Files selected for processing (1)
  • .github/workflows/package_c.yml

📝 Walkthrough

Walkthrough

Updated the Build C library CI workflow matrix in .github/workflows/package_c.yml: tensorflow_build_version changed from 2.18 to 2.20 and tensorflow_version set to ==2.20.*. No other workflow steps, conditional logic, or exported declarations were modified.

Changes

Cohort / File(s) Summary
CI Configuration
.github/workflows/package_c.yml
Updated matrix entries: tensorflow_build_version from 2.182.20; tensorflow_version set from ""==2.20.*. No other changes to steps or logic.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Possibly related PRs

Suggested reviewers

  • wanghan-iapcm
  • iProzd
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'ci: align package_c TensorFlow image and wheel to 2.20' directly reflects the main change: updating the TensorFlow build matrix version from 2.18 to 2.20 in the package_c workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/package_c.yml:
- Around line 24-25: The workflow uses a non-existent Docker image tag
(tensorflow/build:2.21-python3.11) via the
tensorflow_build_version/tensorflow_version entries in package_c.yml; revert the
tensorflow_build_version to a published tag (or pin tensorflow_version to an
available version) and/or wait for TensorFlow to publish 2.21 before changing
these values, and then update the test matrices in test_cc.yml and
test_python.yml to include 2.21 as well so C library builds and integration
tests run against the same TensorFlow version once the image is available.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: d949bcd4-bab9-4170-9550-a40c7050c5ad

📥 Commits

Reviewing files that changed from the base of the PR and between 8f2b3c9 and 17cf613.

📒 Files selected for processing (1)
  • .github/workflows/package_c.yml

@codecov
Copy link

codecov bot commented Mar 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.32%. Comparing base (8f2b3c9) to head (d801888).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5297   +/-   ##
=======================================
  Coverage   82.31%   82.32%           
=======================================
  Files         768      768           
  Lines       77098    77097    -1     
  Branches     3660     3660           
=======================================
+ Hits        63467    63469    +2     
+ Misses      12458    12457    -1     
+ Partials     1173     1171    -2     

☔ 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.

@njzjz-bot njzjz-bot changed the title ci: align package_c TensorFlow image and wheel to 2.21 ci: try TensorFlow 2.20 build image for package_c Mar 7, 2026
@njzjz-bot njzjz-bot changed the title ci: try TensorFlow 2.20 build image for package_c ci: align package_c TensorFlow image and wheel to 2.20 Mar 7, 2026
@njzjz njzjz enabled auto-merge March 7, 2026 11:19
@njzjz njzjz added this pull request to the merge queue Mar 7, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 7, 2026
@njzjz njzjz added this pull request to the merge queue Mar 7, 2026
Merged via the queue into deepmodeling:master with commit 084edd6 Mar 7, 2026
70 checks passed
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