Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upEC2 Discovery dropped all EC2 targets when AWS EC2 API respond 503. #2335
Comments
WeiBanjo
changed the title
EC2 Discovery dropped all EC2 targets when AWS API respond 503.
EC2 Discovery dropped all EC2 targets when AWS EC2 API respond 503.
Jan 10, 2017
brian-brazil
added
the
component/service discovery
label
Jul 7, 2017
brian-brazil
added
priority/P2
kind/bug
labels
Jul 14, 2017
This comment has been minimized.
This comment has been minimized.
|
This needs confirmation. |
This comment has been minimized.
This comment has been minimized.
|
@WeiBanjo is still the case even with the latest version? I did a big refactoring of the SD if you want to give it try and let me know if it still experiences the same bug. |
This comment has been minimized.
This comment has been minimized.
|
@krasi-georgiev Thanks. I will test that version in next couple days. |
This comment has been minimized.
This comment has been minimized.
|
We need some proper testing before the merge so I am on IRC if you encounter anymore bugs. |
This comment has been minimized.
This comment has been minimized.
|
@krasi-georgiev Your build fails on start up. See following logs. My configuration works on Prometheus v2 binary. Is there any EC2 discover related breaking changes I should aware ?
And my configuration passes config check
Some system infos
|
This comment has been minimized.
This comment has been minimized.
|
@WeiBanjo I tested using the config from your first comment, but it seems that you are using a different one with some recording rules in it. @gouthamve mentioned that this might be caused by #3524 so I rebased and uploaded the new version. |
This comment has been minimized.
This comment has been minimized.
|
@krasi-georgiev New build works. I tested EC2 API failure scenario. Prometheus keeps using cached targets after failure. Thanks for the fix. |
This comment has been minimized.
This comment has been minimized.
|
@WeiBanjo glad to hear than , let me know if you find any other bugs as I want to resolve before the merge. |
krasi-georgiev
referenced this issue
Dec 4, 2017
Merged
Decouple the discovery and refactor the retrieval package #3362
This comment has been minimized.
This comment has been minimized.
|
@brancz , @brian-brazil you can close this one. |
brian-brazil
closed this
Feb 17, 2018
This comment has been minimized.
This comment has been minimized.
lock
bot
commented
Mar 22, 2019
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
WeiBanjo commentedJan 10, 2017
•
edited
What did you do?
When AWS EC2 API has issues, Prometheus server loses EC2 targets.
What did you expect to see?
Prometheus should keep stale EC2 targets, when EC2 API call fails.
What did you see instead? Under which circumstances?
Prometheus dropped all EC2 targets when EC2 API respond 503
Environment