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

[CI][Python] CI failures on Python builds due to pytest_cython #40974

Closed
AlenkaF opened this issue Apr 3, 2024 · 1 comment
Closed

[CI][Python] CI failures on Python builds due to pytest_cython #40974

AlenkaF opened this issue Apr 3, 2024 · 1 comment
Assignees
@AlenkaF
Labels
Component: Continuous Integration Type: bug
Milestone
16.0.0

Comments

@AlenkaF
Copy link
Member

AlenkaF commented Apr 3, 2024

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

We are seeing sporadic CI failures on Python builds due to pytest_cython:

  File "/opt/conda/envs/arrow/lib/python3.8/site-packages/pytest_cython/plugin.py", line 77, in <module>
    def _without_suffixes(path: str | pathlib.Path) -> pathlib.Path:
TypeError: unsupported operand type(s) for |: 'type' and 'type'

https://github.com/apache/arrow/actions/runs/8526575827/job/23356153535?pr=40949#step:6:5476

Upstream issue has been merged lgpage/pytest-cython#67 but not yet released so we should add an upper pin for pytest-cython to have green CI.

Component(s)

Continuous Integration
@AlenkaF AlenkaF self-assigned this Apr 3, 2024
@github-actions github-actions bot mentioned this issue Apr 3, 2024
Merged
@llama90 llama90 mentioned this issue Apr 3, 2024
Closed
kou pushed a commit that referenced this issue Apr 4, 2024
@AlenkaF
GH-40974: [CI][Python] CI failures on Python builds due to pytest_cyt…
bbeeb33 
…hon (#40975)

### Rationale for this change

We are seeing sporadic CI failures on Python builds due to `pytest_cython`.

### What changes are included in this PR?

Upper pin of 0.3.0 added for `pytest-cython`.

### Are these changes tested?

With a green CI.

### Are there any user-facing changes?

No.
* GitHub Issue: #40974

Authored-by: AlenkaF <frim.alenka@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou
Copy link
Member

kou commented Apr 4, 2024

Issue resolved by pull request 40975
#40975

@kou kou added this to the 16.0.0 milestone Apr 4, 2024
@kou kou closed this as completed Apr 4, 2024
tolleybot pushed a commit to tmct/arrow that referenced this issue May 2, 2024
@AlenkaF @tolleybot
apacheGH-40974: [CI][Python] CI failures on Python builds due to pyte…
d720db5 
…st_cython (apache#40975)

### Rationale for this change

We are seeing sporadic CI failures on Python builds due to `pytest_cython`.

### What changes are included in this PR?

Upper pin of 0.3.0 added for `pytest-cython`.

### Are these changes tested?

With a green CI.

### Are there any user-facing changes?

No.
* GitHub Issue: apache#40974

Authored-by: AlenkaF <frim.alenka@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
vibhatha pushed a commit to vibhatha/arrow that referenced this issue May 25, 2024
@AlenkaF @vibhatha
apacheGH-40974: [CI][Python] CI failures on Python builds due to pyte…
19cfaea 
…st_cython (apache#40975)

### Rationale for this change

We are seeing sporadic CI failures on Python builds due to `pytest_cython`.

### What changes are included in this PR?

Upper pin of 0.3.0 added for `pytest-cython`.

### Are these changes tested?

With a green CI.

### Are there any user-facing changes?

No.
* GitHub Issue: apache#40974

Authored-by: AlenkaF <frim.alenka@gmail.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
Assignees

@AlenkaF AlenkaF

Labels
Component: Continuous Integration Type: bug
Projects
None yet
Milestone
16.0.0
Development

No branches or pull requests
2 participants
@kou @AlenkaF