open-telemetry
/
opentelemetry-java-instrumentation
mirror of https://github.com/open-telemetry/opentelemetry-java-instrumentation.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Remove onInstrumentationError from LagomTest 

We not longer need it since our instrumentation can handle underlying
loading problem.
This commit is contained in:
Nikolay Martynov 2018-07-30 16:23:07 -04:00
parent 6bc1d1ab8e
commit 6693a93485
1 changed files with 0 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
dd-java-agent/instrumentation/akka-http-10.0/src/lagomTest/groovy/LagomTest.groovy
View File

@ -6,7 +6,6 @@ import datadog.trace.agent.test.AgentTestRunner
import datadog.trace.api.DDSpanTypes
import datadog.trace.api.DDTags
import io.opentracing.tag.Tags
import net.bytebuddy.utility.JavaModule
import play.inject.guice.GuiceApplicationBuilder
import spock.lang.Shared
@ -25,20 +24,6 @@ class LagomTest extends AgentTestRunner {
@Shared
private TestServer server
@Override
protected boolean onInstrumentationError(String typeName, ClassLoader classLoader, JavaModule module, boolean loaded, Throwable throwable) {
if (throwable.getMessage().contains('Cannot resolve type description for akka.stream.impl.VirtualProcessor$WrappedSubscription$$SubscriptionState')) {
// 'akka/stream/impl/VirtualProcessor$WrappedSubscription$PassThrough$.class' declares
// itself an implementation of 'VirtualProcessor$WrappedSubscription$$SubscriptionState',
// but this interface does not exist on the classpath.
// The closest thing on the classpath is 'VirtualProcessor$WrappedSubscription$SubscriptionState' (only one $).
// Looks like a compiler/packaging issue on akka's end. Or maybe this interface is dynamically generated.
return false
}
return super.onInstrumentationError(typeName, classLoader, module, loaded, throwable)
}
def setupSpec() {
server = startServer(defaultSetup()
.withCluster(false)