Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

Fix 382 and 389 #390

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Fix 382 and 389 #390

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3121628
Fix #382
TobiasJacob Jun 22, 2022
5ec2a21
Fix #389
TobiasJacob Jun 22, 2022
e564c6c
Merge branch 'timescale:master' into fix_382_and_389
TobiasJacob Aug 3, 2022
2f86370
Make priority class optional
TobiasJacob Aug 3, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions charts/timescaledb-single/templates/statefulset-timescaledb.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ spec:
{{- end }}
spec:
serviceAccountName: {{ template "timescaledb.serviceAccountName" . }}
priorityClassName: {{ .Values.priorityClassName }}
TobiasJacob marked this conversation as resolved.
Show resolved Hide resolved
securityContext:
# The postgres user inside the TimescaleDB image has uid=1000.
# This configuration ensures the permissions of the mounts are suitable
Expand Down
7 changes: 7 additions & 0 deletions charts/timescaledb-single/values.schema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -823,6 +823,13 @@
"integer",
"null"
]
},
"priorityClassName": {
"minimum": 1,
"type": [
"integer",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be string, and there should be no "minimum"

"null"
]
}
},
"required": [
Expand Down
5 changes: 5 additions & 0 deletions charts/timescaledb-single/values.schema.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -536,6 +536,11 @@ properties:
- "null"
minimum: 11
maximum: 14
priorityClassName:
type:
- string
- "null"
minLength: 1
nameOverride:
type:
- string
Expand Down
32 changes: 23 additions & 9 deletions charts/timescaledb-single/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,11 @@ image:
tag: pg14.3-ts2.7.0-p0
pullPolicy: Always

## @param priorityClassName Name of the existing priority class to be used by kafka pods
## Ref: https://kubernetes.io/docs/concepts/configuration/pod-priority-preemption/
##
priorityClassName: "timescaledb"
TobiasJacob marked this conversation as resolved.
Show resolved Hide resolved

# By default those secrets are randomly generated.
# To prevent misconfiguration, modifications from helm upgrade won't be applied to those secrets.
# As a result changing secrets cannot be done via helm and need manual intervention.
Expand Down Expand Up @@ -52,12 +57,13 @@ secrets:
certificateSecretName: ""

# This secret should contain environment variables that influence pgBackRest.
pgbackrest:
PGBACKREST_REPO1_S3_REGION: ""
PGBACKREST_REPO1_S3_KEY: ""
PGBACKREST_REPO1_S3_KEY_SECRET: ""
PGBACKREST_REPO1_S3_BUCKET: ""
PGBACKREST_REPO1_S3_ENDPOINT: "s3.amazonaws.com"
pgbackrest: {}
# Example configuration
# PGBACKREST_REPO1_S3_REGION: ""
# PGBACKREST_REPO1_S3_KEY: ""
# PGBACKREST_REPO1_S3_KEY_SECRET: ""
# PGBACKREST_REPO1_S3_BUCKET: ""
# PGBACKREST_REPO1_S3_ENDPOINT: "s3.amazonaws.com"

# Selector used to provision your own Secret containing pgbackrest configuration details
# This is mutually exclusive with `pgbackrest` option and takes precedence over it.
Expand All @@ -75,10 +81,18 @@ backup:
start-fast: "y"
repo1-retention-diff: 2
repo1-retention-full: 2
repo1-type: s3
repo1-cipher-type: "none"
repo1-s3-region: us-east-2
repo1-s3-endpoint: s3.amazonaws.com

# Example for s3
# repo1-type: s3
# repo1-s3-region: us-east-2
# repo1-s3-endpoint: s3.amazonaws.com

# Example for azure
# repo1-type: azure
# repo1-azure-account: azurestorageaccount
# repo1-azure-container: timescaledbcontainer
# repo1-azure-key: Ozz7nHipXVvL4M8rqvjlR1gBs/jShsWtaERPnOy3Ja6MRcIK8211Hkvg3N8by3uPBcxkT4K6nrGY+0Hjq58jMz==

# Overriding the archive-push/archive-get sections is most useful in
# very high througput situations. Look at values/high_throuhgput_example.yaml for more details
Expand Down