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

[R][CI] Use rocker/r-ver instead of library/r-base #40323

Closed
jonkeane opened this issue Mar 2, 2024 · 1 comment
Closed

[R][CI] Use rocker/r-ver instead of library/r-base #40323

jonkeane opened this issue Mar 2, 2024 · 1 comment
Assignees
@jonkeane
Labels
Component: Continuous Integration Component: R Type: enhancement
Milestone
16.0.0

Comments

@jonkeane
Copy link
Member

jonkeane commented Mar 2, 2024

Describe the enhancement requested

Switch our runs of library/r-base images to rocker/r-ver. It turns out that library/r-base (and rocker/r-base) are intended to be more bleeding edge than we realized and are sometimes built against debian unstable package channels. This can cause transient package version collisions which aren't relevant to us (and cause false positive failures we need to investigate).

From https://github.com/rocker-org/rocker?tab=readme-ov-file#base-docker-containers:

image description
r-base Current R via apt-get with debian:testing & unstable repos
r-devel R-devel added side-by-side onto r-base (using alias RD)
drd lighter r-devel, built not quite daily
r-ver Specify R version in docker tag. Builds on debian:stable

See: rocker-org/rocker#541

Component(s)

Continuous Integration, R
@github-actions github-actions bot mentioned this issue Mar 2, 2024
Merged
jonkeane added a commit that referenced this issue Mar 2, 2024
@jonkeane
GH-40323: [R] [CI] Use rocker/r-ver instead of library/r-base (#40321)
6406676 
### Rationale for this change

I'll make an issue if this works

### What changes are included in this PR?

Replace `library/r-base` with `rocker/r-ver` to see if that is more stable for our generic minimal / offline / bundled builds 

### Are these changes tested?

They are the tests

### Are there any user-facing changes?

No
* GitHub Issue: #40323

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Jonathan Keane <jkeane@gmail.com>
@jonkeane
Copy link
Member Author

jonkeane commented Mar 2, 2024

Issue resolved by pull request 40321
#40321

@jonkeane jonkeane added this to the 16.0.0 milestone Mar 2, 2024
@jonkeane jonkeane closed this as completed Mar 2, 2024
@kou kou changed the title [R] [CI] Use rocker/r-ver instead of library/r-base [R][CI] Use rocker/r-ver instead of library/r-base Mar 3, 2024
thisisnic pushed a commit to thisisnic/arrow that referenced this issue Mar 8, 2024
@jonkeane @thisisnic
apacheGH-40323: [R] [CI] Use rocker/r-ver instead of library/r-base (a…
3f1b9b9 
…pache#40321)

### Rationale for this change

I'll make an issue if this works

### What changes are included in this PR?

Replace `library/r-base` with `rocker/r-ver` to see if that is more stable for our generic minimal / offline / bundled builds 

### Are these changes tested?

They are the tests

### Are there any user-facing changes?

No
* GitHub Issue: apache#40323

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Jonathan Keane <jkeane@gmail.com>
@assignUser assignUser mentioned this issue Mar 9, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jonkeane jonkeane

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

No branches or pull requests
1 participant
@jonkeane