Fix for xray to work within aws http/rest extensions #23038

patriot1burke · 2022-01-20T01:06:08Z

Reverted code substitution as xray now looks within system properties

quarkus-bot · 2022-01-20T01:37:57Z

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building `7dad206`

Status	Name	Step	Failures	Logs	Raw logs
✖	JVM Tests - JDK 11	`Build`	Failures	Logs	Raw logs
✖	JVM Tests - JDK 11 Windows	`Build`	Failures	Logs	Raw logs
✖	JVM Tests - JDK 17	`Build`	Failures	Logs	Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 11 #

- Failing: integration-tests/amazon-lambda-http-resteasy integration-tests/amazon-lambda-s3event integration-tests/amazon-lambda-stream-handler

📦 integration-tests/amazon-lambda-http-resteasy

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testPostEmpty - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.test404 - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testCustomIDPSecurityContext - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testProxyRequestContext - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testGetText - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testPostBinary - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testContext - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testPostText - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

📦 integration-tests/amazon-lambda-s3event

✖ io.quarkus.it.amazon.lambda.S3EventTestCase.testS3 - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

📦 integration-tests/amazon-lambda-stream-handler

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testSimpleLambdaSuccess - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

⚙️ JVM Tests - JDK 11 Windows #

- Failing: integration-tests/amazon-lambda-http-resteasy integration-tests/amazon-lambda-s3event integration-tests/amazon-lambda-stream-handler

📦 integration-tests/amazon-lambda-http-resteasy

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testPostEmpty - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.test404 - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testCustomIDPSecurityContext - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testProxyRequestContext - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testGetText - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testPostBinary - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testContext - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testPostText - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

📦 integration-tests/amazon-lambda-s3event

✖ io.quarkus.it.amazon.lambda.S3EventTestCase.testS3 - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

📦 integration-tests/amazon-lambda-stream-handler

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testSimpleLambdaSuccess - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

⚙️ JVM Tests - JDK 17 #

- Failing: integration-tests/amazon-lambda-http-resteasy integration-tests/amazon-lambda-s3event integration-tests/amazon-lambda-stream-handler

📦 integration-tests/amazon-lambda-http-resteasy

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testPostEmpty - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.test404 - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testCustomIDPSecurityContext - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testProxyRequestContext - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testGetText - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testPostBinary - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testContext - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testPostText - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

📦 integration-tests/amazon-lambda-s3event

✖ io.quarkus.it.amazon.lambda.S3EventTestCase.testS3 - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

📦 integration-tests/amazon-lambda-stream-handler

✖ io.quarkus.it.amazon.lambda.AmazonLambdaSimpleTestCase.testSimpleLambdaSuccess - More details - Source on GitHub

io.quarkus.amazon.lambda.test.LambdaException
	at io.quarkus.amazon.lambda.test.LambdaResourceManager$3.handleRequest(LambdaResourceManager.java:92)
	at io.undertow.server.RoutingHandler.handleRequest(RoutingHandler.java:92)

quarkus-bot · 2022-01-20T02:25:37Z

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building `e6f8d08`

Status	Name	Step	Failures	Logs	Raw logs
✖	Initial JDK 11 Build	`Build`	Failures	Logs	Raw logs

Failures

⚙️ Initial JDK 11 Build #

- Failing: extensions/amazon-lambda/common-runtime 
! Skipped: devtools/bom-descriptor-json docs extensions/amazon-lambda-http/deployment and 22 more

📦 extensions/amazon-lambda/common-runtime

✖ Failed to execute goal net.revelc.code.formatter:formatter-maven-plugin:2.17.1:validate (default) on project quarkus-amazon-lambda-common: File '/home/runner/work/quarkus/quarkus/extensions/amazon-lambda/common-runtime/src/main/java/io/quarkus/amazon/lambda/runtime/AbstractLambdaPollLoop.java' has not been previously formatted. Please format file and commit before running validation!

null check fix formatting

paulrobinson · 2022-01-24T12:19:16Z

Requested in https://issues.redhat.com/browse/QUARKUS-1722, so proposing a backport to 2.7.

radcortez · 2022-01-24T13:03:39Z

Requested in https://issues.redhat.com/browse/QUARKUS-1722, so proposing a backport to 2.7.

We are still waiting for confirmation that this fix actually solves the problem reported in #22595. Once we get confirmation, it should be fairly straightforward to backport.

patriot1burke · 2022-02-05T14:34:08Z

@gsmet Needs to be backported to 2.7 branch and released with 2.7.1 (Customer issue).

gsmet · 2022-02-05T14:37:23Z

@patriot1burke unless I'm mistaken, the current status is that I already backported it to 2.7.0.Final.

gsmet · 2022-02-05T14:38:37Z

Yes, it's there: #23151 .

quarkus-bot bot added the area/amazon-lambda label Jan 20, 2022

patriot1burke requested a review from geoand January 20, 2022 01:06

radcortez approved these changes Jan 20, 2022

View reviewed changes

patriot1burke force-pushed the 01192022 branch from 7dad206 to e6f8d08 Compare January 20, 2022 02:10

Fix xray to work with rest/http

c34afc7

null check fix formatting

patriot1burke force-pushed the 01192022 branch from e6f8d08 to c34afc7 Compare January 20, 2022 02:29

patriot1burke merged commit da646cf into quarkusio:main Jan 20, 2022

quarkus-bot bot added this to the 2.8 - main milestone Jan 20, 2022

patriot1burke deleted the 01192022 branch January 20, 2022 02:33

paulrobinson added the triage/backport? label Jan 24, 2022

radcortez mentioned this pull request Jan 24, 2022

Quarkus doesn’t send X-Ray traces to AWS for sub-segments in Native build while using RESTEasy “controller” based approach #22595

Closed

gsmet modified the milestones: 2.8 - main, 2.7.0.Final Jan 24, 2022

gsmet removed the triage/backport? label Jan 24, 2022

patriot1burke added the triage/backport? label Feb 5, 2022

gsmet removed the triage/backport? label Feb 6, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix for xray to work within aws http/rest extensions #23038

Fix for xray to work within aws http/rest extensions #23038

patriot1burke commented Jan 20, 2022

quarkus-bot bot commented Jan 20, 2022 •

edited

Failing Jobs - Building `7dad206`

Failures

⚙️ JVM Tests - JDK 11 #

📦 integration-tests/amazon-lambda-http-resteasy

📦 integration-tests/amazon-lambda-s3event

📦 integration-tests/amazon-lambda-stream-handler

⚙️ JVM Tests - JDK 11 Windows #

📦 integration-tests/amazon-lambda-http-resteasy

📦 integration-tests/amazon-lambda-s3event

📦 integration-tests/amazon-lambda-stream-handler

⚙️ JVM Tests - JDK 17 #

📦 integration-tests/amazon-lambda-http-resteasy

📦 integration-tests/amazon-lambda-s3event

📦 integration-tests/amazon-lambda-stream-handler

quarkus-bot bot commented Jan 20, 2022 •

edited

Failing Jobs - Building `e6f8d08`

Failures

⚙️ Initial JDK 11 Build #

📦 extensions/amazon-lambda/common-runtime

paulrobinson commented Jan 24, 2022

radcortez commented Jan 24, 2022

patriot1burke commented Feb 5, 2022

gsmet commented Feb 5, 2022

gsmet commented Feb 5, 2022

Fix for xray to work within aws http/rest extensions #23038

Fix for xray to work within aws http/rest extensions #23038

Conversation

patriot1burke commented Jan 20, 2022

quarkus-bot bot commented Jan 20, 2022 • edited

Failing Jobs - Building 7dad206

Failures

⚙️ JVM Tests - JDK 11 #

📦 integration-tests/amazon-lambda-http-resteasy

📦 integration-tests/amazon-lambda-s3event

📦 integration-tests/amazon-lambda-stream-handler

⚙️ JVM Tests - JDK 11 Windows #

📦 integration-tests/amazon-lambda-http-resteasy

📦 integration-tests/amazon-lambda-s3event

📦 integration-tests/amazon-lambda-stream-handler

⚙️ JVM Tests - JDK 17 #

📦 integration-tests/amazon-lambda-http-resteasy

📦 integration-tests/amazon-lambda-s3event

📦 integration-tests/amazon-lambda-stream-handler

quarkus-bot bot commented Jan 20, 2022 • edited

Failing Jobs - Building e6f8d08

Failures

⚙️ Initial JDK 11 Build #

📦 extensions/amazon-lambda/common-runtime

paulrobinson commented Jan 24, 2022

radcortez commented Jan 24, 2022

patriot1burke commented Feb 5, 2022

gsmet commented Feb 5, 2022

gsmet commented Feb 5, 2022

quarkus-bot bot commented Jan 20, 2022 •

edited

Failing Jobs - Building `7dad206`

quarkus-bot bot commented Jan 20, 2022 •

edited

Failing Jobs - Building `e6f8d08`