kingluo commented on code in PR #8006: URL: https://github.com/apache/apisix/pull/8006#discussion_r1019999749
########## t/discovery/consul_kv.t: ########## @@ -218,6 +218,7 @@ routes: GET /hello --- response_body_like eval qr/server [1-2]/ +--- ignore_error_log Review Comment: I check all `--- ignore_error_log`, and I confirm they are necessary because retries of these test cases occurrences are random, so you could not assert they happen or not. After all, they are just retries, and do not affect the test goals, so we have to ignore them. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
