WIP: [Release] Verify release-19.0.1-rc0#45420
Closed
kou wants to merge 4 commits into
Closed
Conversation
Member
Author
|
@github-actions crossbow submit --group verify-rc-source --param release=19.0.1 --param rc=0 |
kou
requested review from
CurtHagenlocher,
assignUser,
domoritz,
jonkeane,
kevingurney,
raulcd,
sgilmore10,
thisisnic and
trxcllnt
as code owners
|
Revision: 36e0d50 Submitted crossbow builds: ursacomputing/crossbow @ actions-a726d6e4f7
|
Member
Author
|
Revision: apache-arrow-19.0.1-rc0 Submitted crossbow builds: ursacomputing/crossbow @ release-19.0.1-rc0-0
|
Member
Author
|
Hmm. |
Member
Author
|
We may need to backport something for it. |
Member
|
It seems that the nightly wheels are also not working but the jobs are not failing now. For example for the latest nightlies Python tests are not really executed on those jobs! They seem to finish abruptly:
|
Member
|
My first thought after some searching around was that IANA may have changed how they issues their TLS cert (which is still possible) and that we just need a newer version of the certifi package but the jobs appear to be pulling in recent ones. I'm seeing if I can reproduce it locally. I wonder if we may just need to update the base image. |
Member
|
I had hoped to reproduce the issue but wasn't able to. I built the image locally, $ $env:DOCKER_BUILDKIT = 0
archery docker build --no-pull python-wheel-windows-vs2019and ran this: |
Member
|
@github-actions crossbow submit wheel-windows-cp39-amd64 |
|
Revision: 36e0d50 Submitted crossbow builds: ursacomputing/crossbow @ actions-42cab651f2
|
Member
Author
|
It seems that https://cygwin.osuosl.org/noarch/release/tzdata/tzdata-2024a-1.tar.xz removed. We need to use more portable URL. |
Member
|
Thanks @kou. Does it make sense for us to host a copy ourselves? |
Member
Author
|
Hmm... I think that we don't want to update tzdata periodically... We may be able to use https://pypi.org/project/tzdata/ . |
Member
|
That seems like it could work, ❯ python -c "import os; from importlib import resources; print(os.path.join(resources.files('tzdata'), 'zoneinfo'));"
C:\Users\Bryce\Temp\tzinfo-testing\venv\Lib\site-packages\tzdata\zoneinfo
❯ dir C:\Users\Bryce\Temp\tzinfo-testing\venv\Lib\site-packages\tzdata\zoneinfo
Directory: C:\Users\Bryce\Temp\tzinfo-testing\venv\Lib\site-packages\tzdata\zoneinfo
... |
Member
Author
|
Wow! Could you open a PR that uses https://pypi.org/project/tzdata/ ? |
Member
|
Will do that in a sec once I figure out the .bat script part. |
Member
|
PR over at #45425. I didn't test the whole script since it needs to be tested as part of CI. |
Member
|
#45425 is now merged and I'll start working on RC1. Closing this PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR to verify Release Candidate