Skip to content

Comments

ci(python): fix bindings wheel install glob in CI#2165

Closed
kevinjqliu wants to merge 2 commits intoapache:mainfrom
kevinjqliu:kevinjqliu/fix-whl-name-in-ci
Closed

ci(python): fix bindings wheel install glob in CI#2165
kevinjqliu wants to merge 2 commits intoapache:mainfrom
kevinjqliu:kevinjqliu/fix-whl-name-in-ci

Conversation

@kevinjqliu
Copy link
Contributor

@kevinjqliu kevinjqliu commented Feb 23, 2026

Which issue does this PR close?

  • Closes #.

What changes are included in this PR?

Update the wheel install pattern in the Python bindings CI test job to match the actual wheel artifact name from maturin (pyiceberg_core_rust-*.whl).

Found this in #2164's CI

Tested this locally

➜ uvx maturin build --out dist --sdist
...
➜  ll dist                                                     
total 94592
-rw-r--r--@ 1 kevinliu  staff   648K Feb 22 18:22 pyiceberg_core-0.8.0.tar.gz
-rw-r--r--@ 1 kevinliu  staff    45M Feb 22 18:23 pyiceberg_core_rust-0.8.0-cp310-abi3-macosx_11_0_arm64.whl

Are these changes tested?

@kevinjqliu
Copy link
Contributor Author

weird, using uv run create the correct artifact name

➜  uv run maturin build --out dist --sdist
...
➜  python git:(kevinjqliu/fix-whl-name-in-ci) ll dist 
total 89872
-rw-r--r--@ 1 kevinliu  staff    44M Feb 22 18:58 pyiceberg_core-0.8.0-cp310-abi3-macosx_11_0_arm64.whl

@kevinjqliu
Copy link
Contributor Author

turns out, this is a red herring

@kevinjqliu kevinjqliu closed this Feb 23, 2026
@kevinjqliu kevinjqliu deleted the kevinjqliu/fix-whl-name-in-ci branch February 23, 2026 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant