Skip to content

fix build#4642

Merged
istio-testing merged 4 commits intoistio:masterfrom
zirain:config-subscription-ads
May 6, 2023
Merged

fix build#4642
istio-testing merged 4 commits intoistio:masterfrom
zirain:config-subscription-ads

Conversation

@zirain
Copy link
Copy Markdown
Member

@zirain zirain commented May 4, 2023

fixes: #4631

@zirain zirain requested a review from a team May 4, 2023 02:49
@istio-testing istio-testing added do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 4, 2023
@zirain
Copy link
Copy Markdown
Member Author

zirain commented May 4, 2023

/retest

looks like a flaky

@kyessenov
Copy link
Copy Markdown
Contributor

Something is wonky in prometheus output:
stats_test.go:521: text format parsing error in line 155: second TYPE line for metric name "envoy_cluster_assignment_stale", or TYPE reported after samples

@kyessenov
Copy link
Copy Markdown
Contributor

Probably related to envoyproxy/envoy#24998, but I'm not seeing anything wrong in the envoy output manually.

@zirain
Copy link
Copy Markdown
Member Author

zirain commented May 4, 2023

Probably related to envoyproxy/envoy#24998, but I'm not seeing anything wrong in the envoy output manually.

yeah, seems like some space or tab char error

@zirain
Copy link
Copy Markdown
Member Author

zirain commented May 4, 2023

wait envoyproxy/envoy#27173

@zirain zirain changed the title [WIP] fix ads build fix build May 6, 2023
@istio-testing istio-testing removed the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label May 6, 2023
@zirain
Copy link
Copy Markdown
Member Author

zirain commented May 6, 2023

/retest

@istio-testing istio-testing merged commit a7e07f1 into istio:master May 6, 2023
@zirain zirain deleted the config-subscription-ads branch May 7, 2023 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants