Skip to content

[testing] fix non-windows test failure surfacing of resolver_component_tests_runner_invoker#39901

Closed
apolcyn wants to merge 2 commits intogrpc:masterfrom
apolcyn:fix_test_suite
Closed

[testing] fix non-windows test failure surfacing of resolver_component_tests_runner_invoker#39901
apolcyn wants to merge 2 commits intogrpc:masterfrom
apolcyn:fix_test_suite

Conversation

@apolcyn
Copy link
Contributor

@apolcyn apolcyn commented Jun 17, 2025

I was stumped why the failure mentioned in #39813 was only showing up on Windows, and I think this may be related.

I was forcing failures of this test on linux in #39891, and could not actually see a CI failure show up until this adding patch.

@apolcyn apolcyn added release notes: no Indicates if PR should not be in release notes lang/core labels Jun 17, 2025
@apolcyn apolcyn requested a review from drfloob June 17, 2025 23:19
anniefrchz pushed a commit to anniefrchz/grpc that referenced this pull request Jun 25, 2025
…t_tests_runner_invoker (grpc#39901)

I was stumped why the failure mentioned in grpc#39813 was only showing up on Windows, and I think this may be related.

I was forcing failures of this test on linux in grpc#39891, and could not actually see a CI failure show up until this adding patch.

Closes grpc#39901

COPYBARA_INTEGRATE_REVIEW=grpc#39901 from apolcyn:fix_test_suite 8ef10fe
PiperOrigin-RevId: 774988160
paulosjca pushed a commit to paulosjca/grpc that referenced this pull request Aug 23, 2025
…t_tests_runner_invoker (grpc#39901)

I was stumped why the failure mentioned in grpc#39813 was only showing up on Windows, and I think this may be related.

I was forcing failures of this test on linux in grpc#39891, and could not actually see a CI failure show up until this adding patch.

Closes grpc#39901

COPYBARA_INTEGRATE_REVIEW=grpc#39901 from apolcyn:fix_test_suite 8ef10fe
PiperOrigin-RevId: 774988160
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants