Skip to content

Treat failure to collect metric as fatal#83

Merged
SuperQ merged 1 commit intoprometheus-community:masterfrom
cpick:fatal-collection-error
May 13, 2020
Merged

Treat failure to collect metric as fatal#83
SuperQ merged 1 commit intoprometheus-community:masterfrom
cpick:fatal-collection-error

Conversation

@cpick
Copy link

@cpick cpick commented May 1, 2020

Exit on any failure to collect metrics. This will give
stackdriver_exporter a chance to start over and perhaps recover from
transient errors. For example: if it's Google credentials have expired
and have been replaced, it can restart to try to load up new ones.

Closes #66

@SuperQ
Copy link
Contributor

SuperQ commented May 12, 2020

Can you rebase this against master and sign off the DCO (git commit -s --amend)? That should fix up the tests.

Also, if you wouldn't mind adding an entry to the CHANGELOG

* [BUGFIX] Treat failure to collect metric as fatal #83

Exit on any failure to collect metrics.  This will give
`stackdriver_exporter` a chance to start over and perhaps recover from
transient errors.  For example: if it's Google credentials have expired
and have been replaced, it can restart to try to load up new ones.

Signed-off-by: Chris Pick <chris@chrispick.com>
@cpick cpick force-pushed the fatal-collection-error branch from c5c3b88 to c49aace Compare May 12, 2020 16:09
@cpick
Copy link
Author

cpick commented May 12, 2020

Done and done.

Copy link
Contributor

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks

@SuperQ SuperQ merged commit f9cd352 into prometheus-community:master May 13, 2020
SuperQ added a commit that referenced this pull request May 14, 2020
* [CHANGE] Treat failure to collect metric as fatal #83
* [CHANGE] Switch logging to promlog #88
* [FEATURE] Add metrics prefix collect URL param #87

Signed-off-by: Ben Kochie <superq@gmail.com>
@SuperQ SuperQ mentioned this pull request May 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Recover from expired/replaced credentials

2 participants