Nightly build

[github-actions]

Nightly build #10 failed. Please take a look and fix it ASAP.

[iNikem]

Seems like Armeria tests failed. @anuraaga can you take a look please?

[anuraaga]

FYI @trustin @ikhoon @minwoox I won't be able to look until tomorrow but you may want to check this out. Our tests which target the latest version of a library started failing with 1.9.0 which usually indicates an unintended bug or breaking change.

[trustin]

I'm not sure if it's related, but we changed how we initialize the client request context in 1.9.0 - line/armeria#3636

[minwoox]

I think I have to dig more but this change make it work:

  void sendRequestWithCallback(HttpRequest request, String method, URI uri, Map<String, String> headers, RequestResult requestResult) {
    Context ctx = Context.current();
    client.execute(request).aggregate().whenComplete {response, throwable ->
      ctx.makeCurrent().withCloseable {
        requestResult.complete({ response.status().code() }, throwable)
      }
    }
  }

opentelemetry-java-instrumentation/instrumentation/armeria-1.3/testing/src/main/groovy/io/opentelemetry/instrumentation/armeria/v1_3/AbstractArmeriaHttpClientTest.groovy

Lines 47 to 52 in f7a24b0

	@Override
	void sendRequestWithCallback(HttpRequest request, String method, URI uri, Map<String, String> headers, RequestResult requestResult) {
	client.execute(request).aggregate().whenComplete {response, throwable ->
	requestResult.complete({ response.status().code() }, throwable)
	}
	}