Re: [prometheus-developers] Debug Logging for CloudWatch Exporter Pt. 2

75 views
Skip to first unread message
Message has been deleted

Brian Brazil

unread,
May 16, 2020, 4:07:28 AM5/16/20
to Sally Lehman, Prometheus Developers
On Sat, 16 May 2020 at 04:59, Sally Lehman <rot...@gmail.com> wrote:

https://github.com/prometheus/cloudwatch_exporter/pull/225 is an existing pull to add debug logging to this repo that doesn't look like it has firm blockers for a merge.


There are comments on the PR which the author has not addressed, thus it is not ready for merge.
 

It is functional, once merged with master, as I've done here https://github.com/rothsa/cloudwatch_exporter/tree/debug_logging


FINEST: GetMetricStatisticsResult, works. It verifies that the metrics config part is loading, Great! From this I can tell that my config is loading, and I can see the current successful results of the scrape. This pull then, from my perspective is useful and helpful so far.


The contributing page asks to get permission on this list first. I would like to add the following:


1. Logs to verify that the client has authenticated successfully, using role_arn like I expect it to. 

2. Add to or modify GetMetricStatisticsRequest logs to give me additional data surrounding the datapoints response that indicates why all my ListMetrics calls return empty datapoint arrays as shown:

```

FINEST: GetMetricStatisticsRequest: {Namespace: AWS/ELB,MetricName: HealthyHostCount,Dimensions: [{Name: LoadBalancerName,Value: rothsa_lb}],StartTime: Sat May 16 02:36:23 UTC 2020,EndTime: Sat May 16 02:46:23 UTC 2020,Period: 60,Statistics: [Sum, SampleCount, Minimum, Maximum, Average],ExtendedStatistics: [],}

May 16, 2020 2:56:23 AM io.prometheus.cloudwatch.CloudWatchCollector scrape

FINEST: GetMetricStatisticsResult[AWS/ELB/HealthyHostCount]: {Label: HealthyHostCount,Datapoints: []}

```


If I get a positive response here I will create a pull on https://github.com/prometheus/cloudwatch_exporter with the above contributions. 

This is probably best discussed on a PR, as this is all repo-specific.

Brian
 


Thanks for all you do. :)



--
You received this message because you are subscribed to the Google Groups "Prometheus Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-devel...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/115d3e68-4696-457c-8e41-8d2dcd6ab1b5%40googlegroups.com.


--
Reply all
Reply to author
Forward
Message has been deleted
0 new messages