Skip to content

Commit

Permalink
Merge pull request #603 from rstudio/posit-chronicle-make-default-val…
Browse files Browse the repository at this point in the history
…ues-work

Fix posit-chronicle to function with default values
  • Loading branch information
jforest authored Nov 4, 2024
2 parents b9fd471 + 2e45cbd commit 2874c7f
Show file tree
Hide file tree
Showing 6 changed files with 21 additions and 8 deletions.
10 changes: 10 additions & 0 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,17 @@

# package-manager resources
/charts/rstudio-pm/* @rstudio/ppm
/ci/rstudio-pm/* @rstudio/ppm

# connect resources
/charts/rstudio-connect/* @aronatkins @dbkegley @christierney @zackverham
/ci/rstudio-connect/* @aronatkins @dbkegley @christierney @zackverham
/examples/connect/* @aronatkins @dbkegley @christierney @zackverham

# posit-chronicle resources
/charts/posit-chronicle/* @matt-urbina @markrtucker @t-margheim
/ci/posit-chronicle/* @matt-urbina @markrtucker @t-margheim

# rstudio-workbench resources
/charts/rstudio-workbench/* @GCrev @zachhannum
/ci/rstudio-workbench/* @GCrev @zachhannum
2 changes: 1 addition & 1 deletion charts/posit-chronicle/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v2
name: posit-chronicle
description: Official Helm chart for Posit Chronicle Server
version: 0.3.4
version: 0.3.5
appVersion: 2024.09.0
icon: https://rstudio.com/wp-content/uploads/2018/10/RStudio-Logo-Flat.png
home: https://www.posit.co
Expand Down
4 changes: 4 additions & 0 deletions charts/posit-chronicle/NEWS.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Changelog

## 0.3.5

- Change the default value for LocalStorage.Enabled to `true` in order for installations with the default values to work out of the box.

## 0.3.4

- Add helm unit test scaffold.
Expand Down
8 changes: 4 additions & 4 deletions charts/posit-chronicle/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Posit Chronicle

![Version: 0.3.4](https://img.shields.io/badge/Version-0.3.4-informational?style=flat-square) ![AppVersion: 2024.09.0](https://img.shields.io/badge/AppVersion-2024.09.0-informational?style=flat-square)
![Version: 0.3.5](https://img.shields.io/badge/Version-0.3.5-informational?style=flat-square) ![AppVersion: 2024.09.0](https://img.shields.io/badge/AppVersion-2024.09.0-informational?style=flat-square)

#### _Official Helm chart for Posit Chronicle Server_

Expand All @@ -25,11 +25,11 @@ To ensure a stable production deployment:

## Installing the chart

To install the chart with the release name `my-release` at version 0.3.4:
To install the chart with the release name `my-release` at version 0.3.5:

```{.bash}
helm repo add rstudio https://helm.rstudio.com
helm upgrade --install my-release rstudio/posit-chronicle --version=0.3.4
helm upgrade --install my-release rstudio/posit-chronicle --version=0.3.5
```

To explore other chart versions, look at:
Expand Down Expand Up @@ -164,7 +164,7 @@ The credentials Chronicle uses for S3 storage must have the following permission
| config.HTTPS.Certificate | string | `""` | |
| config.HTTPS.Enabled | bool | `false` | |
| config.HTTPS.Key | string | `""` | |
| config.LocalStorage.Enabled | bool | `false` | |
| config.LocalStorage.Enabled | bool | `true` | |
| config.LocalStorage.Location | string | `"./chronicle-data"` | |
| config.LocalStorage.RetentionPeriod | string | `"30d"` | |
| config.Logging.ServiceLog | string | `"STDOUT"` | |
Expand Down
5 changes: 2 additions & 3 deletions charts/posit-chronicle/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -70,9 +70,8 @@ config:
ServiceLogLevel: "INFO"
ServiceLogFormat: "TEXT"
LocalStorage:
# By default when localStorage.enabled=false, the chronicle stateful-set
# won't submit a volume claim or be bound to a persistent volume
Enabled: false
# By default LocalStorage.Enabled=true, so that installs work with the default values
Enabled: true
Location: "./chronicle-data"
RetentionPeriod: "30d"
S3Storage:
Expand Down
Empty file.

0 comments on commit 2874c7f

Please sign in to comment.