Skip to content

GH-49757: [Release][CI] export SSL_CERT_FILE to bypass incompatibility with OpenSSL on RHEL-8#49769

Merged
kou merged 3 commits intoapache:mainfrom
raulcd:GH-49757
Apr 17, 2026
Merged

GH-49757: [Release][CI] export SSL_CERT_FILE to bypass incompatibility with OpenSSL on RHEL-8#49769
kou merged 3 commits intoapache:mainfrom
raulcd:GH-49757

Conversation

@raulcd
Copy link
Copy Markdown
Member

@raulcd raulcd commented Apr 16, 2026

Rationale for this change

The binary wheel verification is taking around 50 minutes locally, It is timing out on CI. After some investigation, see the issue, this is related to: https://bugzilla.redhat.com/show_bug.cgi?id=1053882

What changes are included in this PR?

Export SSL_CERT_FILE before running the verification on almalinux-verify-rc job.

Are these changes tested?

Yes, I tested this locally due to the binary verification not easily being tested on CI.

Locally executed the following which is exactly the same that the biniary verification CI job exercises:

$ time ALMALINUX=8 archery docker run -e TEST_DEFAULT=0 -e TEST_WHEELS=1 -e VERBOSE=1 -e VERIFY_RC=0 -e VERIFY_VERSION=24.0.0 -e GH_TOKEN=$GH_TOKEN almalinux-verify-rc

It took previously ~50 minutes to complete (got clearly stuck on test_fs.py), it currently takes 7 minutes:

+ TEST_SUCCESS=yes
+ echo 'Release candidate 24.0.0-RC0 looks good!'
Release candidate 24.0.0-RC0 looks good!
+ exit 0
+ cleanup
+ '[' yes = yes ']'
+ rm -fr /tmp/arrow-24.0.0.CTLOf

real	7m4.906s
user	0m3.660s
sys	0m1.626s

Are there any user-facing changes?

No

@github-actions
Copy link
Copy Markdown

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

@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Apr 16, 2026
@raulcd raulcd requested a review from pitrou April 16, 2026 09:22
Copy link
Copy Markdown
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

Comment thread compose.yaml Outdated
@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Apr 16, 2026
@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting merge Awaiting merge labels Apr 16, 2026
Comment thread compose.yaml Outdated
Copy link
Copy Markdown
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

@github-actions github-actions bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels Apr 17, 2026
@kou kou merged commit f092b91 into apache:main Apr 17, 2026
50 checks passed
@kou kou removed the awaiting change review Awaiting change review label Apr 17, 2026
@github-actions github-actions bot added the awaiting merge Awaiting merge label Apr 17, 2026
@conbench-apache-arrow
Copy link
Copy Markdown

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

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 2 possible false positives for unstable benchmarks that are known to sometimes produce them.

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.

3 participants