-
Notifications
You must be signed in to change notification settings - Fork 3.9k
xds: Fixed to pretty-print XdsClusterResource with WrrLocality #12037
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
base: master
Are you sure you want to change the base?
Conversation
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.
Add the link to the fusion run logs to verify pretty-print.
Request you to share the details on how to perform the same activity on this defect branch as we have not done this earlier and not aware of the same. |
@shivaspeaks @AgraVator Any updates here as I can see the latest test fusion build has failed for validation of pretty print logs? is this because my user ID vinodhabib is not valid to trigger the same? how to proceed further on the same? Logs link for more - https://fusion2.corp.google.com/ci/kokoro/prod:grpc%2Fjava%2Fmaster%2Fbranch%2Fxds_k8s_lb/activity/8082ce1f-9e4c-42b8-9dd0-78a4746f2db6/log |
This build consists of the new changes of this PR. https://fusion2.corp.google.com/invocations/c3856846-eecb-475c-9623-7959a0f4912c/targets
|
@shivaspeaks @sergiitk @ejona86 As per the log details looks like TypedStruct descriptor details are also missing, I will add the same change and request you to trigger the test fusion build. |
As per the log details that look like RoundRobin descriptor details are also missing as below exception message, I will add the same change and request you to trigger the test fusion build.
I can see the missing entries for pretty print one by one after every build. Latest test fusion build logs - https://fusion2.corp.google.com/invocations/8bab4f8e-3799-49e0-bd2e-e53c02967f90/targets |
Fixed the XdsClusterResource with WrrLocality to pretty-print.
Fixes: #11918