rstudio / rstudio/helm

WB: wrong default of `XDG_CONFIG_DIRS`?

Open
#607 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

team: workbench
Dominant language
Markdown
Stars
46
Forks
40
Avg merge
4h 3m
Merged PRs (30d)
6

Description

XDG_CONFIG_DIRS is set to /mnt/dynamic:/mnt/session-configmap:/mnt/secret-configmap:/mnt/configmap:/mnt/load-balancer/.

However, as stated in https://docs.posit.co/ide/server-pro/rstudio_pro_sessions/customizing_session_settings.html, files like rstudio-prefs.json are only read when XDG_CONFIG_DIRS points to the directory containing them. This would actually be /mnt/session-configmap/rstudio.

The consequences are that files like config.session.rstudio-prefs.json are never read and result in an non-fatal error during session startup.

Setting

RSTUDIO_CONFIG_DIR: /mnt/session-configmap/rstudio

as a pod env var solves this.

(I've just discovered this recently and have to yet verify this in other instances. The fact falling over this now in a fresh install is a bit strange)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No file or test is named. Start by locating the chart defaults and templates that set XDG_CONFIG_DIRS, then compare the rendered value with the documented /mnt/session-configmap/rstudio path. Done means the relevant session configuration files are read without the reported non-fatal startup error.

Written by the indexing model from the issue text.

Assessment

Tech stack
helm
Domain
devops, infrastructure
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.