rstudio / helm

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

Chronicle README Workbench example fixes #445

Closed tnederlof closed 6 months ago

tnederlof commented 8 months ago

The chronicle helm chart README for Workbench has

pod:
  sidecar:
    - name: chronicle-agent
      image: posit-chronicle:2023.11.3
      volumeMounts:
      - name: CHRONICLE_PRODUCT_CLUSTER_ID
        value: "posit-cluster-1"
      - name: logs
        mountPath: "/var/lib/rstudio-server/audit"
      env:
      - name: CHRONICLE_SERVER_ADDRESS
        value: "http://chronicle-server.default.svc.cluster.local"

This section is invalid as volumeMounts doesn't take a value. Perhaps it needs to be under env like in the Connect example.

      volumeMounts:
      - name: CHRONICLE_PRODUCT_CLUSTER_ID
        value: "posit-cluster-1"

Also, perhaps describe the purpose of CHRONICLE_PRODUCT_CLUSTER_ID if it's even needed?

Additionally, this example does not work without adding:

  volumes:
    - name: logs
      emptyDir: {}
  volumeMounts:
    - name: logs
      mountPath: "/var/lib/rstudio-server/audit"
AlexMapley commented 8 months ago

Thank you for calling this out @tnederlof, this is a bug in our documentation taken from https://github.com/rstudio/platform-helm/blob/main/apps/workbench/values.yaml.gotmpl#L38-L50 and needs to be fixed!

AlexMapley commented 8 months ago

https://github.com/rstudio/helm/pull/446/files