Merge pull request #22 from Financial-Times/remove-graphite-integration

Remove graphite integration
Financial-Times · Sep 19, 2018 · 66e184c · 66e184c
2 parents b92b0a2 + 0da409e
commit 66e184c
Show file tree

Hide file tree

Showing 5 changed files with 77 additions and 123 deletions.
diff --git a/Gopkg.lock b/Gopkg.lock
diff --git a/Gopkg.toml b/Gopkg.toml
@@ -24,11 +24,6 @@
 #   go-tests = true
 #   unused-packages = true
 
-
-[[constraint]]
-  name = "github.com/Financial-Times/base-ft-rw-app-go"
-  version = "1.0.2"
-
 [[constraint]]
   name = "github.com/Financial-Times/go-fthealth"
   version = "0.4.0"

diff --git a/helm/basic-tme-transformer/templates/deployment.yaml b/helm/basic-tme-transformer/templates/deployment.yaml
@@ -55,15 +55,6 @@ spec:
             secretKeyRef:
               name: global-secrets
               key: tme.token
-        - name: GRAPHITE_ADDRESS
-          valueFrom:
-            configMapKeyRef:
-              name: global-config
-              key: graphite.address
-        - name: GRAPHITE_PREFIX
-          value: 'coco.services.{{required "The target_env value is required for this app. Use helm upgrade ... --set target_env=... when installing." .Values.target_env}}.{{ .Values.service.name }}'
-        - name: LOG_METRICS
-          value: "{{ .Values.env.LOG_METRICS }}"
         - name: WRITER_ENDPOINT
           value: {{ .Values.env.WRITER_ENDPOINT }}
         - name: WRITER_WORKERS

diff --git a/helm/basic-tme-transformer/values.yaml b/helm/basic-tme-transformer/values.yaml
@@ -15,6 +15,5 @@ resources:
     memory: 2Gi
 env:
   BASE_URL: "http://basic-tme-transformer:8080/transformers"
-  LOG_METRICS: "false"
   WRITER_ENDPOINT: "http://concepts-rw-s3:8080/concepts"
   WRITER_WORKERS: "10"