Repository navigation
Use noble in Workbench default + finish PythonPackageRepository migration in examples - #957
Merged
Merged
Conversation
bschwedler
approved these changes
Oct 6, 2026
bschwedler
left a comment
Contributor
There was a problem hiding this comment.
Thanks for catching and fixing this Sam!
…tion in examples -kubernetes.yaml example to packagemanager.posit.co / noble / PythonPackageRepository, but a few user-facing defaults/examples were missed: - charts/rstudio-workbench/values.yaml: the default session repos.conf CRAN URL still used jammy (22.04) -> noble (24.04). Bumped chart 0.22.3 -> 0.22.4 + NEWS; regenerated README (README.md + README.md.gotmpl). - examples/connect/.../rstudio-connect-recommended-app-config.yaml (the non-Kubernetes / launcher off-host example): still used jammy and still set the Python index via a pip.conf ConfigMap + volume + mount. Switched to noble and the PythonPackageRepository "PyPI" config stanza (recommended since Connect 2026.05.0), dropping the pip.conf plumbing. Kept the netrc/curlrc volumes (authenticated repos). - examples/workbench/.../rstudio-workbench-recommended-app-config.yaml: jammy -> noble. Follow-up to #946 / #947. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
samcofer
force-pushed
the
workbench-noble-connect-pm-followup
branch
from
October 6, 2026 21:15
07ca2a6 to
ff26263
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #956. Follow-up to #946 / #947.
#947 moved the Connect chart default, both charts'
lint/values, and the Connect-kubernetes.yamlexample topackagemanager.posit.co/noble/PythonPackageRepository. A repo sweep found the legacy domain is now gone everywhere, but these user-facing defaults/examples were missed:Changes
Workbench chart default —
charts/rstudio-workbench/values.yamlrepos.confCRAN URLjammy→noble. Direct Workbench analog of the Connect-only chart-default fix in update connect-pm integration instructions #947.0.22.3→0.22.4+ NEWS entry; regeneratedREADME.md/README.md.gotmpl.Connect non-Kubernetes example —
examples/connect/application-configuration/rstudio-connect-recommended-app-config.yamlpackagemanager.posit.co,noble, andPythonPackageRepository#946 named two example files; update connect-pm integration instructions #947 only touched-kubernetes.yaml. This launcher/off-host variant still usedjammyand set the Python index via apip-config-connectConfigMap + volume +/etc/pip.confmount.jammy→noble; replaced the pip.conf plumbing with aPythonPackageRepository "PyPI"entry inconfig(recommended since Connect 2026.05.0), matching the pattern update connect-pm integration instructions #947 used for-kubernetes.yaml. Kept thenetrc/curlrcvolumes (authenticated repos — a separate concern).Workbench example —
examples/workbench/application-configuration/rstudio-workbench-recommended-app-config.yamlrepos.confCRAN URLjammy→noble. (Workbench Python index stays inconfig.session.pip.conf, which is already correct — noPythonPackageRepositoryequivalent exists for Workbench.)Deliberately not touched
charts/*/NEWS.mdhistorical entries referencingbionic/jammy.charts/rstudio-connect/tests/**-jammycontent-base image tags (test fixtures, not user-facing).examples/yaml/daemonset-r-session-complete-*+other-charts/prepull-daemonset(pinned to EOLbionic-era product versions).Validation
helm dependency build+helm lint .onrstudio-workbench→0 chart(s) failed.helm templaterendersCRAN=https://packagemanager.posit.co/cran/__linux__/noble/latest.ci/lintvalues asserted on the oldjammyCRAN default.🤖 Generated with Claude Code