Merge pull request #496 from rstudio/tylfin/ppm-unset-server-addr

Fix: Unset the default [Server].Address option for PPM

charts/rstudio-pm/Chart.yaml

@@ -1,6 +1,6 @@
 name: rstudio-pm
 description: Official Helm chart for RStudio Package Manager
-version: 0.5.23
+version: 0.5.24
 apiVersion: v2
 appVersion: 2024.04.0
 icon: https://rstudio.com/wp-content/uploads/2018/10/RStudio-Logo-Flat.png

charts/rstudio-pm/NEWS.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.5.24
+
+- Unset the `[Server].Address` default value as this can cause issues when serving from a domain name.
+
 ## 0.5.23
 
 - Update default Posit Package Manager version to 2024.04.0-18

charts/rstudio-pm/README.md

@@ -1,6 +1,6 @@
 # Posit Package Manager
 
-![Version: 0.5.23](https://img.shields.io/badge/Version-0.5.23-informational?style=flat-square) ![AppVersion: 2024.04.0](https://img.shields.io/badge/AppVersion-2024.04.0-informational?style=flat-square)
+![Version: 0.5.24](https://img.shields.io/badge/Version-0.5.24-informational?style=flat-square) ![AppVersion: 2024.04.0](https://img.shields.io/badge/AppVersion-2024.04.0-informational?style=flat-square)
 
 #### _Official Helm chart for RStudio Package Manager_
 
@@ -21,11 +21,11 @@ To ensure a stable production deployment, please:
 
 ## Installing the Chart
 
-To install the chart with the release name `my-release` at version 0.5.23:
+To install the chart with the release name `my-release` at version 0.5.24:
 
 ```bash
 helm repo add rstudio https://helm.rstudio.com
-helm upgrade --install my-release rstudio/rstudio-pm --version=0.5.23
+helm upgrade --install my-release rstudio/rstudio-pm --version=0.5.24
 ```
 
 To explore other chart versions, take a look at:
@@ -154,7 +154,7 @@ The Helm `config` values are converted into the `rstudio-pm.gcfg` service config
 | awsAccessKeyId | bool | `false` | awsAccessKeyId is the access key id for s3 access, used also to gate file creation |
 | awsSecretAccessKey | string | `nil` | awsSecretAccessKey is the secret access key, needs to be filled if access_key_id is |
 | command | bool | `false` | command is the pod's run command. By default, it uses the container's default |
-| config | object | `{"HTTP":{"Listen":":4242"},"Metrics":{"Enabled":true},"Server":{"Address":"http://localhost:4242"}}` | config is a nested map of maps that generates the rstudio-pm.gcfg file |
+| config | object | `{"HTTP":{"Listen":":4242"},"Metrics":{"Enabled":true}}` | config is a nested map of maps that generates the rstudio-pm.gcfg file |
 | enableMigration | bool | `true` | Enable migrations for shared storage (if necessary) using Helm hooks. |
 | enableSandboxing | bool | `true` | Enable sandboxing of Git builds, which requires elevated security privileges for the Package Manager container. |
 | extraContainers | list | `[]` | sidecar container list |

charts/rstudio-pm/values.yaml

@@ -231,8 +231,6 @@ serviceMonitor:
 
 # -- config is a nested map of maps that generates the rstudio-pm.gcfg file
 config:
-  Server:
-    Address: http://localhost:4242
   HTTP:
     Listen: :4242
   Metrics: