Skip to content
Snippets Groups Projects

[MONIT-4003] metrics: add default node selector option

Closed Guillermo Facundo Colunga requested to merge monit_4003 into qa
2 unresolved threads

This chart includes some components that are composed of a single pod, like: prometheus and fluentbit to collect and forward metrics or kube state to expose internal kubernetes api metrics.

This commit adds a new configuration field (merics.defaultNodeSelector) that allows to set a common nodeSelector for all those components. In case that any of those components adds a nodeSelector that one will have more priority.

Merge request reports

Merge request pipeline #8244665 passed

Merge request pipeline passed for d4272c0c

Requires 1 approval from eligible users.

Closed by Guillermo Facundo ColungaGuillermo Facundo Colunga 6 months ago (Oct 4, 2024 1:25pm UTC)

Merge details

  • The changes were not merged into qa.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
38 38 {{- if .Values.metrics.fluentbit.extraVolumeMounts }}
39 39 {{- toYaml .Values.metrics.fluentbit.extraVolumeMounts | nindent 6 }}
40 40 {{- end }}
41 {{- with .Values.metrics.fluentbit.nodeSelector }}
41 {{- with .Values.metrics.defaultNodeSelector }}
  • 38 38 {{- if .Values.metrics.fluentbit.extraVolumeMounts }}
    39 39 {{- toYaml .Values.metrics.fluentbit.extraVolumeMounts | nindent 6 }}
    40 40 {{- end }}
    41 {{- with .Values.metrics.fluentbit.nodeSelector }}
    41 {{- with .Values.metrics.defaultNodeSelector }}
    42 42 nodeSelector:
    43 {{- if .Values.metrics.fluentbit.nodeSelector }}
  • Changes already merged by !24 (merged).

  • Please register or sign in to reply
    Loading