Skip to content

ARROW-10905: [Python] Add support for Python 3.9 Windows wheels - #8916

Closed
terencehonles wants to merge 3 commits into
apache:masterfrom
terencehonles:arrow-10905
Closed

ARROW-10905: [Python] Add support for Python 3.9 Windows wheels#8916
terencehonles wants to merge 3 commits into
apache:masterfrom
terencehonles:arrow-10905

Conversation

@terencehonles

Copy link
Copy Markdown
Contributor

Picking back up from #8386 in case I can handle this or someone else wants to work off of it. Likely will need to be updated based on #8881 / #8778.

@github-actions

Copy link
Copy Markdown

@nealrichardson

Copy link
Copy Markdown
Member

#8881 has merged now so I think this needs to be rebased.

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

#8881 has merged now so I think this needs to be rebased.

Thanks for the heads up @nealrichardson

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

backing out Sutou Kouhei's changes temporarily to see where things stand without the subsequent modifications.

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp38 wheel-win-cp39 conda-win-vs2017-py39

@github-actions

Copy link
Copy Markdown

Revision: dbbb36c78c17674912f02b1c0227cadc2e8e06a5

Submitted crossbow builds: ursa-labs/crossbow @ actions-849

TaskStatus
conda-win-vs2017-py39Azure
wheel-win-cp38Github Actions
wheel-win-cp39Github Actions

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp39

@github-actions

Copy link
Copy Markdown

Revision: 5cdc1d90d1b5a66b79b161ca9738351753e6b418

Submitted crossbow builds: ursa-labs/crossbow @ actions-850

TaskStatus
wheel-win-cp39Github Actions

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp39

@github-actions

Copy link
Copy Markdown

Revision: abb3bc027c5608ff59a24969ff21f25380935752

Submitted crossbow builds: ursa-labs/crossbow @ actions-851

TaskStatus
wheel-win-cp39Github Actions

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp39

@github-actions

Copy link
Copy Markdown

Revision: d4afd67e24eb5bbd0d525352db35af6393ceab76

Submitted crossbow builds: ursa-labs/crossbow @ actions-852

TaskStatus
wheel-win-cp39Github Actions

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp39

@github-actions

Copy link
Copy Markdown

Revision: 3d641a16c261d8fba103958606fa4b6f197b6483

Submitted crossbow builds: ursa-labs/crossbow @ actions-853

TaskStatus
wheel-win-cp39Github Actions

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp36m wheel-win-cp37m wheel-win-cp38

@github-actions

Copy link
Copy Markdown

Revision: 3d641a16c261d8fba103958606fa4b6f197b6483

Submitted crossbow builds: ursa-labs/crossbow @ actions-855

TaskStatus
wheel-win-cp36mGithub Actions
wheel-win-cp37mGithub Actions
wheel-win-cp38Github Actions

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-* conda-win-*

@github-actions

Copy link
Copy Markdown

Revision: 774ccd0e8f130dac9bef2debecad8b59f993cb15

Submitted crossbow builds: ursa-labs/crossbow @ actions-859

TaskStatus
conda-win-vs2017-py36-r36Azure
conda-win-vs2017-py37-r40Azure
conda-win-vs2017-py38Azure
conda-win-vs2017-py39Azure
wheel-win-cp36mGithub Actions
wheel-win-cp37mGithub Actions
wheel-win-cp38Github Actions
wheel-win-cp39Github Actions

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp36m

@github-actions

Copy link
Copy Markdown

Revision: e218e7e0e81fdff71a0e9f573c550adade797199

Submitted crossbow builds: ursa-labs/crossbow @ actions-860

TaskStatus
wheel-win-cp36mGithub Actions

Comment threaddev/tasks/python-wheels/win-build.bat Outdated
@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp36m

@github-actions

Copy link
Copy Markdown

Revision: 5b0c9f0a210bcbdb9a04e069bbfc4ba9ceccf0b6

Submitted crossbow builds: ursa-labs/crossbow @ actions-862

TaskStatus
wheel-win-cp36mGithub Actions

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp36m

@github-actions

Copy link
Copy Markdown

Revision: 6adf5c21621af57987a605d0fc97eed111706fcb

Submitted crossbow builds: ursa-labs/crossbow @ actions-863

TaskStatus
wheel-win-cp36mGithub Actions

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-* conda-win-*

@github-actions

Copy link
Copy Markdown

Revision: 56c378b

Submitted crossbow builds: ursa-labs/crossbow @ actions-864

TaskStatus
conda-win-vs2017-py36-r36Azure
conda-win-vs2017-py37-r40Azure
conda-win-vs2017-py38Azure
conda-win-vs2017-py39Azure
wheel-win-cp36mGithub Actions
wheel-win-cp37mGithub Actions
wheel-win-cp38Github Actions
wheel-win-cp39Github Actions

@terencehonles

terencehonles commented Jan 9, 2021

Copy link
Copy Markdown
ContributorAuthor

@xhochy when you get a chance it looks like #8881 did not fix building on Python 3.6 as with my change and without my change (master) both seem to fail on finding OpenSSL.

I'll try to see if I can poke around further, but not sure how far I can get.

@terencehonles

terencehonles commented Jan 9, 2021

Copy link
Copy Markdown
ContributorAuthor

@kou any chance you know what to do about:

 File "arrow/dev/tasks/crossbow.py", line 1662, in upload_artifacts
tag_name=tag, target_commitish=sha, method=method, patterns=patterns
File "arrow/dev/tasks/crossbow.py", line 460, in github_overwrite_release_assets
repo = self.as_github_repo()
File "arrow/dev/tasks/crossbow.py", line 384, in as_github_repo
self._github_repo = github.repository(username, reponame)
File "C:\Miniconda\lib\site-packages\github3\github.py", line 1981, in repository
json = self._json(self._get(url), 200)
File "C:\Miniconda\lib\site-packages\github3\models.py", line 156, in _json
raise exceptions.error_for(response)
github3.exceptions.AuthenticationFailed: 401 Bad credentials

from https://dev.azure.com/ursa-labs/crossbow/_build/results?buildId=22650&view=logs&j=4c86bc1b-1091-5192-4404-c74dfaad23e7&t=ec99a26b-0264-5e86-36fb-9cfd0ca0f9f3&l=37 (conda builds)

I probably didn't need to rebuild everything since that step will probably fail across the board (I'll check each of them since I hadn't done that in #8916 (comment) )

@xhochy

Copy link
Copy Markdown
Member

@xhochy when you get a chance it looks like #8881 did not fix building on Python 3.6 as with my change and without my change (master) both seem to fail on finding OpenSSL.

I'll try to see if I can poke around further, but not sure how far I can get.

The OpenSSL issue is not problem. This just skips the SSL-supporting part of thrift, we don't need that. I actually fail to parse the log to find what is actually the issue though.

@xhochy

Copy link
Copy Markdown
Member

@xhochy when you get a chance it looks like #8881 did not fix building on Python 3.6 as with my change and without my change (master) both seem to fail on finding OpenSSL.
I'll try to see if I can poke around further, but not sure how far I can get.

The OpenSSL issue is not problem. This just skips the SSL-supporting part of thrift, we don't need that. I actually fail to parse the log to find what is actually the issue though.

OpenSSL is an issue but for gRPC!

Comment threaddev/tasks/python-wheels/win-build.bat Outdated
@kou

kou commented Jan 9, 2021

Copy link
Copy Markdown
Member

@kou any chance you know what to do about:

It seems that we need to update a secret variable registered in https://dev.azure.com/ursa-labs/crossbow/ but I don't have permission to update it. It seems that we don't care about conda-win-* tasks in this pull request because we don't touch anything for conda-win-* in this pull request.

@kou

kou commented Jan 9, 2021

Copy link
Copy Markdown
Member

It may be better that we wait for #9096 because it uses vcpkg instead of conda.
It doesn't have the OpenSSL problem: #9096 (comment)

Co-authored-by: Uwe L. Korn <xhochy@users.noreply.github.com>
@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp36m wheel-win-cp39

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

It may be better that we wait for #9096 because it uses vcpkg instead of conda.
It doesn't have the OpenSSL problem: #9096 (comment)

Ok, I'll just test out the suggestion. Not sure when @kszucs plans on trying to get #9096 in.

@github-actions

Copy link
Copy Markdown

Revision: 5df8d6b

Submitted crossbow builds: ursa-labs/crossbow @ actions-865

TaskStatus
wheel-win-cp36mGithub Actions
wheel-win-cp39Github Actions

@kszucs

Copy link
Copy Markdown
Member

@kou any chance you know what to do about:

It seems that we need to update a secret variable registered in https://dev.azure.com/ursa-labs/crossbow/ but I don't have permission to update it. It seems that we don't care about conda-win-* tasks in this pull request because we don't touch anything for conda-win-* in this pull request.

I updated the secret and also invited you to the project.

@terencehonles

Copy link
Copy Markdown
ContributorAuthor

@github-actions crossbow submit wheel-win-cp36m

@github-actions

Copy link
Copy Markdown

Revision: c532cd1

Submitted crossbow builds: ursa-labs/crossbow @ actions-871

TaskStatus
wheel-win-cp36mGithub Actions

@kszucs

Copy link
Copy Markdown
Member

Superseded by #9096.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@terencehonles@nealrichardson@xhochy@kou@kszucs