Skip to content

Commit

Permalink
Comment out one nginx metric/log from containerinsight_eks_prometheus…
Browse files Browse the repository at this point in the history
… test. (#335)

#334
  • Loading branch information
straussb authored Aug 27, 2021
1 parent f64ec73 commit 2646aee
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
"controller_class": {},
"controller_namespace": {},
"controller_pod": {},
"nginx_ingress_controller_nginx_process_connections_total": {},
"state": {}
},
"required": [
Expand All @@ -47,7 +46,6 @@
"controller_class",
"controller_namespace",
"controller_pod",
"nginx_ingress_controller_nginx_process_connections_total",
"state"
],
"additionalProperties": true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,15 +43,16 @@
value: {{cloudWatchContext.nginx.namespace}}
- name: Service
value: {{cloudWatchContext.nginx.namespace}}-ingress-nginx-controller-metrics
- metricName: nginx_ingress_controller_nginx_process_connections_total
namespace: ContainerInsights/Prometheus
dimensions:
- name: ClusterName
value: {{cloudWatchContext.clusterName}}
- name: Namespace
value: {{cloudWatchContext.nginx.namespace}}
- name: Service
value: {{cloudWatchContext.nginx.namespace}}-ingress-nginx-controller-metrics
# Commenting this metric, https://github.com/aws-observability/aws-otel-test-framework/issues/334
# - metricName: nginx_ingress_controller_nginx_process_connections_total
# namespace: ContainerInsights/Prometheus
# dimensions:
# - name: ClusterName
# value: {{cloudWatchContext.clusterName}}
# - name: Namespace
# value: {{cloudWatchContext.nginx.namespace}}
# - name: Service
# value: {{cloudWatchContext.nginx.namespace}}-ingress-nginx-controller-metrics
- metricName: nginx_ingress_controller_requests
namespace: ContainerInsights/Prometheus
dimensions:
Expand Down

0 comments on commit 2646aee

Please sign in to comment.