Skip to content

Commit

Permalink
feat: add possibility of overriting the namespace
Browse files Browse the repository at this point in the history
  • Loading branch information
rzjfr committed Sep 10, 2024
1 parent 4b3ba57 commit 990f966
Show file tree
Hide file tree
Showing 60 changed files with 105 additions and 31 deletions.
11 changes: 11 additions & 0 deletions charts/jaeger-operator/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -47,3 +47,14 @@ Create chart name and version as used by the chart label.
app.kubernetes.io/name: {{ include "jaeger-operator.name" . }}
app.kubernetes.io/instance: {{ .Release.Name }}
{{- end }}

{{/*
Determine Namespace based on the inputs
*/}}
{{- define "jaeger-operator.namespace" -}}
{{- if .Values.namespaceOverride }}
{{- .Values.namespaceOverride -}}
{{- else -}}
{{- .Release.Namespace -}}
{{- end -}}
{{- end -}}
6 changes: 3 additions & 3 deletions charts/jaeger-operator/templates/certificate.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@ apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
name: {{ default "jaeger-operator-service-cert" .Values.certs.certificate.secretName }}
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
spec:
dnsNames:
- "{{ default "jaeger-operator-webhook-service" .Values.webhooks.service.name }}.{{ .Release.Namespace }}.svc"
- "{{ default "jaeger-operator-webhook-service" .Values.webhooks.service.name }}.{{ .Release.Namespace }}.svc.cluster.local"
- "{{ default "jaeger-operator-webhook-service" .Values.webhooks.service.name }}.{{ include "jaeger-operator.namespace" . }}.svc"
- "{{ default "jaeger-operator-webhook-service" .Values.webhooks.service.name }}.{{ include "jaeger-operator.namespace" . }}.svc.cluster.local"
issuerRef:
{{- if .Values.certs.issuer.create }}
kind: Issuer
Expand Down
2 changes: 1 addition & 1 deletion charts/jaeger-operator/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ include "jaeger-operator.fullname" . }}
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
labels:
{{ include "jaeger-operator.labels" . | indent 4 }}
{{- with .Values.extraLabels }}
Expand Down
2 changes: 1 addition & 1 deletion charts/jaeger-operator/templates/issuer.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ apiVersion: cert-manager.io/v1
kind: Issuer
metadata:
name: {{ default "selfsigned-issuer" .Values.certs.issuer.name }}
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
spec:
selfSigned: {}
{{- end }}
2 changes: 1 addition & 1 deletion charts/jaeger-operator/templates/jaeger.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ apiVersion: jaegertracing.io/v1
kind: Jaeger
metadata:
name: {{ include "jaeger-operator.fullname" . }}-jaeger
namespace: {{ default .Release.Namespace .Values.jaeger.namespace }}
namespace: {{ default .Values.jaeger.namespace (include "jaeger-operator.namespace" .) }}
{{- with .Values.jaeger.spec }}
spec:
{{ toYaml . | indent 2}}
Expand Down
4 changes: 2 additions & 2 deletions charts/jaeger-operator/templates/mutating-webhook.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ webhooks:
clientConfig:
service:
name: {{ default "jaeger-operator-webhook-service" .Values.webhooks.service.name }}
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
path: /mutate-v1-deployment
failurePolicy: Ignore
name: deployment.sidecar-injector.jaegertracing.io
Expand All @@ -39,7 +39,7 @@ webhooks:
clientConfig:
service:
name: {{ default "jaeger-operator-webhook-service" .Values.webhooks.service.name }}
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
path: /mutate-jaegertracing-io-v1-jaeger
failurePolicy: Fail
name: mjaeger.kb.io
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger-operator/templates/psp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: policy/v1beta1
kind: PodSecurityPolicy
metadata:
name: {{ include "jaeger-operator.fullname" . }}-operator-psp
namespace: {{ include "jaeger-operator.namespace" . }}
labels:
{{ include "jaeger-operator.labels" . | indent 4 }}
spec:
Expand Down
4 changes: 2 additions & 2 deletions charts/jaeger-operator/templates/role-binding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,12 @@ kind: {{ if .Values.rbac.clusterRole }}Cluster{{ end }}RoleBinding
apiVersion: rbac.authorization.k8s.io/v1
metadata:
name: {{ include "jaeger-operator.fullname" . }}
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
labels:
{{ include "jaeger-operator.labels" . | indent 4 }}
subjects:
- kind: ServiceAccount
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
name: {{ include "jaeger-operator.serviceAccountName" . }}
roleRef:
kind: {{ if .Values.rbac.clusterRole }}Cluster{{ end }}Role
Expand Down
2 changes: 1 addition & 1 deletion charts/jaeger-operator/templates/role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ kind: {{ if .Values.rbac.clusterRole }}Cluster{{ end }}Role
apiVersion: rbac.authorization.k8s.io/v1
metadata:
name: {{ include "jaeger-operator.fullname" . }}
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
labels:
{{ include "jaeger-operator.labels" . | indent 4 }}
rules:
Expand Down
2 changes: 1 addition & 1 deletion charts/jaeger-operator/templates/service-account.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ apiVersion: v1
kind: ServiceAccount
metadata:
name: {{ include "jaeger-operator.serviceAccountName" . }}
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
labels:
{{ include "jaeger-operator.labels" . | indent 4 }}
{{- if .Values.serviceAccount.annotations }}
Expand Down
4 changes: 2 additions & 2 deletions charts/jaeger-operator/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v1
kind: Service
metadata:
name: {{ include "jaeger-operator.fullname" . }}-metrics
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
labels:
{{ include "jaeger-operator.labels" . | indent 4 }}
{{- with .Values.serviceExtraLabels }}
Expand Down Expand Up @@ -33,7 +33,7 @@ metadata:
labels:
{{ include "jaeger-operator.labels" . | indent 4 }}
name: {{ default "jaeger-operator-webhook-service" .Values.webhooks.service.name }}
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
{{- if .Values.webhooks.service.annotations }}
annotations:
{{ toYaml .Values.webhooks.service.annotations | indent 4 }}
Expand Down
3 changes: 2 additions & 1 deletion charts/jaeger-operator/templates/validating-webhook.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,14 @@ metadata:
annotations:
cert-manager.io/inject-ca-from: {{ default .Release.Namespace .Values.certs.certificate.namespace }}/{{ default "jaeger-operator-service-cert" .Values.certs.certificate.secretName }}
name: jaeger-operator-validating-webhook-configuration
namespace: {{ include "jaeger-operator.namespace" . }}
webhooks:
- admissionReviewVersions:
- v1
clientConfig:
service:
name: {{ default "jaeger-operator-webhook-service" .Values.webhooks.service.name }}
namespace: {{ .Release.Namespace }}
namespace: {{ include "jaeger-operator.namespace" . }}
path: /validate-jaegertracing-io-v1-jaeger
failurePolicy: Fail
name: vjaeger.kb.io
Expand Down
2 changes: 2 additions & 0 deletions charts/jaeger-operator/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@
# This is a YAML-formatted file.
# Declare variables to be passed into your templates.

namespaceOverride: ""

image:
repository: jaegertracing/jaeger-operator
tag: 1.57.0
Expand Down
11 changes: 11 additions & 0 deletions charts/jaeger/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -235,6 +235,17 @@ We truncate at 63 chars because some Kubernetes name fields are limited to this
{{- end -}}
{{- end -}}

{{/*
Determine Namespace based on the inputs
*/}}
{{- define "jaeger.namespace" -}}
{{- if .Values.namespaceOverride }}
{{- .Values.namespaceOverride -}}
{{- else -}}
{{- .Release.Namespace -}}
{{- end -}}
{{- end -}}

{{- define "cassandra.host" -}}
{{- if .Values.provisionDataStore.cassandra -}}
{{- if .Values.storage.cassandra.nameOverride }}
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/agent-ds.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: apps/v1
kind: DaemonSet
metadata:
name: {{ template "jaeger.agent.name" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: agent
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/agent-sa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: ServiceAccount
metadata:
name: {{ template "jaeger.agent.serviceAccountName" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: agent
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/agent-svc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: Service
metadata:
name: {{ template "jaeger.agent.name" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: agent
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/allinone-agent-svc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: Service
metadata:
name: {{ template "jaeger.agent.name" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: service-agent
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/allinone-collector-svc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: Service
metadata:
name: {{ template "jaeger.collector.name" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: service-collector
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/allinone-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: {{ include "jaeger.fullname" . }}-sampling-strategies
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: all-in-one
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/allinone-deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ template "jaeger.fullname" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: all-in-one
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/allinone-ing.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
name: {{ template "jaeger.query.name" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: all-in-one
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/allinone-sa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: ServiceAccount
metadata:
name: {{ template "jaeger.fullname" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: all-in-one
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/cassandra-schema-job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ apiVersion: batch/v1
kind: Job
metadata:
name: {{ include "jaeger.fullname" . }}-cassandra-schema
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: cassandra-schema
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/cassandra-schema-sa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: ServiceAccount
metadata:
name: {{ template "jaeger.cassandraSchema.serviceAccountName" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: cassandra-schema
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/cassandra-secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: Secret
metadata:
name: {{ include "jaeger.fullname" . }}-cassandra
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
type: Opaque
Expand Down
2 changes: 1 addition & 1 deletion charts/jaeger/templates/collector-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@ apiVersion: v1
kind: ConfigMap
metadata:
name: {{ include "jaeger.fullname" . }}-sampling-strategies
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: collector
data:
strategies.json: |-
{{ tpl .Values.collector.samplingConfig . | indent 4 }}
{{- end }}

3 changes: 2 additions & 1 deletion charts/jaeger/templates/collector-deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ template "jaeger.collector.name" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: collector
Expand Down Expand Up @@ -38,7 +39,7 @@ spec:
securityContext:
{{- toYaml .Values.collector.podSecurityContext | nindent 8 }}
serviceAccountName: {{ template "jaeger.collector.serviceAccountName" . }}
{{- include "collector.imagePullSecrets" . | nindent 6 }}
{{- include "collector.imagePullSecrets" . | nindent 6 }}
{{- if .Values.collector.initContainers }}
initContainers:
{{- toYaml .Values.collector.initContainers | nindent 8 }}
Expand Down
11 changes: 6 additions & 5 deletions charts/jaeger/templates/collector-hpa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: {{ include "common.capabilities.hpa.apiVersion" ( dict "context" $ )
kind: HorizontalPodAutoscaler
metadata:
name: {{ template "jaeger.collector.name" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: collector
Expand All @@ -22,22 +23,22 @@ spec:
target:
averageUtilization: {{ .Values.collector.autoscaling.targetMemoryUtilizationPercentage }}
type: Utilization
{{- else }}
{{- else }}
targetAverageUtilization: {{ .Values.collector.autoscaling.targetMemoryUtilizationPercentage }}
{{- end }}
{{- end }}
{{- end }}
- type: Resource
resource:
name: cpu
{{- if eq "autoscaling/v2" (include "common.capabilities.hpa.apiVersion" ( dict "context" $ )) }}
{{- if eq "autoscaling/v2" (include "common.capabilities.hpa.apiVersion" ( dict "context" $ )) }}
target:
averageUtilization: {{ .Values.collector.autoscaling.targetCPUUtilizationPercentage | default 80 }}
type: Utilization
{{- else }}
{{- else }}
targetAverageUtilization: {{ .Values.collector.autoscaling.targetCPUUtilizationPercentage | default 80 }}
{{- end }}
{{- with .Values.collector.autoscaling.behavior }}
behavior:
{{- toYaml . | nindent 4 }}
{{- end }}
{{- end }}
{{- end }}
1 change: 1 addition & 0 deletions charts/jaeger/templates/collector-ing.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ apiVersion: {{ include "common.capabilities.ingress.apiVersion" $ }}
kind: Ingress
metadata:
name: {{ template "jaeger.collector.name" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
{{- if .Values.collector.ingress.labels }}
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/collector-sa.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: ServiceAccount
metadata:
name: {{ template "jaeger.collector.serviceAccountName" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: collector
Expand Down
4 changes: 3 additions & 1 deletion charts/jaeger/templates/collector-servicemonitor.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ metadata:
name: {{ template "jaeger.collector.name" . }}
{{- if .Values.collector.serviceMonitor.namespace }}
namespace: {{ .Values.collector.serviceMonitor.namespace }}
{{- else }}
namespace: {{ include "jaeger.namespace" . }}
{{- end }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
Expand Down Expand Up @@ -36,7 +38,7 @@ spec:
{{- end }}
namespaceSelector:
matchNames:
- {{ .Release.Namespace }}
- {{ include "jaeger.namespace" . }}
selector:
matchLabels:
app.kubernetes.io/component: collector
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/collector-svc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: Service
metadata:
name: {{ template "jaeger.collector.name" . }}
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: collector
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/elasticsearch-secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: v1
kind: Secret
metadata:
name: {{ include "jaeger.fullname" . }}-elasticsearch
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
annotations:
Expand Down
1 change: 1 addition & 0 deletions charts/jaeger/templates/es-index-cleaner-cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ apiVersion: {{ include "common.capabilities.cronjob.apiVersion" $ }}
kind: CronJob
metadata:
name: {{ include "jaeger.fullname" . }}-es-index-cleaner
namespace: {{ include "jaeger.namespace" . }}
labels:
{{- include "jaeger.labels" . | nindent 4 }}
app.kubernetes.io/component: es-index-cleaner
Expand Down
Loading

0 comments on commit 990f966

Please sign in to comment.