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 upMarathon SD assumes all applications have a port. #1711
Comments
This comment has been minimized.
This comment has been minimized.
|
This is fixed in https://github.com/prometheus/prometheus/releases/tag/0.19.3 |
fabxc
closed this
Jun 14, 2016
This comment has been minimized.
This comment has been minimized.
lock
bot
commented
Mar 24, 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. |
lock
bot
locked and limited conversation to collaborators
Mar 24, 2019
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
rdacquisto commentedJun 6, 2016
•
edited
The following line is throwing a panic:
prometheus/retrieval/discovery/marathon/marathon.go
Line 229 in b5bfb50
This is because
task.Ports[0]assumes a length greater than 0, but the marathon configuration for the app is as follows:There appears to be a bad assumption that all apps will have ports.
The app that is causing this error is marathon-lb.
The work around is to add a random port definition, but this isn't ideal.