Skip to content

Commit

Permalink
Merge pull request #115 from codecademy-engineering/lindsay-job-bugfix
Browse files Browse the repository at this point in the history
remove datadog.enabled from jobs
  • Loading branch information
lindsaylandry authored Feb 7, 2024
2 parents 4ee43c3 + 272f36f commit 60dc52a
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
2 changes: 1 addition & 1 deletion charts/service/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: service
description: A generic k8s service chart
type: application
version: 1.8.11
version: 1.8.12
maintainers:
- email: [email protected]
name: devops
5 changes: 0 additions & 5 deletions charts/service/templates/jobs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ spec:
labels:
{{- with $datadog }}
{{- if and .service .env }}
tags.datadoghq.com/enabled: {{ .enabled | quote }}
tags.datadoghq.com/env: {{ .env }}
tags.datadoghq.com/service: {{ .service }}
tags.datadoghq.com/version: {{ .version | quote }}
Expand All @@ -41,10 +40,6 @@ spec:
valueFrom:
fieldRef:
fieldPath: status.hostIP
- name: DATADOG__ENABLED
valueFrom:
fieldRef:
fieldPath: metadata.labels['tags.datadoghq.com/enabled']
- name: DD_ENV
valueFrom:
fieldRef:
Expand Down

0 comments on commit 60dc52a

Please sign in to comment.