Ian Rudie
Ian Rudie
Yeah, Virtual service is not where we want it. It was just a proto where I found the marker for a (admittedly sloppy) copy/paste example. Sorry for the confusion there.
For the 1.22 release I think we can go ahead and mark this one as closed and create some follow ups for the future work/refinements/enhancements @louiscryan highlighted. @linsun @howardjohn, any...
@MorrisLaw there are a number of tests that send to workload traffic in the ambient integ suite but the waypoint won't have your new annotation to allow all traffic so...
nice! looks like fixing that istioctl bug did the trick
@moshevayner is taking a look at this. I don't have permission to assign to him so just self assigning for now so that folks know someone is looking at it
I dropped the assignment to avoid confusion or unnecessary delay. I think we can reprioritize and let someone else pick it up
cc @linsun and @louiscryan
@keithmattix, The `#` prefix is not used elsewhere making this a bit awkward. In `waypoint-for` a traffic type of `none` (without the hash) can be chosen for instance. It was...
@istio/wg-networking-maintainers-pilot, is anyone able to review and provide feedback on whether this is suitable to move forward or requires further refinements?
IMO this makes more sense as a `task` type doc than a `reference` and at any rate I don't think it makes much sense nested under virtual service reference.