GH-40112: [CI][Packaging][Conda] Fix installing GCS testbench on python 3.9 #40120
Conversation
|
|
| @@ -17,7 +17,7 @@ | |||
|
|
|||
| # don't add pandas here, because it is not a mandatory test dependency | |||
| boto3 # not a direct dependency of s3fs, but needed for our s3fs fixture | |||
| cffi | |||
| cffi<1.16.0 # cffi 1.16.0 with Python 3.9 has an incompatibility with grpcio (required for GCS testbench) | |||
There was a problem hiding this comment.
Good catch!
Can we restrict this pin only for Python 3.9? It seems that we can do it with requirements.txt but I don't know whether we can do it with conda...
There was a problem hiding this comment.
I don't know if it's possible with conda. The only docs I could find about the version specification syntax was this https://docs.conda.io/projects/conda/en/4.6.0/_downloads/52a95608c49671267e40c689e0bc00ca/conda-cheatsheet.pdf
github-actions
bot
added
awaiting changes
|
We can fix that on the conda-forge side so that you don't have to restrict anything here. Do you have an upstream bug report I could reference to argue for retroactively adding the pin? |
|
Better fix was merged #40131 |
Rationale for this change
cffi1.16.0 conda release for python 3.9 was modified 12 days ago and it seems to have brokengrpcio==1.59.0.What changes are included in this PR?
Restrict
cffito < 1.16.0. Perhaps someone should investigate further and get the build working again with the 1.16.0 release but I think at least in the short term restricting the version like this is the best option.Are these changes tested?
No new functionality to test. Should fix existing CI failures.
Are there any user-facing changes?
No