Skip to content

Commit dc2e3de

Browse files
Merge pull request #82 from jarretlavallee/gh_81
(GH-81) Enable client ssl cert for metrics
2 parents dbcc8bb + 7c08745 commit dc2e3de

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

files/pe_metrics.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,10 +124,11 @@ def retrieve_additional_metrics(host, port, use_ssl, metrics_type, metrics)
124124

125125
host_url = generate_host_url(host, port, use_ssl)
126126

127+
metrics_array = []
127128
endpoint = "#{host_url}/metrics/v2/read"
128129
metrics_output = post_endpoint(endpoint, use_ssl, metrics.to_json)
130+
return metrics_array if metrics_output.empty?
129131

130-
metrics_array = []
131132
metrics.each_index do |index|
132133
metric_name = metrics[index]['name']
133134
metric_data = metrics_output[index]

files/tk_metrics

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
#!/opt/puppetlabs/puppet/bin/ruby
22

3-
# Metrics endpoints for our Trapper Keeper services do not require a client certificate.
4-
USE_CLIENTCERT = false
3+
USE_CLIENTCERT = true
54

65
require_relative 'pe_metrics'
76

0 commit comments

Comments
 (0)