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 upbug in SD k8s missing wg.Wait. #4506
Comments
cofyc
referenced this issue
Aug 16, 2018
Merged
Wait for all internal discoveries are done before exiting. #4508
This comment has been minimized.
This comment has been minimized.
|
sorry, my mistake, ctx.Done() notifies all goroutines to exit, but it's better to wait all goroutines are done before exiting from |
gouthamve
closed this
in
#4508
Aug 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. |
lock
bot
locked and limited conversation to collaborators
Mar 22, 2019
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
krasi-georgiev commentedAug 15, 2018
prometheus/discovery/kubernetes/kubernetes.go
Lines 363 to 370 in 0b4d22b
this is missing wg.Wait so looks like a bug.
ping @cofyc