Update to Groovy 4

conventions/src/main/kotlin/otel.java-conventions.gradle.kts

@@ -75,8 +75,12 @@ tasks.withType<JavaCompile>().configureEach {
 // Groovy and Scala compilers don't actually understand --release option
 afterEvaluate {
   tasks.withType<GroovyCompile>().configureEach {
-    sourceCompatibility = otelJava.minJavaVersionSupported.get().majorVersion
-    targetCompatibility = otelJava.minJavaVersionSupported.get().majorVersion
+    var javaVersion = otelJava.minJavaVersionSupported.get().majorVersion
+    if (javaVersion == "8") {
+      javaVersion = "1.8"
+    }
+    sourceCompatibility = javaVersion
+    targetCompatibility = javaVersion
   }
   tasks.withType<ScalaCompile>().configureEach {
     sourceCompatibility = otelJava.minJavaVersionSupported.get().majorVersion

dependencyManagement/build.gradle.kts

@@ -14,8 +14,8 @@ rootProject.extra["versions"] = dependencyVersions
 val otelVersion = "1.12.0"
 rootProject.extra["otelVersion"] = otelVersion
 
-// Need both BOM and -all
-val groovyVersion = "3.0.9"
+// Need both BOM and groovy jars
+val groovyVersion = "4.0.1"
 
 // We don't force libraries we instrument to new versions since we compile and test against specific
 // old baseline versions
@@ -32,7 +32,7 @@ val groovyVersion = "3.0.9"
 val DEPENDENCY_BOMS = listOf(
   "com.fasterxml.jackson:jackson-bom:2.13.1",
   "com.google.guava:guava-bom:31.0.1-jre",
-  "org.codehaus.groovy:groovy-bom:${groovyVersion}",
+  "org.apache.groovy:groovy-bom:${groovyVersion}",
   "io.opentelemetry:opentelemetry-bom:${otelVersion}",
   "io.opentelemetry:opentelemetry-bom-alpha:${otelVersion}-alpha",
   "org.junit:junit-bom:5.8.2"
@@ -109,11 +109,12 @@ val DEPENDENCIES = listOf(
   "org.awaitility:awaitility:4.1.1",
   "com.google.code.findbugs:annotations:3.0.1u2",
   "com.google.code.findbugs:jsr305:3.0.2",
-  "org.codehaus.groovy:groovy-all:${groovyVersion}",
+  "org.apache.groovy:groovy:${groovyVersion}",
+  "org.apache.groovy:groovy-json:${groovyVersion}",
   "org.junit-pioneer:junit-pioneer:1.5.0",
   "org.objenesis:objenesis:3.2",
-  "org.spockframework:spock-core:2.0-groovy-3.0",
-  "org.spockframework:spock-junit4:2.0-groovy-3.0",
+  "org.spockframework:spock-core:2.2-M1-groovy-4.0",
+  "org.spockframework:spock-junit4:2.2-M1-groovy-4.0",
   "org.scala-lang:scala-library:2.11.12",
   "org.springframework.boot:spring-boot-dependencies:2.3.1.RELEASE",
 )

instrumentation/apache-dubbo-2.7/testing/build.gradle.kts

@@ -13,7 +13,7 @@ dependencies {
   implementation("javax.annotation:javax.annotation-api:1.3.2")
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/apache-httpclient/apache-httpclient-4.3/testing/build.gradle.kts

@@ -7,7 +7,7 @@ dependencies {
 
   api("org.apache.httpcomponents:httpclient:4.3")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/armeria-1.3/testing/build.gradle.kts

@@ -10,7 +10,7 @@ dependencies {
 
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/aws-lambda/aws-lambda-core-1.0/testing/build.gradle.kts

@@ -12,7 +12,7 @@ dependencies {
 
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
   implementation("com.github.stefanbirkner:system-lambda")

instrumentation/aws-lambda/aws-lambda-events-2.2/testing/build.gradle.kts

@@ -13,7 +13,7 @@ dependencies {
 
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
   implementation("com.github.stefanbirkner:system-lambda")

instrumentation/aws-sdk/aws-sdk-1.11/testing/build.gradle.kts

@@ -20,7 +20,7 @@ dependencies {
 
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/aws-sdk/aws-sdk-2.2/testing/build.gradle.kts

@@ -19,7 +19,7 @@ dependencies {
 
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/couchbase/couchbase-2-common/javaagent-unit-tests/build.gradle.kts

@@ -3,7 +3,7 @@ plugins {
 }
 
 dependencies {
-  testImplementation("org.codehaus.groovy:groovy-all")
+  testImplementation("org.apache.groovy:groovy")
   testImplementation("org.spockframework:spock-core")
 
   testImplementation(project(":instrumentation-api"))

instrumentation/couchbase/couchbase-common/testing/build.gradle.kts

@@ -10,7 +10,7 @@ dependencies {
   api("com.couchbase.client:java-client:2.5.0")
   api("org.springframework.data:spring-data-couchbase:2.0.0.RELEASE")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/grails-3.0/javaagent/build.gradle.kts

@@ -44,3 +44,26 @@ dependencies {
   testLibrary("org.springframework.boot:spring-boot-autoconfigure:$springBootVersion")
   testLibrary("org.springframework.boot:spring-boot-starter-tomcat:$springBootVersion")
 }
+
+// exclude groovy 4 from testing-common
+configurations.configureEach {
+  exclude("org.apache.groovy", "groovy")
+  exclude("org.apache.groovy", "groovy-json")
+}
+
+configurations.configureEach {
+  if (!name.contains("muzzle")) {
+    resolutionStrategy {
+      eachDependency {
+        // groovy compilation fails when spock is removed
+        // just set spock and groovy to compatible versions
+        if (requested.group == "org.spockframework") {
+          useVersion("2.1-groovy-3.0")
+        }
+        if (requested.group == "org.codehaus.groovy") {
+          useVersion("3.0.9")
+        }
+      }
+    }
+  }
+}

instrumentation/jaxrs/jaxrs-2.0/jaxrs-2.0-common/testing/build.gradle.kts

@@ -6,7 +6,7 @@ dependencies {
   api(project(":testing-common"))
   api("javax.ws.rs:javax.ws.rs-api:2.0")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
   implementation("org.slf4j:slf4j-api")

instrumentation/jdbc/testing/build.gradle.kts

@@ -8,5 +8,5 @@ plugins {
 }
 
 dependencies {
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
 }

instrumentation/jetty-httpclient/jetty-httpclient-9.2/testing/build.gradle.kts

@@ -12,7 +12,7 @@ dependencies {
 
   implementation("org.junit.jupiter:junit-jupiter-api")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/lettuce/lettuce-5.1/testing/build.gradle.kts

@@ -10,7 +10,7 @@ dependencies {
   implementation("org.testcontainers:testcontainers")
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/log4j/log4j-context-data/log4j-context-data-2-common/testing/build.gradle.kts

@@ -9,7 +9,7 @@ dependencies {
 
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 

instrumentation/logback/logback-mdc-1.0/testing/build.gradle.kts

@@ -11,7 +11,7 @@ dependencies {
 
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/logback/logback-mdc-1.0/testing/src/main/groovy/io/opentelemetry/instrumentation/logback/v1_0/AbstractLogbackTest.groovy

@@ -60,14 +60,14 @@ abstract class AbstractLogbackTest extends InstrumentationSpecification {
   def "ids when span"() {
     when:
     Span span1 = runWithSpan("test") {
-      logger.info("log message 1")
+      AbstractLogbackTest.logger.info("log message 1")
       Span.current()
     }
 
     logger.info("log message 2")
 
     Span span2 = runWithSpan("test 2") {
-      logger.info("log message 3")
+      AbstractLogbackTest.logger.info("log message 3")
       Span.current()
     }
 

instrumentation/mongo/mongo-3.1/testing/build.gradle.kts

@@ -7,7 +7,7 @@ dependencies {
 
   compileOnly("org.mongodb:mongo-java-driver:3.1.0")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/mongo/mongo-common/testing/build.gradle.kts

@@ -8,7 +8,7 @@ dependencies {
   api(project(":testing-common"))
   api("org.testcontainers:mongodb:${versions["org.testcontainers"]}")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/okhttp/okhttp-3.0/testing/build.gradle.kts

@@ -9,7 +9,7 @@ dependencies {
 
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/play-ws/play-ws-common/testing/build.gradle.kts

@@ -8,7 +8,7 @@ dependencies {
   api(project(":testing-common"))
   api("com.typesafe.play:play-ahc-ws-standalone_$scalaVersion:1.0.2")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/ratpack/ratpack-1.4/testing/build.gradle.kts

@@ -10,7 +10,7 @@ dependencies {
   implementation("io.ratpack:ratpack-core:1.4.0")
   implementation("io.ratpack:ratpack-test:1.4.0")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/reactor/reactor-3.1/testing/build.gradle.kts

@@ -7,7 +7,7 @@ dependencies {
 
   api("io.projectreactor:reactor-core:3.1.0.RELEASE")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/restlet/restlet-1.0/testing/build.gradle.kts

@@ -17,7 +17,7 @@ dependencies {
   implementation("org.restlet:org.restlet.ext.spring:1.1.5")
   implementation("org.springframework:spring:2.5.6")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 

instrumentation/restlet/restlet-2.0/testing/build.gradle.kts

@@ -14,7 +14,7 @@ dependencies {
   implementation("org.restlet.jse:org.restlet:2.0.2")
   implementation("org.restlet.jse:org.restlet.ext.spring:2.0.2")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/rocketmq-client-4.8/testing/build.gradle.kts

@@ -7,7 +7,7 @@ dependencies {
   implementation("org.apache.rocketmq:rocketmq-test:4.8.0")
 
   implementation("com.google.guava:guava")
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/rxjava/rxjava-2.0/testing/build.gradle.kts

@@ -9,7 +9,7 @@ dependencies {
 
   implementation("com.google.guava:guava")
 
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/rxjava/rxjava-3-common/testing/build.gradle.kts

@@ -9,7 +9,7 @@ dependencies {
 
   implementation("io.opentelemetry:opentelemetry-extension-annotations")
   implementation("com.google.guava:guava")
-  implementation("org.codehaus.groovy:groovy-all")
+  implementation("org.apache.groovy:groovy")
   implementation("io.opentelemetry:opentelemetry-api")
   implementation("org.spockframework:spock-core")
 }

instrumentation/servlet/servlet-2.2/javaagent/src/test/groovy/HttpServletResponseTest.groovy

@@ -3,9 +3,9 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import groovy.servlet.AbstractHttpServlet
 import io.opentelemetry.api.trace.SpanKind
 import io.opentelemetry.instrumentation.test.AgentInstrumentationSpecification
+import javax.servlet.http.HttpServlet
 import spock.lang.Subject
 
 import javax.servlet.ServletOutputStream
@@ -30,7 +30,7 @@ class HttpServletResponseTest extends AgentInstrumentationSpecification {
   }
 
   def setup() {
-    def servlet = new AbstractHttpServlet() {}
+    def servlet = new HttpServlet() {}
     // We need to call service so HttpServletAdvice can link the request to the response.
     servlet.service((ServletRequest) request, (ServletResponse) response)
     clearExportedData()
@@ -93,7 +93,7 @@ class HttpServletResponseTest extends AgentInstrumentationSpecification {
         throw ex
       }
     }
-    def servlet = new AbstractHttpServlet() {}
+    def servlet = new HttpServlet() {}
     // We need to call service so HttpServletAdvice can link the request to the response.
     servlet.service((ServletRequest) request, (ServletResponse) response)
     clearExportedData()

instrumentation/servlet/servlet-2.2/javaagent/src/test/groovy/TestServlet2.groovy

@@ -3,9 +3,9 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import groovy.servlet.AbstractHttpServlet
 import io.opentelemetry.instrumentation.test.base.HttpServerTest
 import io.opentelemetry.instrumentation.testing.junit.http.ServerEndpoint
+import javax.servlet.http.HttpServlet
 import javax.servlet.http.HttpServletRequest
 import javax.servlet.http.HttpServletResponse
 
@@ -18,7 +18,7 @@ import static io.opentelemetry.instrumentation.testing.junit.http.ServerEndpoint
 
 class TestServlet2 {
 
-  static class Sync extends AbstractHttpServlet {
+  static class Sync extends HttpServlet {
     @Override
     protected void service(HttpServletRequest req, HttpServletResponse resp) {
       req.getRequestDispatcher()