-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
bump istio-proxy #50813
bump istio-proxy #50813
Conversation
@@ -2422,7 +2422,7 @@ func buildQuery(src, dst echo.Instance) prometheus.Query { | |||
destns := dst.NamespaceName() | |||
|
|||
labels := map[string]string{ | |||
"reporter": "destination", | |||
"reporter": "waypoint", |
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.
we fixed a bunch of labels, seems like we should have more changes here @keithmattix
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.
Maybe need to add some more(new) labels
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.
Yeah there should be more things, but I don't see unknown checks below. Is the test ignoring them?
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.
Will this cause issues when moving the istio.io tests back to the main branch after we release 1.22.0?
it won't, c-ares on 1.30 didn't upgraded. |
Please provide a description of this PR:
fix: #50698
This PR contains three parts: