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

Add Values.podLabels and Values.podAnnotations #217

Merged
merged 23 commits into from
May 13, 2024
Merged

Add Values.podLabels and Values.podAnnotations #217

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
a316966
Add terminationGracePeriodSeconds, podAnnotations, podLabels
vostres Apr 30, 2024
480e280
Merge remote-tracking branch 'origin/master' into feature/termination…
vostres Apr 30, 2024
02dc16f
Add podAnnotations, podLabels, terminationGracePeriodSeconds to value…
vostres Apr 30, 2024
5d560de
Refactor if-clauses to check Values.podAnnotations
vostres Apr 30, 2024
4419de0
Merge branch 'master' into feature/terminationGracePeriodSeconds
vostres May 7, 2024
761df1a
Remove logic for terminationGracePeriodSeconds
vostres May 7, 2024
da2982d
Add podAnnotations/Labels to producer-app-cleanup-job
vostres May 7, 2024
45788ad
Add podAnnotations/Labels to streams-app-cleanup-job
vostres May 7, 2024
5625c64
Add podAnnotations/Labels to producer-app
vostres May 7, 2024
df0abe9
Remove empty lines EOF
vostres May 7, 2024
a69d44b
Fix typo
vostres May 7, 2024
87c60fe
Update README.md for streams-app chart
vostres May 8, 2024
862fc1b
Update README.md for producer-app chart
vostres May 8, 2024
5796156
Add Values.annotations to the metadata for cleanup jobs
vostres May 8, 2024
a558bac
Remove Values.podAnnotations.consumerGroup
vostres May 13, 2024
29115dd
Add Values.annotations to job.yaml for producer-app
vostres May 13, 2024
a09ce6d
Update annotations/labels rclone-copy chart
vostres May 13, 2024
95932fb
Delete space
vostres May 13, 2024
d2eb12d
Add spaces to streams-app-cleanup-job chart
vostres May 13, 2024
327f824
Delete spaces
vostres May 13, 2024
7cfa7d8
Delete blank spaces before "-"
vostres May 13, 2024
b2599c3
Delete consumerGroup for pod
vostres May 13, 2024
af9af2e
Resolve merge conflicts
vostres May 13, 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
6 changes: 3 additions & 3 deletions charts/producer-app-cleanup-job/templates/job.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,19 +15,19 @@ spec:
ttlSecondsAfterFinished: 30
template:
metadata:
{{- if or .Values.awsRole .Values.annotations }}
philipp94831 marked this conversation as resolved.
Show resolved Hide resolved
{{- if or .Values.awsRole .Values.podAnnotations }}
annotations:
{{- if .Values.awsRole }}
iam.amazonaws.com/role: {{ .Values.awsRole }}
{{- end }}
{{- range $key, $value := .Values.annotations }}
{{- range $key, $value := .Values.podAnnotations }}
{{ $key | quote }}: {{ $value | quote }}
{{- end }}
{{- end }}
labels:
app: {{ template "producer-app-cleanup-job.name" . }}
release: {{ .Release.Name }}
{{- range $key, $value := .Values.labels }}
{{- range $key, $value := .Values.podLabels }}
{{ $key }}: {{ $value }}
{{- end }}
spec:
Expand Down
4 changes: 4 additions & 0 deletions charts/producer-app-cleanup-job/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,3 +87,7 @@ javaOptions:
# - "-XX:MinRAMPercentage=50.0"

backoffLimit: 6

podAnnotations: []

podLabels: []
6 changes: 3 additions & 3 deletions charts/producer-app/templates/pod.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,19 +1,19 @@
{{- define "producer-app.podTemplate" -}}
{{- $root := . -}}
metadata:
{{- if or .Values.awsRole .Values.annotations }}
vostres marked this conversation as resolved.
Show resolved Hide resolved
{{- if or .Values.awsRole .Values.podAnnotations }}
annotations:
{{- if .Values.awsRole }}
iam.amazonaws.com/role: {{ .Values.awsRole }}
{{- end }}
{{- range $key, $value := .Values.annotations }}
{{- range $key, $value := .Values.podAnnotations }}
{{ $key | quote }}: {{ $value | quote }}
{{- end }}
{{- end }}
labels:
app: {{ template "producer-app.name" . }}
release: {{ .Release.Name }}
{{- range $key, $value := .Values.labels }}
{{- range $key, $value := .Values.podLabels }}
{{ $key }}: {{ $value }}
{{- end }}
spec:
Expand Down
4 changes: 4 additions & 0 deletions charts/producer-app/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,3 +121,7 @@ livenessProbe: {}

# Arbitrary Probe v1 definition for producer: https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.25/#probe-v1-core
readinessProbe: {}

podAnnotations: []

podLabels: []
6 changes: 3 additions & 3 deletions charts/streams-app-cleanup-job/templates/job.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,19 +15,19 @@ spec:
ttlSecondsAfterFinished: 30
template:
metadata:
{{- if or .Values.awsRole .Values.annotations }}
philipp94831 marked this conversation as resolved.
Show resolved Hide resolved
{{- if or .Values.awsRole .Values.podAnnotations }}
annotations:
{{- if .Values.awsRole }}
iam.amazonaws.com/role: {{ .Values.awsRole }}
{{- end }}
{{- range $key, $value := .Values.annotations }}
{{- range $key, $value := .Values.podAnnotations }}
{{ $key | quote }}: {{ $value | quote }}
{{- end }}
{{- end }}
labels:
app: {{ template "streams-app-cleanup-job.name" . }}
release: {{ .Release.Name }}
{{- range $key, $value := .Values.labels }}
{{- range $key, $value := .Values.podLabels }}
{{ $key }}: {{ $value }}
{{- end }}
spec:
Expand Down
4 changes: 4 additions & 0 deletions charts/streams-app-cleanup-job/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,3 +100,7 @@ javaOptions:
# - "-XX:MinRAMPercentage=50.0"

backoffLimit: 6

podAnnotations: []

podLabels: []
8 changes: 4 additions & 4 deletions charts/streams-app/templates/deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ spec:
release: {{ .Release.Name }}
template:
metadata:
{{- if or (or (or .Values.awsRole .Values.prometheus.jmx.enabled) .Values.autoscaling.consumerGroup) .Values.annotations }}
{{- if or (or (or .Values.awsRole .Values.prometheus.jmx.enabled) .Values.autoscaling.consumerGroup) .Values.podAnnotations }}
philipp94831 marked this conversation as resolved.
Show resolved Hide resolved
annotations:
{{- if .Values.awsRole }}
iam.amazonaws.com/role: {{ .Values.awsRole }}
Expand All @@ -48,17 +48,17 @@ spec:
prometheus.io/scrape: "true"
prometheus.io/port: {{ .Values.prometheus.jmx.port | quote }}
{{- end }}
{{- range $key, $value := .Values.annotations }}
{{- range $key, $value := .Values.podAnnotations }}
{{ $key | quote }}: {{ $value | quote }}
{{- end }}
{{- if and .Values.autoscaling.consumerGroup (not .Values.annotations.consumerGroup) }}
{{- if and .Values.autoscaling.consumerGroup (not .Values.podAnnotations.consumerGroup) }}
vostres marked this conversation as resolved.
Show resolved Hide resolved
consumerGroup: {{ .Values.autoscaling.consumerGroup | quote }}
{{- end }}
{{- end }}
labels:
app: {{ template "streams-app.name" . }}
release: {{ .Release.Name }}
{{- range $key, $value := .Values.labels }}
{{- range $key, $value := .Values.podLabels }}
{{ $key }}: {{ $value }}
{{- end }}
spec:
Expand Down
4 changes: 4 additions & 0 deletions charts/streams-app/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -200,3 +200,7 @@ persistence:
## GKE, AWS & OpenStack)
##
# storageClass: ""

podAnnotations: []

podLabels: []
Loading