Skip to content

GH-49623: [CI][Python] Install built wheel on Windows verification and test in isolation#49624

Merged
raulcd merged 3 commits intoapache:mainfrom
raulcd:GH-49623
Mar 31, 2026
Merged

GH-49623: [CI][Python] Install built wheel on Windows verification and test in isolation#49624
raulcd merged 3 commits intoapache:mainfrom
raulcd:GH-49623

Conversation

@raulcd
Copy link
Copy Markdown
Member

@raulcd raulcd commented Mar 31, 2026

Rationale for this change

Verification job is currently failing on Windows.

What changes are included in this PR?

With the changes on sckit-build-core we changed how we build the wheel on Windows verification. Previously we were doing editable installations. For verification we should install the built wheel and test on isolation, not in-source.

Are these changes tested?

Yes via CI

Are there any user-facing changes?

No

@raulcd
Copy link
Copy Markdown
Member Author

raulcd commented Mar 31, 2026

@github-actions crossbow submit verify-rc-source-windows

@github-actions
Copy link
Copy Markdown

⚠️ GitHub issue #49623 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Mar 31, 2026
@github-actions
Copy link
Copy Markdown

Revision: 52c1e80

Submitted crossbow builds: ursacomputing/crossbow @ actions-21ee3a48ec

Task Status
verify-rc-source-windows GitHub Actions

@raulcd
Copy link
Copy Markdown
Member Author

raulcd commented Mar 31, 2026

@github-actions crossbow submit verify-rc-source-windows

@github-actions
Copy link
Copy Markdown

Revision: c86f2ec

Submitted crossbow builds: ursacomputing/crossbow @ actions-d1e4a1c363

Task Status
verify-rc-source-windows GitHub Actions

@raulcd
Copy link
Copy Markdown
Member Author

raulcd commented Mar 31, 2026

@github-actions crossbow submit verify-rc-source-windows

@github-actions
Copy link
Copy Markdown

Revision: 4e5fcbe

Submitted crossbow builds: ursacomputing/crossbow @ actions-157c1a6a9f

Task Status
verify-rc-source-windows GitHub Actions

@raulcd raulcd marked this pull request as ready for review March 31, 2026 14:42
Copy link
Copy Markdown
Member Author

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

This fixes CI for Windows verification and is related to the changes I did for scikit-build-core. I'll just merge it. Happy to follow up if necessary.

@raulcd raulcd merged commit 5c78ddf into apache:main Mar 31, 2026
10 checks passed
@raulcd raulcd removed the awaiting committer review Awaiting committer review label Mar 31, 2026
@github-actions github-actions bot added the awaiting changes Awaiting changes label Mar 31, 2026
@raulcd raulcd deleted the GH-49623 branch March 31, 2026 14:48
@conbench-apache-arrow
Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit 5c78ddf.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 25 possible false positives for unstable benchmarks that are known to sometimes produce them.

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

Labels

awaiting changes Awaiting changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant