Skip to content

Conversation

@raulcd
Copy link
Member

@raulcd raulcd commented Dec 5, 2023

Rationale for this change

The verification task for Almalinux 8 was failing.

What changes are included in this PR?

Add required python3.11-devel to the Docker image.

Are these changes tested?

Yes via archery task.

Are there any user-facing changes?

No

@github-actions
Copy link

github-actions bot commented Dec 5, 2023

⚠️ GitHub issue #39072 has been automatically assigned in GitHub to PR creator.

@raulcd

This comment was marked as outdated.

@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Dec 5, 2023
@raulcd
Copy link
Member Author

raulcd commented Dec 5, 2023

@github-actions crossbow submit wheel-manylinux-2-28-cp38-amd64

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@raulcd
Copy link
Member Author

raulcd commented Dec 5, 2023

@github-actions crossbow submit verify-rc-source-python-linux-almalinux-8-amd64

@github-actions

This comment was marked as outdated.

@raulcd
Copy link
Member Author

raulcd commented Dec 5, 2023

@github-actions crossbow submit verify-rc-source-python-linux-almalinux-8-amd64

@github-actions
Copy link

github-actions bot commented Dec 5, 2023

Revision: f5f3c66

Submitted crossbow builds: ursacomputing/crossbow @ actions-2e6b7e4517

Task Status
verify-rc-source-python-linux-almalinux-8-amd64 Github Actions

@raulcd
Copy link
Member Author

raulcd commented Dec 5, 2023

@github-actions crossbow submit wheel-manylinux-2-28-cp38-amd64

@github-actions
Copy link

github-actions bot commented Dec 5, 2023

Revision: f5f3c66

Submitted crossbow builds: ursacomputing/crossbow @ actions-b5ae508eed

Task Status
wheel-manylinux-2-28-cp38-amd64 Github Actions

@raulcd raulcd changed the title GH-39072: [Release][CI] Fix Python not found on Almalinux 8 GH-39072: [Release][CI] Python3.11-devel is required for the verification job on Almalinux 8 Dec 5, 2023
@raulcd raulcd marked this pull request as ready for review December 5, 2023 11:39
@raulcd raulcd requested review from assignUser and kou as code owners December 5, 2023 11:39
ninja-build \
nodejs \
openssl-devel \
python3.11-devel \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we be less specific here? Does python3-devel exist?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it exists based on my packages search but I am trying it. If the job fails I'll revert the change

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

python3-devel seems to be 3.6.8 and it fails as seen here.

@raulcd
Copy link
Member Author

raulcd commented Dec 5, 2023

@github-actions crossbow submit verify-rc-source-python-linux-almalinux-8-amd64

@github-actions
Copy link

github-actions bot commented Dec 5, 2023

Revision: c18f8b9

Submitted crossbow builds: ursacomputing/crossbow @ actions-e5690d2257

Task Status
verify-rc-source-python-linux-almalinux-8-amd64 Github Actions

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Dec 5, 2023
This reverts commit c18f8b9.
@raulcd
Copy link
Member Author

raulcd commented Dec 5, 2023

@github-actions crossbow submit verify-rc-source-python-linux-almalinux-8-amd64

@github-actions github-actions bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels Dec 5, 2023
@github-actions
Copy link

github-actions bot commented Dec 5, 2023

Revision: 4433f1f

Submitted crossbow builds: ursacomputing/crossbow @ actions-b2b9b35bcf

Task Status
verify-rc-source-python-linux-almalinux-8-amd64 Github Actions

@kou kou changed the title GH-39072: [Release][CI] Python3.11-devel is required for the verification job on Almalinux 8 GH-39072: [Release][CI] Python3.11-devel is required for the verification job on AlmaLinux 8 Dec 5, 2023
Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kou kou merged commit b326917 into apache:main Dec 5, 2023
@kou kou removed the awaiting change review Awaiting change review label Dec 5, 2023
@github-actions github-actions bot added the awaiting merge Awaiting merge label Dec 5, 2023
raulcd added a commit that referenced this pull request Dec 6, 2023
…tion job on AlmaLinux 8 (#39073)

### Rationale for this change

The verification task for Almalinux 8 was failing.

### What changes are included in this PR?

Add required python3.11-devel to the Docker image.

### Are these changes tested?

Yes via archery task.

### Are there any user-facing changes?

No

* Closes: #39072

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit b326917.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

dgreiss pushed a commit to dgreiss/arrow that referenced this pull request Feb 19, 2024
…rification job on AlmaLinux 8 (apache#39073)

### Rationale for this change

The verification task for Almalinux 8 was failing.

### What changes are included in this PR?

Add required python3.11-devel to the Docker image.

### Are these changes tested?

Yes via archery task.

### Are there any user-facing changes?

No

* Closes: apache#39072

Authored-by: Raúl Cumplido <raulcumplido@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

Labels

awaiting merge Awaiting merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Release][CI] Verification job on AlmaLinux 8 fails to find correct Python

3 participants