Easwar Swaminathan
Easwar Swaminathan
@markdroth @ejona86 @dfawley : Would one of you be able to review this? Thanks.
For aggregate clusters, all configuration comes from the underlying clusters, including LRS. https://github.com/grpc/proposal/blob/master/A75-xds-aggregate-cluster-behavior-fixes.md contains the latest spec on how aggregate clusters should work. A74 is related as well.
That PR does not handle the issue mentioned in here. That PR only adds server config to the `ClusterUpdate` struct (this work was not done as part of A47, but...
@bozaro : Thank you for figuring this out and sending us a PR. There are a few options available to fix this issue: - Make breaking API changes to this...
@jonmchan : Would be able to handle the last comment from @apolcyn ? Thanks
Ping @DenisPavlov
@ejona86 @temawi Same here with an unsigned CLA.
@thisisnotapril : Gentle ping on this one
Thanks for the comment, Mark !! We will ensure that the added API is an internal-only API.