[SPARK-56039][INFRA] Install remotes R package for dev/infra/Dockerfile#54872
[SPARK-56039][INFRA] Install remotes R package for dev/infra/Dockerfile#54872dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
remotes R package for dev/infra/Dockerfile#54872Conversation
|
So I've got another PR over in https://github.com/apache/spark/pull/54838/changes which is a follow on to the fix for the other R docker container challenge, main difference is also swittching to using pack since devtools is deprecated (it does keep devtools for one package though for the template, we might want to remove that template long term). WDYT @dongjoon-hyun |
|
@holdenk . At first glance, your PR is about different file, isn't it? This PR aims to recover the failure of 
|
|
Yes more do we want to switch to modern non devtools install util here as well (pak). |
|
FYI, last night, there was an outage in the master branch and it's fixed already. 
|
Let me review and comment on your PR, @holdenk ~ |
|
Could you review this PR to recover 
|
|
Thank you, @peter-toth ! |
|
Since the remaining running CI test pipelines on this PR are irrelevant to this PR, I'm going to merge this to re-trigger branch-4.0. |
|
Merged to master/4.1/4.0. |
…rfile` ### What changes were proposed in this pull request? This PR aims to recover the `Cache base image` GitHub Action CI by installing `remotes` R package for `dev/infra/Dockerfile`. Currently, it broke `branch-4.0`. ### Why are the changes needed? Currently, `Build / Cache base image / main (push)` GitHub Action job has been broken due to the new `devtools` 2.5.0 release. This PR aims to installation `remotes` package explicitly because we cannot pin `devtools` itself. - https://github.com/r-lib/devtools/releases/tag/v2.5.0 (2026-03-16) - `branch-4.0`: https://github.com/apache/spark/actions/runs/23209985590/job/67455852567 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Recover the CIs. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Code (claude-opus-4-6) Closes #54872 from dongjoon-hyun/SPARK-56039. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> (cherry picked from commit 6c5d707) Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
…rfile` ### What changes were proposed in this pull request? This PR aims to recover the `Cache base image` GitHub Action CI by installing `remotes` R package for `dev/infra/Dockerfile`. Currently, it broke `branch-4.0`. ### Why are the changes needed? Currently, `Build / Cache base image / main (push)` GitHub Action job has been broken due to the new `devtools` 2.5.0 release. This PR aims to installation `remotes` package explicitly because we cannot pin `devtools` itself. - https://github.com/r-lib/devtools/releases/tag/v2.5.0 (2026-03-16) - `branch-4.0`: https://github.com/apache/spark/actions/runs/23209985590/job/67455852567 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Recover the CIs. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Code (claude-opus-4-6) Closes #54872 from dongjoon-hyun/SPARK-56039. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> (cherry picked from commit 6c5d707) Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> (cherry picked from commit 10dd228) Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
3 participants
What changes were proposed in this pull request?
This PR aims to recover the
Cache base imageGitHub Action CI by installingremotesR package fordev/infra/Dockerfile. Currently, it brokebranch-4.0.Why are the changes needed?
Currently,
Build / Cache base image / main (push)GitHub Action job has been broken due to the newdevtools2.5.0 release. This PR aims to installationremotespackage explicitly because we cannot pindevtoolsitself.branch-4.0: https://github.com/apache/spark/actions/runs/23209985590/job/67455852567Does this PR introduce any user-facing change?
No.
How was this patch tested?
Recover the CIs.
Was this patch authored or co-authored using generative AI tooling?
Generated-by: Claude Code (claude-opus-4-6)