Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

Commit

Permalink
[stable/node-problem-detector] adds custom-plugin-monitor config (#10549
Browse files Browse the repository at this point in the history
)

* [stable/node-problem-detector] adds --custom-plugin-monitors flag

Signed-off-by: Joy Bhattacherjee <[email protected]>

* [stable/node-problem-detector] adds documentation custom_plugin_monitor

Signed-off-by: Joy Bhattacherjee <[email protected]>

* [stable/node-problem-detector] updates readme with correct default value for custompluginmonitor

Signed-off-by: Joy Bhattacherjee <[email protected]>

* [stable/node-problem-detector] updates chart version to 1.1.3

Signed-off-by: Joy Bhattacherjee <[email protected]>
  • Loading branch information
Hashfyre authored and k8s-ci-robot committed Jan 15, 2019
1 parent e0120f2 commit b3ef3a0
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 17 deletions.
2 changes: 1 addition & 1 deletion stable/node-problem-detector/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
name: node-problem-detector
version: "1.1.2"
version: "1.1.3"
appVersion: v0.6.1
home: https://github.com/kubernetes/node-problem-detector
description: Installs the node-problem-detector daemonset for monitoring extra attributes on nodes
Expand Down
31 changes: 16 additions & 15 deletions stable/node-problem-detector/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,21 +34,22 @@ Custom System log monitor config files can be created, see [here](https://github

The following table lists the configurable parameters for this chart and their default values.

| Parameter | Description | Default |
| ------------------------|--------------------------------------------|--------------------------------------------------------------|
| `affinity` | Map of node/pod affinities | `{}` |
| `annotations` | Optional daemonset annotations | `{}` |
| `fullnameOverride` | Override the fullname of the chart | `nil` |
| `image.pullPolicy` | Image pull policy | `IfNotPresent` |
| `image.repository` | Image | `k8s.gcr.io/node-problem-detector` |
| `image.tag` | Image tag | `v0.6.1` |
| `nameOverride` | Override the name of the chart | `nil` |
| `rbac.create` | RBAC | `true` |
| `resources` | Pod resource requests and limits | `{}` |
| `settings.log_monitors` | System log monitor config files | `/config/kernel-monitor.json`, `/config/docker-monitor.json` |
| `serviceAccount.create` | Whether a ServiceAccount should be created | `true` |
| `serviceAccount.name` | Name of the ServiceAccount to create | Generated value from template |
| `tolerations` | Optional daemonset tolerations | `[]` |
| Parameter | Description | Default |
| ----------------------------------|--------------------------------------------|--------------------------------------------------------------|
| `affinity` | Map of node/pod affinities | `{}` |
| `annotations` | Optional daemonset annotations | `{}` |
| `fullnameOverride` | Override the fullname of the chart | `nil` |
| `image.pullPolicy` | Image pull policy | `IfNotPresent` |
| `image.repository` | Image | `k8s.gcr.io/node-problem-detector` |
| `image.tag` | Image tag | `v0.6.1` |
| `nameOverride` | Override the name of the chart | `nil` |
| `rbac.create` | RBAC | `true` |
| `resources` | Pod resource requests and limits | `{}` |
| `settings.log_monitors` | System log monitor config files | `/config/kernel-monitor.json`, `/config/docker-monitor.json` |
| `settings.custom_plugin_monitors` | Custom plugin monitor config files | `[]` |
| `serviceAccount.create` | Whether a ServiceAccount should be created | `true` |
| `serviceAccount.name` | Name of the ServiceAccount to create | Generated value from template |
| `tolerations` | Optional daemonset tolerations | `[]` |

Specify each parameter using the `--set key=value[,key=value]` argument to `helm install` or provide a YAML file containing the values for the above parameters:

Expand Down
2 changes: 1 addition & 1 deletion stable/node-problem-detector/templates/daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ spec:
command:
- "/bin/sh"
- "-c"
- "exec /node-problem-detector --logtostderr --system-log-monitors={{- range $index, $monitor := .Values.settings.log_monitors }}{{if ne $index 0}},{{end}}/config/{{ $monitor }}{{- end }}"
- "exec /node-problem-detector --logtostderr --system-log-monitors={{- range $index, $monitor := .Values.settings.log_monitors }}{{if ne $index 0}},{{end}}/config/{{ $monitor }}{{- end }} {{- if .Values.settings.custom_plugin_monitors }} --custom-plugin-monitors={{- range $index, $monitor := .Values.settings.custom_plugin_monitors }}{{if ne $index 0}},{{end}}{{ $monitor }}{{- end }} {{- end }}"
securityContext:
privileged: true
env:
Expand Down
1 change: 1 addition & 0 deletions stable/node-problem-detector/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ settings:
log_monitors:
- kernel-monitor.json
- docker-monitor.json
custom_plugin_monitors: []

hostpath:
logdir: /var/log/
Expand Down

0 comments on commit b3ef3a0

Please sign in to comment.