Sergii Tkachenko

Results 84 comments of Sergii Tkachenko

Also, that CSDS dump doesn't show any endpoints. It should be something like ```json "endpoints": [{ "locality": { "subZone": "ib:us-central1-a_2827676496526826176_neg" }, "lbEndpoints": [{ "endpoint": { "address": { "socketAddress": { "address":...

Could you please try to reproduce this with v1.63.1 or v1.64.0? v1.63.0 contained a few bugs that were fixed in v1.64.0 and backported to v1.63.1: https://github.com/grpc/grpc-java/releases/tag/v1.63.1.

tests (11) (pull_request): ``` TomcatTransportTest > clientChecksInboundMetadataSize_header FAILED java.lang.NullPointerException at CoyoteOutputStream.java:154 ``` android: ``` > Task :grpc-android-interop-testing:mergeDexDebug ERROR:/home/kbuilder/.gradle/caches/modules-2/files-2.1/com.google.errorprone/error_prone_annotations/2.36.0/227d4d4957ccc3dc5761bd897e3a0ee587e750a7/error_prone_annotations-2.36.0.jar: D8: java.lang.NullPointerException > Task :grpc-android-interop-testing:lintVitalAnalyzeRelease > Task :grpc-android-interop-testing:minifyReleaseWithR8 WARNING:Missing classes detected while...

`TomcatTransportTest` failure was a fluke.

@HiramSilvey I expect it to be merged. High-level design has been approved some time ago. In the past few weeks we needed to reconsider some details and reach the consensus...

I don't think to suppressing `RuntimeError` when no aio loop is running (as proposed in #39523) is the right thing to do. Note that in the reproduction case `aio.secure_channel("http://localhost/", grpc.local_channel_credentials())`...

Found some historical context for the current behavior (create loops on demand instead of RuntimeError): #23377. We may have to preserve this approach after all. Will investigate further.

The last Linux Artifacts with `org.gradle.jvmargs=-Xmx24g -XX:MaxMetaspaceSize=2g` took 29 min, 35 sec.