-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Support citadel endpoint #3962
Support citadel endpoint #3962
Conversation
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.
We still have comments saying that because of backward compatibility we support not specifying the endpoint for galley and pilot. This is not the case anymore, removing them would be nice
https://github.com/DataDog/integrations-core/pull/3962/files#diff-0654f0b39e01e79e8678617e2d69bdf2L287
Besides, I really like how generic it is to add a new istio endpoint, this is very readable.
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.
Looks good. Left few minor comments.
27b5cac
to
39d37e8
Compare
This adds supports for monitoring the citadel endpoint.