rstudio / helm

Helm Resources for RStudio Products
MIT License
36 stars 28 forks source link

Update the default Package Manager config to take advantage of R and Python autodetection #473

Closed SamEdwardes closed 8 months ago

SamEdwardes commented 8 months ago

The current Package Manager docker image and helm chart have hard-coded R and Python configurations.

Related

The issue

Docker

https://github.com/rstudio/rstudio-docker-products/blob/e0fa3ab39e64e6e9b03febcbcb97e71a96210596/package-manager/rstudio-pm.gcfg#L12-L14

; Git sources require a configured R installation. R is often installed at `/usr/lib/R`
; or `/usr/lib64/R`.
RVersion = /opt/R/default/

Helm

# -- config is a nested map of maps that generates the rstudio-pm.gcfg file
config:
  Server:
    RVersion: "/opt/R/3.6.2/"
  HTTP:
    Listen: :4242
  Metrics:
    Enabled: true

Hard coding these values can lead to errors when starting Package Manager. In particular, this is an issue with hard coding the value into Helm because there is no guarantee that /opt/R/3.6.2/ is installed.

Proposed fix