Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WFCORE-6249] Eliminating deprecated sun.jdk module usages #5445

Merged
merged 2 commits into from Apr 17, 2023

Conversation

ropalka
Copy link
Contributor

@ropalka ropalka commented Apr 6, 2023

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Apr 6, 2023
@wildfly wildfly deleted a comment from wildfly-ci Apr 14, 2023
@wildfly wildfly deleted a comment from wildfly-ci Apr 14, 2023
@wildfly wildfly deleted a comment from wildfly-ci Apr 14, 2023
@wildfly wildfly deleted a comment from wildfly-ci Apr 14, 2023
@wildfly wildfly deleted a comment from wildfly-ci Apr 14, 2023
@wildfly wildfly deleted a comment from wildfly-ci Apr 14, 2023
@wildfly wildfly deleted a comment from wildfly-ci Apr 14, 2023
@wildfly wildfly deleted a comment from wildfly-ci Apr 14, 2023
@wildfly wildfly deleted a comment from wildfly-ci Apr 14, 2023
@wildfly-ci
Copy link

Core -> Full Integration Build 12212 outcome was FAILURE using a merge of a3a14e8
Summary: Tests failed: 1 (1 new), passed: 4013, ignored: 61 Build time: 02:55:59

Failed tests

org.wildfly.test.integration.observability.opentelemetry.ContextPropagationTestCase(layers-test).testContextPropagation: java.lang.AssertionError: Failed to get expected spans. Got:
Recording traceparent
/ContextPropagationTestCase/contextProp2
HTTP GET
Making second request
/ContextPropagationTestCase/contextProp1
java.lang.AssertionError: 
Failed to get expected spans. Got:
Recording traceparent
/ContextPropagationTestCase/contextProp2
HTTP GET
Making second request
/ContextPropagationTestCase/contextProp1
	at deployment.ContextPropagationTestCase.war//org.wildfly.test.integration.observability.opentelemetry.exporter.InMemorySpanExporter.assertSpanCount(InMemorySpanExporter.java:61)
	at deployment.ContextPropagationTestCase.war//org.wildfly.test.integration.observability.opentelemetry.exporter.InMemorySpanExporter.getFinishedSpanItems(InMemorySpanExporter.java:50)
	at deployment.ContextPropagationTestCase.war//org.wildfly.test.integration.observability.opentelemetry.exporter.InMemorySpanExporter$Proxy$_$$_WeldClientProxy.getFinishedSpanItems(Unknown Source)
	at deployment.ContextPropagationTestCase.war//org.wildfly.test.integration.observability.opentelemetry.ContextPropagationTestCase.testContextPropagation(ContextPropagationTestCase.java:68)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at deployment.arquillian-service//org.jboss.as.arquillian.service.ArquillianService$ExtendedJMXTestRunner.doRunTestMethod(ArquillianService.java:202)
	at deployment.arquillian-service//org.jboss.as.arquillian.service.ArquillianService$ExtendedJMXTestRunner.runTestMethod(ArquillianService.java:178)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)


@wildfly-ci
Copy link

Core -> Full Integration Build 12213 outcome was FAILURE using a merge of a3a14e8
Summary: Tests failed: 1, passed: 4013, ignored: 61 Build time: 02:03:35

Failed tests

org.wildfly.test.integration.observability.opentelemetry.ContextPropagationTestCase(layers-test).testContextPropagation: java.lang.AssertionError: Failed to get expected spans. Got:
Recording traceparent
/ContextPropagationTestCase/contextProp2
HTTP GET
Making second request
/ContextPropagationTestCase/contextProp1
java.lang.AssertionError: 
Failed to get expected spans. Got:
Recording traceparent
/ContextPropagationTestCase/contextProp2
HTTP GET
Making second request
/ContextPropagationTestCase/contextProp1
	at deployment.ContextPropagationTestCase.war//org.wildfly.test.integration.observability.opentelemetry.exporter.InMemorySpanExporter.assertSpanCount(InMemorySpanExporter.java:61)
	at deployment.ContextPropagationTestCase.war//org.wildfly.test.integration.observability.opentelemetry.exporter.InMemorySpanExporter.getFinishedSpanItems(InMemorySpanExporter.java:50)
	at deployment.ContextPropagationTestCase.war//org.wildfly.test.integration.observability.opentelemetry.exporter.InMemorySpanExporter$Proxy$_$$_WeldClientProxy.getFinishedSpanItems(Unknown Source)
	at deployment.ContextPropagationTestCase.war//org.wildfly.test.integration.observability.opentelemetry.ContextPropagationTestCase.testContextPropagation(ContextPropagationTestCase.java:68)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at deployment.arquillian-service//org.jboss.as.arquillian.service.ArquillianService$ExtendedJMXTestRunner.doRunTestMethod(ArquillianService.java:202)
	at deployment.arquillian-service//org.jboss.as.arquillian.service.ArquillianService$ExtendedJMXTestRunner.runTestMethod(ArquillianService.java:178)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)


@ropalka
Copy link
Contributor Author

ropalka commented Apr 17, 2023

Hello @yersan this PR is awaiting your approval.

@yersan
Copy link
Collaborator

yersan commented Apr 17, 2023

Hi @ropalka , yes, I'm keeping an eye on it, we need to go with full confidence at this stage and "
Galleon Full integration - Linux" failed twice with your latest changes, apparently it is unrelated, but better to go with all in gree for now

@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Apr 17, 2023
@yersan yersan merged commit e26bec0 into wildfly:main Apr 17, 2023
12 checks passed
@yersan
Copy link
Collaborator

yersan commented Apr 17, 2023

Thanks @ropalka

@ropalka ropalka deleted the WFCORE-6249 branch April 18, 2023 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
4 participants