Add more metrics and tags to CI Visibility telemetry (#7223)

Author: nikita-tkachenko-datadog

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/CiVisibilityRepoServices.java

@@ -4,6 +4,7 @@
 import datadog.trace.api.Config;
 import datadog.trace.api.civisibility.config.ModuleExecutionSettings;
 import datadog.trace.api.civisibility.telemetry.CiVisibilityMetricCollector;
+import datadog.trace.api.civisibility.telemetry.tag.Provider;
 import datadog.trace.api.git.GitInfoProvider;
 import datadog.trace.civisibility.ci.CIInfo;
 import datadog.trace.civisibility.ci.CIProviderInfo;
@@ -45,8 +46,8 @@ public class CiVisibilityRepoServices {
 
   final String repoRoot;
   final String moduleName;
+  final Provider ciProvider;
   final Map<String, String> ciTags;
-  final boolean supportedCiProvider;
 
   final GitDataUploader gitDataUploader;
   final RepoIndexProvider repoIndexProvider;
@@ -56,11 +57,12 @@ public class CiVisibilityRepoServices {
 
   CiVisibilityRepoServices(CiVisibilityServices services, Path path) {
     CIProviderInfo ciProviderInfo = services.ciProviderInfoFactory.createCIProviderInfo(path);
+    ciProvider = ciProviderInfo.getProvider();
+
     CIInfo ciInfo = ciProviderInfo.buildCIInfo();
     repoRoot = ciInfo.getCiWorkspace();
     moduleName = getModuleName(services.config, path, ciInfo);
     ciTags = new CITagsProvider().getCiTags(ciInfo);
-    supportedCiProvider = ciProviderInfo.isSupportedCiProvider();
 
     gitDataUploader =
         buildGitDataUploader(

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/CiVisibilitySystem.java

@@ -210,7 +210,7 @@ private static BuildSystemSession.Factory buildSystemSessionFactory(
           repoServices.repoRoot,
           startCommand,
           startTime,
-          repoServices.supportedCiProvider,
+          repoServices.ciProvider,
           services.config,
           services.metricCollector,
           testModuleRegistry,
@@ -262,7 +262,7 @@ private static TestFrameworkSession.Factory headlessTestFrameworkEssionFactory(
       return new HeadlessTestSession(
           projectName,
           startTime,
-          repoServices.supportedCiProvider,
+          repoServices.ciProvider,
           services.config,
           services.metricCollector,
           testDecorator,
@@ -282,7 +282,7 @@ private static CIVisibility.SessionFactory manualApiSessionFactory(
       return new ManualApiTestSession(
           projectName,
           startTime,
-          repoServices.supportedCiProvider,
+          repoServices.ciProvider,
           services.config,
           services.metricCollector,
           testDecorator,

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/AppVeyorInfo.java

@@ -4,6 +4,7 @@
 import static datadog.trace.api.git.GitUtils.normalizeTag;
 import static datadog.trace.civisibility.utils.FileUtils.expandTilde;
 
+import datadog.trace.api.civisibility.telemetry.tag.Provider;
 import datadog.trace.api.git.CommitInfo;
 import datadog.trace.api.git.GitInfo;
 import datadog.trace.api.git.PersonInfo;
@@ -112,4 +113,9 @@ private PersonInfo buildGitCommitAuthor() {
         System.getenv(APPVEYOR_REPO_COMMIT_AUTHOR_NAME),
         System.getenv(APPVEYOR_REPO_COMMIT_AUTHOR_EMAIL));
   }
+
+  @Override
+  public Provider getProvider() {
+    return Provider.APPVEYOR;
+  }
 }

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/AwsCodePipelineInfo.java

@@ -1,5 +1,6 @@
 package datadog.trace.civisibility.ci;
 
+import datadog.trace.api.civisibility.telemetry.tag.Provider;
 import datadog.trace.api.git.GitInfo;
 
 class AwsCodePipelineInfo implements CIProviderInfo {
@@ -26,4 +27,9 @@ public CIInfo buildCIInfo() {
             AWS_CODEPIPELINE_ARN)
         .build();
   }
+
+  @Override
+  public Provider getProvider() {
+    return Provider.AWS;
+  }
 }

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/AzurePipelinesInfo.java

@@ -6,6 +6,7 @@
 import static datadog.trace.api.git.GitUtils.normalizeTag;
 import static datadog.trace.civisibility.utils.FileUtils.expandTilde;
 
+import datadog.trace.api.civisibility.telemetry.tag.Provider;
 import datadog.trace.api.git.CommitInfo;
 import datadog.trace.api.git.GitInfo;
 import datadog.trace.api.git.PersonInfo;
@@ -134,4 +135,9 @@ private PersonInfo buildGitCommitAuthor() {
         System.getenv(AZURE_BUILD_REQUESTED_FOR_ID),
         System.getenv(AZURE_BUILD_REQUESTED_FOR_EMAIL));
   }
+
+  @Override
+  public Provider getProvider() {
+    return Provider.AZP;
+  }
 }

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BitBucketInfo.java

@@ -5,6 +5,7 @@
 import static datadog.trace.api.git.GitUtils.normalizeTag;
 import static datadog.trace.civisibility.utils.FileUtils.expandTilde;
 
+import datadog.trace.api.civisibility.telemetry.tag.Provider;
 import datadog.trace.api.git.CommitInfo;
 import datadog.trace.api.git.GitInfo;
 import datadog.trace.util.Strings;
@@ -75,4 +76,9 @@ private String buildPipelineId() {
     }
     return id;
   }
+
+  @Override
+  public Provider getProvider() {
+    return Provider.BITBUCKET;
+  }
 }

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BitriseInfo.java

@@ -5,6 +5,7 @@
 import static datadog.trace.api.git.GitUtils.normalizeTag;
 import static datadog.trace.civisibility.utils.FileUtils.expandTilde;
 
+import datadog.trace.api.civisibility.telemetry.tag.Provider;
 import datadog.trace.api.git.CommitInfo;
 import datadog.trace.api.git.GitInfo;
 import datadog.trace.api.git.PersonInfo;
@@ -65,4 +66,9 @@ private String buildGitCommit() {
       return System.getenv(BITRISE_GIT_COMMIT);
     }
   }
+
+  @Override
+  public Provider getProvider() {
+    return Provider.BITRISE;
+  }
 }

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BuddyInfo.java

@@ -4,6 +4,7 @@
 import static datadog.trace.api.git.GitUtils.normalizeBranch;
 import static datadog.trace.api.git.GitUtils.normalizeTag;
 
+import datadog.trace.api.civisibility.telemetry.tag.Provider;
 import datadog.trace.api.git.CommitInfo;
 import datadog.trace.api.git.GitInfo;
 import datadog.trace.api.git.PersonInfo;
@@ -65,4 +66,9 @@ private PersonInfo buildGitCommiter() {
     return new PersonInfo(
         System.getenv(BUDDY_GIT_COMMIT_AUTHOR), System.getenv(BUDDY_GIT_COMMIT_EMAIL));
   }
+
+  @Override
+  public Provider getProvider() {
+    return Provider.BUDDYCI;
+  }
 }

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BuildkiteInfo.java

@@ -6,6 +6,7 @@
 import static datadog.trace.civisibility.utils.FileUtils.expandTilde;
 import static datadog.trace.util.Strings.toJson;
 
+import datadog.trace.api.civisibility.telemetry.tag.Provider;
 import datadog.trace.api.git.CommitInfo;
 import datadog.trace.api.git.GitInfo;
 import datadog.trace.api.git.PersonInfo;
@@ -84,4 +85,9 @@ private PersonInfo buildGitCommitAuthor() {
     return new PersonInfo(
         System.getenv(BUILDKITE_GIT_AUTHOR_NAME), System.getenv(BUILDKITE_GIT_AUTHOR_EMAIL));
   }
+
+  @Override
+  public Provider getProvider() {
+    return Provider.BUILDKITE;
+  }
 }

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/CIProviderInfo.java

@@ -1,5 +1,6 @@
 package datadog.trace.civisibility.ci;
 
+import datadog.trace.api.civisibility.telemetry.tag.Provider;
 import datadog.trace.api.git.GitInfo;
 
 public interface CIProviderInfo {
@@ -8,7 +9,5 @@ public interface CIProviderInfo {
 
   CIInfo buildCIInfo();
 
-  default boolean isSupportedCiProvider() {
-    return true;
-  }
+  Provider getProvider();
 }