Skip to content

Commit

Permalink
Merge pull request lsst-sqre#3910 from lsst-sqre/u/ajt/fix_pdb
Browse files Browse the repository at this point in the history
Turn off PDB for monitoring/telegraf
  • Loading branch information
athornton authored Nov 21, 2024
2 parents 4b99d26 + 2a5ba2e commit 94d6733
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 0 deletions.
1 change: 1 addition & 0 deletions applications/monitoring/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ Monitoring suite: InfluxDB2, Chronograf, telegraf
| influxdb2.ingress | object | disabled, must be enabled and configured at each site | InfluxDB2 ingress configuration. |
| influxdb2.livenessProbe.failureThreshold | int | `10` | Number of checks to conclude whether InfluxDB has died |
| influxdb2.livenessProbe.periodSeconds | int | `10` | Period between checks for whether InfluxDB is still alive |
| influxdb2.pdb | object | disabled; nonsensical for single replica | InfluxDB2 pod disruption budget. |
| influxdb2.resources | object | See `values.yaml` | Resource limits and requests for the InfluxDB server instance |
| influxdb2.startupProbe.enabled | bool | `true` | Whether to enable a startup probe |
| influxdb2.startupProbe.failureThreshold | int | `60` | Number of checks to conclude whether InfluxDB won't start. High to allow up to 10 minutes for startup, because checking many shards can be slow. |
Expand Down
5 changes: 5 additions & 0 deletions applications/monitoring/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,11 @@ influxdb2:
# -- Where we store secrets to run the server
existingSecret: monitoring

# -- InfluxDB2 pod disruption budget.
# @default -- disabled; nonsensical for single replica
pdb:
create: false

# -- InfluxDB2 ingress configuration.
# @default -- disabled, must be enabled and configured at each site
ingress:
Expand Down
1 change: 1 addition & 0 deletions applications/telegraf/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ Application telemetry collection service
| telegraf.env[0].valueFrom.secretKeyRef.name | string | `"telegraf"` | |
| telegraf.mountPoints[0].mountPath | string | `"/etc/telegraf-generated"` | |
| telegraf.mountPoints[0].name | string | `"telegraf-generated-config"` | |
| telegraf.pdb.create | bool | `false` | |
| telegraf.podLabels."hub.jupyter.org/network-access-hub" | string | `"true"` | |
| telegraf.rbac.clusterWide | bool | `true` | |
| telegraf.resources.limits.cpu | string | `"1"` | |
Expand Down
2 changes: 2 additions & 0 deletions applications/telegraf/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ telegraf:
requests:
memory: "350Mi"
cpu: "50m"
pdb:
create: false
args:
- "--config"
- "/etc/telegraf-generated/telegraf-generated.conf"
Expand Down

0 comments on commit 94d6733

Please sign in to comment.