-
Notifications
You must be signed in to change notification settings - Fork 327
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
kuma-prometheus-sd: implement xDS client and integrate with Prometheus
via file_sd
discovery
#537
kuma-prometheus-sd: implement xDS client and integrate with Prometheus
via file_sd
discovery
#537
Conversation
36cdeb7
to
c306473
Compare
0ff5645
to
437ae8e
Compare
c306473
to
b5b0065
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are tests coming in a separate PR?
} | ||
|
||
log.Info("closing an xDS stream ...") | ||
if err := stream.Close(); err != nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see that Close()
was already called in defer. Why call it two times?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reworked
437ae8e
to
5fc2b57
Compare
43e9e6f
to
1c2aa5a
Compare
5fc2b57
to
7a6467a
Compare
1c2aa5a
to
648a255
Compare
7a6467a
to
975d14d
Compare
648a255
to
a678c9b
Compare
975d14d
to
481de6d
Compare
a678c9b
to
bfbf0f3
Compare
bfbf0f3
to
fc7a341
Compare
Summary
Prometheus
viafile_sd
discovery