add links

Author: jaydeluca

docs/instrumentation-list.yaml

@@ -1651,6 +1651,7 @@ libraries:
   - name: c3p0-0.9
     description: The c3p0 instrumentation provides connection pool metrics for c3p0
       data sources.
+    library_link: https://github.com/swaldman/c3p0
     source_path: instrumentation/c3p0-0.9
     scope:
       name: io.opentelemetry.c3p0-0.9
@@ -2971,6 +2972,7 @@ libraries:
   - name: google-http-client-1.19
     description: This instrumentation enables HTTP CLIENT spans and metrics for Google
       HTTP Client requests.
+    library_link: https://github.com/googleapis/google-http-java-client
     source_path: instrumentation/google-http-client-1.19
     scope:
       name: io.opentelemetry.google-http-client-1.19
@@ -3059,6 +3061,7 @@ libraries:
   graphql:
   - name: graphql-java-12.0
     description: This instrumentation enables spans for GraphQL Java operations.
+    library_link: https://www.graphql-java.com/
     source_path: instrumentation/graphql-java/graphql-java-12.0
     scope:
       name: io.opentelemetry.graphql-java-12.0

instrumentation-docs/src/main/java/io/opentelemetry/instrumentation/docs/internal/InstrumentationMetaData.java

@@ -25,6 +25,10 @@ public class InstrumentationMetaData {
 
   private String classification;
 
+  @JsonProperty("library_link")
+  @Nullable
+  private String libraryLink;
+
   private List<ConfigurationOption> configurations = Collections.emptyList();
 
   public InstrumentationMetaData() {
@@ -35,10 +39,12 @@ public InstrumentationMetaData(
       @Nullable String description,
       String classification,
       @Nullable Boolean disabledByDefault,
+      @Nullable String libraryLink,
       @Nullable List<ConfigurationOption> configurations) {
     this.classification = classification;
     this.disabledByDefault = disabledByDefault;
     this.description = description;
+    this.libraryLink = libraryLink;
     this.configurations = Objects.requireNonNullElse(configurations, Collections.emptyList());
   }
 
@@ -77,4 +83,13 @@ public List<ConfigurationOption> getConfigurations() {
   public void setConfigurations(@Nullable List<ConfigurationOption> configurations) {
     this.configurations = Objects.requireNonNullElse(configurations, Collections.emptyList());
   }
+
+  @Nullable
+  public String getLibraryLink() {
+    return libraryLink;
+  }
+
+  public void setLibraryLink(@Nullable String libraryLink) {
+    this.libraryLink = libraryLink;
+  }
 }

instrumentation-docs/src/main/java/io/opentelemetry/instrumentation/docs/utils/YamlHelper.java

@@ -198,6 +198,9 @@ private static void addMetadataProperties(
       if (module.getMetadata().getDescription() != null) {
         moduleMap.put("description", module.getMetadata().getDescription());
       }
+      if (module.getMetadata().getLibraryLink() != null) {
+        moduleMap.put("library_link", module.getMetadata().getLibraryLink());
+      }
       if (module.getMetadata().getDisabledByDefault()) {
         moduleMap.put("disabled_by_default", module.getMetadata().getDisabledByDefault());
       }

instrumentation-docs/src/test/java/io/opentelemetry/instrumentation/docs/utils/YamlHelperTest.java

@@ -43,6 +43,7 @@ void testPrintInstrumentationList() throws Exception {
             "Spring Web 6.0 instrumentation",
             InstrumentationClassification.LIBRARY.toString(),
             true,
+            null,
             null);
 
     modules.add(
@@ -114,6 +115,7 @@ void testGenerateInstrumentationYamlSeparatesClassifications() throws Exception
             "Spring Web 6.0 instrumentation",
             InstrumentationClassification.LIBRARY.toString(),
             false,
+            null,
             List.of(
                 new ConfigurationOption(
                     "otel.instrumentation.spring-web-6.0.enabled",
@@ -134,7 +136,7 @@ void testGenerateInstrumentationYamlSeparatesClassifications() throws Exception
 
     InstrumentationMetaData internalMetadata =
         new InstrumentationMetaData(
-            null, InstrumentationClassification.INTERNAL.toString(), null, null);
+            null, InstrumentationClassification.INTERNAL.toString(), null, null, null);
 
     modules.add(
         new InstrumentationModule.Builder()
@@ -148,7 +150,7 @@ void testGenerateInstrumentationYamlSeparatesClassifications() throws Exception
 
     InstrumentationMetaData customMetadata =
         new InstrumentationMetaData(
-            null, InstrumentationClassification.CUSTOM.toString(), null, null);
+            null, InstrumentationClassification.CUSTOM.toString(), null, null, null);
 
     Map<InstrumentationType, Set<String>> externalAnnotationsVersions =
         Map.of(
@@ -214,6 +216,7 @@ void testMetadataParser() throws JsonProcessingException {
             description: test description
             classification: internal
             disabled_by_default: true
+            library_link: https://example.com/test-library
             configurations:
               - name: otel.instrumentation.common.db-statement-sanitizer.enabled
                 description: Enables statement sanitization for database queries.
@@ -233,6 +236,7 @@ void testMetadataParser() throws JsonProcessingException {
     assertThat(metadata.getClassification()).isEqualTo(InstrumentationClassification.INTERNAL);
     assertThat(metadata.getDescription()).isEqualTo("test description");
     assertThat(metadata.getDisabledByDefault()).isEqualTo(true);
+    assertThat(metadata.getLibraryLink()).isEqualTo("https://example.com/test-library");
   }
 
   @Test
@@ -241,6 +245,18 @@ void testMetadataParserWithOnlyLibraryEntry() throws JsonProcessingException {
     InstrumentationMetaData metadata = YamlHelper.metaDataParser(input);
     assertThat(metadata.getClassification()).isEqualTo(InstrumentationClassification.INTERNAL);
     assertThat(metadata.getDescription()).isNull();
+    assertThat(metadata.getLibraryLink()).isNull();
+    assertThat(metadata.getDisabledByDefault()).isFalse();
+    assertThat(metadata.getConfigurations()).isEmpty();
+  }
+
+  @Test
+  void testMetadataParserWithOnlyLibraryLink() throws JsonProcessingException {
+    String input = "library_link: https://example.com/only-link";
+    InstrumentationMetaData metadata = YamlHelper.metaDataParser(input);
+    assertThat(metadata.getClassification()).isEqualTo(InstrumentationClassification.LIBRARY);
+    assertThat(metadata.getDescription()).isNull();
+    assertThat(metadata.getLibraryLink()).isEqualTo("https://example.com/only-link");
     assertThat(metadata.getDisabledByDefault()).isFalse();
     assertThat(metadata.getConfigurations()).isEmpty();
   }
@@ -485,4 +501,80 @@ void testTelemetryGroupsAreSorted() throws Exception {
 
     assertThat(yaml1).isEqualTo(yaml2);
   }
+
+  @Test
+  void testYamlGenerationWithLibraryLink() throws Exception {
+    List<InstrumentationModule> modules = new ArrayList<>();
+    Map<InstrumentationType, Set<String>> targetVersions = new HashMap<>();
+    targetVersions.put(
+        InstrumentationType.JAVAAGENT, new HashSet<>(List.of("com.example:test-library:[1.0.0,)")));
+
+    InstrumentationMetaData metadataWithLink =
+        new InstrumentationMetaData(
+            "Test library instrumentation with link",
+            InstrumentationClassification.LIBRARY.toString(),
+            false,
+            "https://example.com/test-library-docs",
+            emptyList());
+
+    modules.add(
+        new InstrumentationModule.Builder()
+            .srcPath("instrumentation/test-lib/test-lib-1.0")
+            .instrumentationName("test-lib-1.0")
+            .namespace("test-lib")
+            .group("test-lib")
+            .targetVersions(targetVersions)
+            .metadata(metadataWithLink)
+            .build());
+
+    InstrumentationMetaData metadataWithoutLink =
+        new InstrumentationMetaData(
+            "Test library instrumentation without link",
+            InstrumentationClassification.LIBRARY.toString(),
+            false,
+            null,
+            emptyList());
+
+    modules.add(
+        new InstrumentationModule.Builder()
+            .srcPath("instrumentation/other-lib/other-lib-1.0")
+            .instrumentationName("other-lib-1.0")
+            .namespace("other-lib")
+            .group("other-lib")
+            .targetVersions(targetVersions)
+            .metadata(metadataWithoutLink)
+            .build());
+
+    StringWriter stringWriter = new StringWriter();
+    BufferedWriter writer = new BufferedWriter(stringWriter);
+
+    YamlHelper.generateInstrumentationYaml(modules, writer);
+    writer.flush();
+
+    String expectedYaml =
+        """
+            libraries:
+              other-lib:
+              - name: other-lib-1.0
+                description: Test library instrumentation without link
+                source_path: instrumentation/other-lib/other-lib-1.0
+                scope:
+                  name: io.opentelemetry.other-lib-1.0
+                target_versions:
+                  javaagent:
+                  - com.example:test-library:[1.0.0,)
+              test-lib:
+              - name: test-lib-1.0
+                description: Test library instrumentation with link
+                library_link: https://example.com/test-library-docs
+                source_path: instrumentation/test-lib/test-lib-1.0
+                scope:
+                  name: io.opentelemetry.test-lib-1.0
+                target_versions:
+                  javaagent:
+                  - com.example:test-library:[1.0.0,)
+            """;
+
+    assertThat(expectedYaml).isEqualTo(stringWriter.toString());
+  }
 }

instrumentation/activej-http-6.0/metadata.yaml

@@ -1 +1,2 @@
 description: This instrumentation enables SERVER spans and metrics for the ActiveJ HTTP server.
+library_link: https://activej.io/

instrumentation/akka/akka-actor-2.3/metadata.yaml

@@ -1 +1,2 @@
 description: This instrumentation provides context propagation for Akka actors, it does not emit any telemetry on its own.
+library_link: https://doc.akka.io/docs/akka/current/typed/index.html

instrumentation/akka/akka-actor-fork-join-2.5/metadata.yaml

@@ -1 +1,2 @@
 description: This instrumentation provides context propagation for the Akka Fork-Join Pool, it does not emit any telemetry on its own.
+library_link: https://doc.akka.io/docs/akka/current/typed/index.html

instrumentation/akka/akka-http-10.0/metadata.yaml

@@ -1 +1,2 @@
 description: This instrumentation enables CLIENT and SERVER spans and metrics for the Akka HTTP client and server.
+library_link: https://doc.akka.io/docs/akka-http/current/index.html

instrumentation/alibaba-druid-1.0/metadata.yaml

@@ -1,3 +1,4 @@
 description: >
   The Alibaba Druid instrumentation generates database connection pool metrics for druid data
   sources.
+library_link: https://github.com/alibaba/druid

instrumentation/c3p0-0.9/metadata.yaml

@@ -1 +1,2 @@
 description: The c3p0 instrumentation provides connection pool metrics for c3p0 data sources.
+library_link: https://github.com/swaldman/c3p0