Skip to content

Fix RStudio installation on Debian 13 (trixie)#1397

Open
DiwakarRana wants to merge 1 commit into
GoogleCloudDataproc:mainfrom
DiwakarRana:main
Open

Fix RStudio installation on Debian 13 (trixie)#1397
DiwakarRana wants to merge 1 commit into
GoogleCloudDataproc:mainfrom
DiwakarRana:main

Conversation

@DiwakarRana

@DiwakarRana DiwakarRana commented Jul 1, 2026

Copy link
Copy Markdown

This PR updates the script to be compatible with Debian 13 (Trixie) and Ubuntu 24.04 (Noble) while maintaining full backward compatibility with older Dataproc versions:

  1. Robust GPG Key Import:
    -----Added a fallback to gpg if apt-key is missing (deprecated in newer OS versions).
    -----Explicitly installs dirmngr alongside gnupg, which is required for GPG keyserver operations on minimal images.
    -----Uses the update_apt_get helper with retries to mitigate transient network failures during key import.
  2. Bypassed add-apt-repository & Dynamic CRAN Suffix:
    -----Writes the CRAN repository definition directly to /etc/apt/sources.list.d/cran.list (faster and works out-of-the-box).
    -----Dynamically selects the CRAN repository suffix: uses cran46 for Debian 12/13 (which transitioned to R 4.6+) and falls back to cran40 for Ubuntu 22/24.
  3. Upgraded RStudio Server Version (libssl3 Compatibility):
    -----For Debian 12 (bookworm), Debian 13 (trixie), Ubuntu 22.04 (jammy), and Ubuntu 24.04 (noble), the script now installs RStudio Server 2023.09.1-494 (which supports libssl3) instead of 1.2.5019 (which required the deprecated libssl1.1).
    -----Ensures the systemd service file path is correctly mapped for these newer OS versions when running in passwordless mode.

Detailed Changes:
Modified rstudio/rstudio.sh:

  1. Updated GPG key import logic to support gpg fallback, install dirmngr, and use update_apt_get.
  2. Implemented dynamic CRAN suffix selection (cran46 vs cran40) and direct writing to cran.list.
  3. Updated RSTUDIO_SERVER_VERSION and service_file path logic to include bookworm, trixie, jammy, and noble.
  4. Updated the supported_os list to include Debian 13 and Ubuntu 24.04.

@google-cla

google-cla Bot commented Jul 1, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

The pull request updates rstudio/rstudio.sh to support Debian trixie and handles the deprecation of apt-key by falling back to gpg for key management. It also simplifies the CRAN repository configuration by writing directly to a sources list file. The review feedback suggests improving the robustness of the package installation by using the update_apt_get helper, installing dirmngr alongside gnupg, and quoting variables. Additionally, it recommends dynamically setting the CRAN repository suffix to -cran46/ for Debian bookworm and trixie to ensure compatibility with newer R versions.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread rstudio/rstudio.sh Outdated
Comment thread rstudio/rstudio.sh Outdated
@DiwakarRana DiwakarRana force-pushed the main branch 2 times, most recently from ae57d42 to a518a99 Compare July 1, 2026 16:29
Comment thread rstudio/rstudio.sh
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants