move dependency downloads to build task

Author: daniel-mohedano

dd-smoke-tests/maven/build.gradle

@@ -20,13 +20,67 @@ jar {
   }
 }
 
+// Pre-cache Maven dependencies for smoke tests projects during build phase
+tasks.register('mvnBuild') {
+  description = 'Pre-cache Maven dependencies for all test projects'
+
+  inputs.files fileTree("$projectDir/src/test/resources") {
+    include "**/pom.xml"
+    include "**/settings.xml"
+  }
+  outputs.dir "$projectDir/.gradle/maven-dependencies-cache"
+
+  doLast {
+    file("$projectDir/.gradle/maven-dependencies-cache").mkdirs()
+
+    def pomFiles = fileTree("$projectDir/src/test/resources") {
+      include "**/pom.xml"
+    }
+
+    def processedProjects = [] as Set
+
+    pomFiles.each { pomFile ->
+      def projectDir = pomFile.parentFile
+      def projectKey = projectDir.name
+
+      if (!processedProjects.contains(projectKey)) {
+        processedProjects.add(projectKey)
+
+        logger.info("Pre-caching dependencies for Maven project: ${projectKey}")
+
+        def processBuilder = new ProcessBuilder()
+        processBuilder.directory(projectDir)
+        processBuilder.command("$rootDir/mvnw", "dependency:go-offline", "-B")
+
+        // Handle proxy configuration if available
+        if (project.hasProperty("mavenRepositoryProxy")) {
+          processBuilder.environment().put("MAVEN_REPOSITORY_PROXY", project.property("mavenRepositoryProxy"))
+        }
+
+        def process = processBuilder.start()
+        def exitCode = process.waitFor()
+
+        if (exitCode != 0) {
+          logger.warn("Failed to cache dependencies for project ${projectKey}, exit code: ${exitCode}")
+        } else {
+          logger.info("Successfully cached dependencies for project: ${projectKey}")
+        }
+      }
+    }
+  }
+}
+
+tasks.named("compileTestGroovy").configure {
+  dependsOn 'mvnBuild'
+}
+
 tasks.withType(Test).configureEach {
   dependsOn "shadowJar"
   jvmArgs "-Ddatadog.smoketest.maven.jar.path=${tasks.shadowJar.archiveFile.get()}"
 
   // overriding the default timeout set in configure_tests.gradle, as Maven smoke
   // tests might run for a longer duration
-  timeout = Duration.of(20, ChronoUnit.MINUTES)
+  timeout = Duration.of(15, ChronoUnit.MINUTES)
 
   if (project.hasProperty("mavenRepositoryProxy")) {
     // propagate proxy URL to tests, to then propagate it to nested Gradle builds

dd-smoke-tests/maven/src/test/groovy/datadog/smoketest/MavenSmokeTest.groovy

@@ -270,17 +270,10 @@ class MavenSmokeTest extends CiVisibilitySmokeTest {
   }
 
   /**
-   * Sometimes Maven has problems downloading project dependencies because of intermittent network issues.
-   * Here, in order to reduce flakiness, we ensure that all of the dependencies are loaded (retrying if necessary),
-   * before proceeding with running the build
+   * Dependencies are now pre-cached during the Gradle build phase via the mvnBuild task.
+   * This method still handles runtime-specific dependencies that are added by the tracer.
    */
   private void givenMavenDependenciesAreLoaded(String projectName, String mavenVersion, Map<String, String> additionalEnvVars = [:]) {
-    if (LOADED_DEPENDENCIES.add("$projectName:$mavenVersion")) {
-      retryUntilSuccessfulOrNoAttemptsLeft(["dependency:go-offline"], additionalEnvVars)
-    }
-    // dependencies below are download separately
-    // because they are not declared in the project,
-    // but are added at runtime by the tracer
     if (LOADED_DEPENDENCIES.add("com.datadoghq:dd-javac-plugin:$JAVAC_PLUGIN_VERSION")) {
       retryUntilSuccessfulOrNoAttemptsLeft(["dependency:get", "-Dartifact=com.datadoghq:dd-javac-plugin:$JAVAC_PLUGIN_VERSION".toString()], additionalEnvVars)
     }