-
Notifications
You must be signed in to change notification settings - Fork 437
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
add target clusters to parameters api #10091
base: main
Are you sure you want to change the base?
Conversation
Visit the preview URL for this PR (updated for commit 2e74981): https://gloo-edge--pr10091-params-add-target-cl-4xmz2t3f.web.app (expires Fri, 11 Oct 2024 22:08:34 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 77c2b86e287749579b7ff9cadb81e099042ef677 |
I mentioned this when we discussed originally that I don't love the idea of adding concepts to this API that aren't used by |
I'll set something up to review with product some decisions that have been made here to achieve this PoC, we can adjust based off product feedback. |
reference: https://github.com/solo-io/gloo-mesh-enterprise/issues/18646