Merge pull request #74 from meisterplan/feature/KNUTH-129111-set-retentionlogging-annota

Add new logging control annotations (collect, retention)

Author: itd-ndr

.github/workflows/ci.yaml

@@ -4,6 +4,10 @@ on:
     branches: [master]
   pull_request:
     branches: [master]
+permissions:
+  checks: write
+  contents: read
+  pull-requests: write
 jobs:
   test:
     runs-on: ubuntu-latest

CHANGELOG.md

@@ -1,5 +1,9 @@
 # spring-service
 
+## 5.5.0
+
+- Add support for logging annotations
+
 ## 5.4.0
 
 - Allow to configure `overwriteRequestRetries`
@@ -239,6 +243,10 @@ deployment** with a short 503 service disruption due to a [bug in the ingress co
 
 # cronjob
 
+## 3.2.0
+
+- Add support for logging annotations
+
 ## 3.1.1
 
 - Remove now unnecessary, confusing Helm chart labels on prom resources

charts/cronjob/Chart.yaml

@@ -1,4 +1,4 @@
 apiVersion: v1
 name: cronjob
 description: A generalized cronjob that can access secrets.
-version: 3.1.1
+version: 3.2.0

charts/cronjob/templates/_helpers.tpl

@@ -0,0 +1,12 @@
+{{- define "k8s.annotations" }}
+{{- $annotations := dict }}
+{{- if ne .Values.logging.collect true }}
+{{-     $_ := set $annotations "com.meisterplan.logs/collect" (printf "%v" .Values.logging.collect) }}
+{{- end }}
+{{- if ne .Values.logging.retention "default" }}
+{{-     $_ := set $annotations "com.meisterplan.logs/retention" .Values.logging.retention }}
+{{- end }}
+{{- if $annotations }}
+{{-     toYaml $annotations }}
+{{- end }}
+{{- end }}

charts/cronjob/templates/cronjob.yaml

@@ -3,6 +3,11 @@ apiVersion: batch/v1
 metadata:
   name: "{{ required "cronJobName must be set!" .Values.cronJobName }}"
   namespace: "{{ required "Namespace must be set!" .Values.namespace }}"
+  {{- $annotations := include "k8s.annotations" . -}}
+  {{- if $annotations }}
+  annotations:
+    {{- $annotations | nindent 4 }}
+  {{- end }}
 spec:
   concurrencyPolicy: "Replace"
   schedule: {{ required "Schedule must be set!" .Values.schedule | quote }}

charts/cronjob/values.yaml

@@ -36,6 +36,10 @@ resources:
   cpu:
     guarantee: 100m
 
+logging:
+  collect: true # can be toggled to turn off log collection
+  retention: default # can be set to 'verbose' to shorten retention
+
 env:
   fromSecret: {}
   #  SOME_SECRET:

charts/spring-service/Chart.yaml

@@ -1,4 +1,4 @@
 apiVersion: v1
 name: spring-service
 description: A generalized deployment for Meisterplan Spring Boot services in Kubernetes.
-version: 5.4.0
+version: 5.5.0

…e/templates/post-deployment/_helpers.tpl …ts/spring-service/templates/_helpers.tplcharts/spring-service/templates/post-deployment/_helpers.tpl renamed to charts/spring-service/templates/_helpers.tpl charts/spring-service/templates/post-deployment/_helpers.tpl renamed to charts/spring-service/templates/_helpers.tpl

@@ -13,3 +13,16 @@
 {{ $.Values.ingress.public.subDomain }}.{{ required "ingress.clusterDomain must be set!" $.Values.ingress.clusterDomain }}
 {{- end -}}
 {{- end }}
+
+{{- define "k8s.annotations" }}
+{{- $annotations := dict }}
+{{- if ne .Values.logging.collect true }}
+{{-     $_ := set $annotations "com.meisterplan.logs/collect" (printf "%v" .Values.logging.collect) }}
+{{- end }}
+{{- if ne .Values.logging.retention "default" }}
+{{-     $_ := set $annotations "com.meisterplan.logs/retention" .Values.logging.retention }}
+{{- end }}
+{{- if $annotations }}
+{{-     toYaml $annotations }}
+{{- end }}
+{{- end }}

charts/spring-service/templates/deployment.yaml

@@ -3,6 +3,11 @@ apiVersion: apps/v1
 metadata:
   name: "{{ .Values.serviceName }}-{{ required "Image Tag must be set!" .Values.image.tag }}"
   namespace: "{{ required "Namespace must be set!" .Values.namespace }}"
+  {{- $annotations := include "k8s.annotations" . -}}
+  {{- if $annotations }}
+  annotations:
+    {{- $annotations | nindent 4 }}
+  {{- end }}
 spec:
   replicas: {{ .Values.replicaCount }}
   {{- if .Values.timeouts.progressDeadlineSeconds }}

charts/spring-service/values.yaml

@@ -37,6 +37,10 @@ resources:
   cpu:
     guarantee: 100m
 
+logging:
+  collect: true # can be toggled to turn off log collection
+  retention: default # can be set to 'verbose' to shorten retention
+
 ingress:
   clusterDomain: "" # domain of the cluster, e.g. meisterplan.io
   public: