Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactoring Kubernetes recommanded labels #78

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

Refactoring Kubernetes recommanded labels #78

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
626eccb
Cleanup Kubernetes recommanded labels
nlamirault May 2, 2024
d2f7137
fix: typo
nlamirault May 2, 2024
2f0e6eb
bump chart version
nlamirault May 2, 2024
a747e28
additional labels for Prometheus Rule
nlamirault May 3, 2024
b6ecd76
cleanup
nlamirault May 3, 2024
52e993f
fix(helm): duplicate labels
nlamirault Sep 4, 2024
3d0f9b8
fix(helm): typo
nlamirault Sep 4, 2024
06bcd4f
feat(helm): refactoring selector
nlamirault Sep 4, 2024
75d8174
fix(helm): cleanup
nlamirault Sep 4, 2024
20c674e
feat(helm): default selector
nlamirault Sep 4, 2024
2067857
Merge branch 'main' into feat/clean-kubernetes-labels
nlamirault Sep 4, 2024
d7da1b7
fix(helm): remove
nlamirault Sep 4, 2024
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
2 changes: 1 addition & 1 deletion charts/quickwit/Chart.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: quickwit
description: Sub-second search & analytics engine on cloud storage.
type: application
version: 0.7.0
version: 0.8.0
appVersion: "v0.8.2"
keywords:
- quickwit
Expand Down
1 change: 0 additions & 1 deletion charts/quickwit/templates/_helpers.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ Common labels
*/}}
{{- define "quickwit.labels" -}}
helm.sh/chart: {{ include "quickwit.chart" . }}
{{ include "quickwit.selectorLabels" . }}
{{- if .Chart.AppVersion }}
app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
{{- end }}
Expand Down
2 changes: 2 additions & 0 deletions charts/quickwit/templates/configmap-bootstrap.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ metadata:
name: {{ include "quickwit.fullname" . }}-bootstrap
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.selectorLabels" . | nindent 4 }}
app.kubernetes.io/component: config
data:
{{- range .Values.seed.indexes }}
{{ .index_id }}.yaml: |-
Expand Down
2 changes: 2 additions & 0 deletions charts/quickwit/templates/configmap.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@ metadata:
name: {{ include "quickwit.fullname" . }}
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.selectorLabels" . | nindent 4 }}
app.kubernetes.io/component: config
data:
node.yaml: |-
{{- toYaml .Values.config | nindent 4 }}
5 changes: 3 additions & 2 deletions charts/quickwit/templates/control-plane-deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ kind: Deployment
metadata:
name: {{ include "quickwit.fullname" . }}-control-plane
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.control_plane.selectorLabels" . | nindent 4 }}
annotations:
{{- with .Values.annotations }}
{{- toYaml . | nindent 4 }}
Expand All @@ -29,7 +30,7 @@ spec:
{{- toYaml . | nindent 8 }}
{{- end }}
labels:
{{- include "quickwit.additionalLabels" . | nindent 8 }}
{{- include "quickwit.labels" . | nindent 8 }}
{{- include "quickwit.control_plane.selectorLabels" . | nindent 8 }}
spec:
{{- with .Values.imagePullSecrets }}
Expand Down
2 changes: 2 additions & 0 deletions charts/quickwit/templates/prometheusrule.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ metadata:
name: {{ include "quickwit.fullname" . }}
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.selectorLabels" . | nindent 4 }}
app.kubernetes.io/component: metrics
{{- with .Values.prometheusRule.additionalLabels }}
{{- toYaml . | nindent 4 }}
{{- end }}
Expand Down
7 changes: 6 additions & 1 deletion charts/quickwit/templates/service.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ metadata:
name: {{ include "quickwit.fullname" . }}-searcher
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.searcher.selectorLabels" . | nindent 4 }}
annotations:
{{- with .Values.service.annotations }}
{{- toYaml . | nindent 4 }}
Expand Down Expand Up @@ -36,6 +37,7 @@ metadata:
name: {{ include "quickwit.fullname" . }}-headless
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.selectorLabels" . | nindent 4 }}
annotations:
{{- with .Values.service.annotations }}
{{- toYaml . | nindent 4 }}
Expand Down Expand Up @@ -72,6 +74,7 @@ metadata:
name: {{ include "quickwit.fullname" . }}-indexer
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.indexer.selectorLabels" . | nindent 4 }}
annotations:
{{- with .Values.service.annotations }}
{{- toYaml . | nindent 4 }}
Expand Down Expand Up @@ -104,6 +107,7 @@ metadata:
name: {{ include "quickwit.fullname" . }}-metastore
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.metastore.selectorLabels" . | nindent 4 }}
annotations:
{{- with .Values.service.annotations }}
{{- toYaml . | nindent 4 }}
Expand Down Expand Up @@ -136,6 +140,7 @@ metadata:
name: {{ include "quickwit.fullname" . }}-control-plane
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.control_plane.selectorLabels" . | nindent 4 }}
annotations:
{{- with .Values.service.annotations }}
{{- toYaml . | nindent 4 }}
Expand All @@ -161,14 +166,14 @@ spec:
name: grpc
selector:
{{- include "quickwit.control_plane.selectorLabels" . | nindent 4 }}

---
apiVersion: v1
kind: Service
metadata:
name: {{ include "quickwit.fullname" . }}-janitor
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.janitor.selectorLabels" . | nindent 4 }}
annotations:
{{- with .Values.service.annotations }}
{{- toYaml . | nindent 4 }}
Expand Down
2 changes: 2 additions & 0 deletions charts/quickwit/templates/serviceaccount.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ metadata:
name: {{ include "quickwit.serviceAccountName" . }}
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.selectorLabels" . | nindent 4 }}
app.kubernetes.io/component: rbac
{{- with .Values.serviceAccount.annotations }}
annotations:
{{- toYaml . | nindent 4 }}
Expand Down
2 changes: 2 additions & 0 deletions charts/quickwit/templates/servicemonitor.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ metadata:
name: {{ include "quickwit.fullname" . }}
labels:
{{- include "quickwit.labels" . | nindent 4 }}
{{- include "quickwit.selectorLabels" . | nindent 4 }}
app.kubernetes.io/component: metrics
{{- with .Values.serviceMonitor.additionalLabels }}
{{- toYaml . | nindent 4 }}
{{- end }}
Expand Down
Loading