Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI][Python] Crossbow wheel artifacts for Windows #36450

Closed
pitrou opened this issue Jul 4, 2023 · 2 comments · Fixed by #36466
Closed

[CI][Python] Crossbow wheel artifacts for Windows #36450

pitrou opened this issue Jul 4, 2023 · 2 comments · Fixed by #36466

Comments

@pitrou
Copy link
Member

pitrou commented Jul 4, 2023

Describe the enhancement requested

Non-Windows crossbow workflows publish their wheel artifacts, examples:

However, for some reason there are no wheel artifacts on similar Windows build, example:
https://github.com/ursacomputing/crossbow/actions/runs/5452999747

Component(s)

Continuous Integration, Python

@pitrou
Copy link
Member Author

pitrou commented Jul 4, 2023

@raulcd

@raulcd
Copy link
Member

raulcd commented Jul 4, 2023

I'll investigate why they don't appear on the summary because the assets are still published as a release:
https://github.com/ursacomputing/crossbow/releases/tag/actions-1c2290f238-github-wheel-windows-cp39-amd64

@kou kou closed this as completed in #36466 Jul 5, 2023
kou added a commit that referenced this issue Jul 5, 2023
### Rationale for this change

It's useful for debugging on local.

### What changes are included in this PR?

Add `actions/upload-artifact` step like other jobs for manylinux and macOS.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* Closes: #36450

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou kou added this to the 13.0.0 milestone Jul 5, 2023
westonpace pushed a commit to westonpace/arrow that referenced this issue Jul 7, 2023
…he#36466)

### Rationale for this change

It's useful for debugging on local.

### What changes are included in this PR?

Add `actions/upload-artifact` step like other jobs for manylinux and macOS.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* Closes: apache#36450

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants