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.

Sign up for GitHub

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

Jump to bottom

[Python][CI] Pin cython less than 3.0 on verification script #36756

Closed
raulcd opened this issue Jul 18, 2023 · 0 comments · Fixed by #36757
Closed

[Python][CI] Pin cython less than 3.0 on verification script #36756

raulcd opened this issue Jul 18, 2023 · 0 comments · Fixed by #36757
Assignees
@raulcd
Labels
Component: Continuous Integration Component: Python
Milestone
13.0.0

Comments

@raulcd
Copy link
Member

raulcd commented Jul 18, 2023

Describe the bug, including details regarding any error messages, version, and platform.

Currently some of our verification tasks are failing for 13.0.0 due to us installing cython from the verify script:

  maybe_setup_virtualenv cython numpy setuptools_scm setuptools || exit 1

We did pin Cython<3 on other places here:
#36744
And there is an issue opened to fix pyarrow to be compatible with Cython >= 3.0

Component(s)

Continuous Integration, Python
raulcd added a commit to raulcd/arrow that referenced this issue Jul 18, 2023
@raulcd
apacheGH-36756: [CI][Python] Install Cython < 3.0 on verify-release-c…
f9cccec 
…andidate script
@github-actions github-actions bot mentioned this issue Jul 18, 2023
Merged
jorisvandenbossche pushed a commit that referenced this issue Jul 19, 2023
@raulcd
GH-36756: [CI][Python] Install Cython < 3.0 on verify-release-candida…
3ce9e1d 
…te script (#36757)

### Rationale for this change

Some of our verification tasks fail for 13.0.0

### What changes are included in this PR?

Pin Cython to be less than 3.0

### Are these changes tested?

Archery

### Are there any user-facing changes?

No
* Closes: #36756

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
@jorisvandenbossche jorisvandenbossche added this to the 13.0.0 milestone Jul 19, 2023
raulcd added a commit that referenced this issue Jul 19, 2023
@raulcd
GH-36756: [CI][Python] Install Cython < 3.0 on verify-release-candida…
6aff232 
…te script (#36757)

### Rationale for this change

Some of our verification tasks fail for 13.0.0

### What changes are included in this PR?

Pin Cython to be less than 3.0

### Are these changes tested?

Archery

### Are there any user-facing changes?

No
* Closes: #36756

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
chelseajonesr pushed a commit to chelseajonesr/arrow that referenced this issue Jul 20, 2023
@raulcd @chelseajonesr
apacheGH-36756: [CI][Python] Install Cython < 3.0 on verify-release-c…
6b1fecb 
…andidate script (apache#36757)

### Rationale for this change

Some of our verification tasks fail for 13.0.0

### What changes are included in this PR?

Pin Cython to be less than 3.0

### Are these changes tested?

Archery

### Are there any user-facing changes?

No
* Closes: apache#36756

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
R-JunmingChen pushed a commit to R-JunmingChen/arrow that referenced this issue Aug 20, 2023
@raulcd @R-JunmingChen
apacheGH-36756: [CI][Python] Install Cython < 3.0 on verify-release-c…
78bf6e9 
…andidate script (apache#36757)

### Rationale for this change

Some of our verification tasks fail for 13.0.0

### What changes are included in this PR?

Pin Cython to be less than 3.0

### Are these changes tested?

Archery

### Are there any user-facing changes?

No
* Closes: apache#36756

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@raulcd raulcd

Labels
Component: Continuous Integration Component: Python
Projects
None yet
Milestone
13.0.0
Development

Successfully merging a pull request may close this issue.

GH-36756: [CI][Python] Install Cython < 3.0 on verify-release-candidate script raulcd/arrow
2 participants
@raulcd @jorisvandenbossche