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.

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

GH-45377: [CI][R] Ensure install R on ubuntu-24.04 runner for R nightly build jobs #45464

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

eitsupi
Copy link
Contributor

@eitsupi eitsupi commented Feb 7, 2025

Rationale for this change

Related to #45377.
As a result of the recent change in the ubuntu-latest GHA runner from ubuntu-22.04 to ubuntu-24.04, it appears that the tools included in the runner from the start have changed and jobs that assume R is already installed will now fail.
Installing R the job should now succeed.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions bot added the awaiting review Awaiting review label Feb 7, 2025
@kou kou changed the title MINOR: [CI][R] Ensure install R on ubuntu-24.04 runner for R nightly build jobs GH-45477: [CI][R] Ensure install R on ubuntu-24.04 runner for R nightly build jobs Feb 7, 2025
Copy link

github-actions bot commented Feb 7, 2025

❌ GitHub issue #45477 could not be retrieved.

@kou kou changed the title GH-45477: [CI][R] Ensure install R on ubuntu-24.04 runner for R nightly build jobs GH-45377: [CI][R] Ensure install R on ubuntu-24.04 runner for R nightly build jobs Feb 7, 2025
@kou
Copy link
Member

kou commented Feb 7, 2025

@raulcd @assignUser Do you have any idea how to test this...?

Copy link

github-actions bot commented Feb 7, 2025

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

@@ -60,7 +60,7 @@ jobs:
repository: ursacomputing/crossbow
ref: main
- name: Set up Python
uses: actions/setup-python@42375524e23c412d93fb67b49958b491fce71c38 # v5.4.0
uses: actions/setup-python@42375524e23c412d93fb67b49958b491fce71c38 # v5.4.0
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't recall editing this, but perhaps the line ending characters have been changed (there was a warning about mixing LF and CRLF when opened on GitHub. I did the edit in VSCode on GitHub, which probably automatically unified the end-of-line characters).

@github-actions github-actions bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting committer review Awaiting committer review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants