-
Notifications
You must be signed in to change notification settings - Fork 517
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
VHDS Support #529
VHDS Support #529
Conversation
…test pkg to make adding new listeners easier Signed-off-by: mtagstyle <anthony.tran@live.ca>
Signed-off-by: mtagstyle <anthony.tran@live.ca>
@mtagstyle would you mind rebase this? I'll start giving it a look |
…into envoyproxy-main Signed-off-by: mtagstyle <anthony.tran@live.ca>
Signed-off-by: mtagstyle <anthony.tran@live.ca>
Sorry for the delay, I've rebased as requested. |
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.
This looks good no comments from me.
@alecholmez I don't think I have permissions to merge it in. So I leave it at your discretion. |
@mtagstyle ah oops sorry, merging! Thanks for working this |
These changes allow for VHDS to be used in the management server. This is an initial PR and I would appreciate feedback on anything I may not have considered yet (API usage, missing test cases, etc.)
Currently, if the ConfigSource under the VHDS portion is set to anything other than "DELTA_GRPC", Envoy will complain about it not being a supported protocol type. For more information, please refer to #525