-
Notifications
You must be signed in to change notification settings - Fork 505
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 #525
Comments
Hey @mtagstyle , I haven't tried to use VHDS, so I don't really know the details. As far as consistency checks go, I'd suggest starting loosely and tightening up the check as you learn more about how Envoy treats VHDS. In general, the direction we are moving towards in the cache is to reduce the coupling to specific envoy types. So if we can support VHDS in all the cache types without introducing hard bindings to the VHDS type, that would be better IMHO. |
Ok some follow up after playing around:
I don't think the consistency checking for the existence of virtual hosts, as the VHDS source for routes never explicitly calls out any routes. |
@dmitri-d Could you confirm if my comment above is true? |
This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions. |
This issue has been automatically closed because it has not had activity in the last 37 days. If this issue is still valid, please ping a maintainer and ask them to label it as "help wanted" or "no stalebot". Thank you for your contributions. |
(Feel free to mark this as a dupe if necessary)
I'm planning on adding VHDS support, but having trouble understanding the history of this feature request...
VHDS was initially requested in #310. This issue was linked against #323, which "fixes" 310. Could someone comment on what they mean by "fixes" the issue?
Next some questions regarding implementation:
Server
SimpleCache
I.e. Can someone confirm that: There will never be any resources that need to explicitly reference a virtual host when using VHDS.
The text was updated successfully, but these errors were encountered: