Browse Source

docs: drop suppressing-instrumentation page and script (#5694)

Patrice Chalin 3 years ago
parent
commit
e908b7fc71

+ 2 - 1
README.md

@@ -104,7 +104,7 @@ We support an impressively huge number of [libraries and frameworks](docs/suppor
 a majority of the most popular [application servers](docs/supported-libraries.md#application-servers)...right out of the box!
 [Click here to see the full list](docs/supported-libraries.md) and to learn more about
 [disabled instrumentation](docs/supported-libraries.md#disabled-instrumentations)
-and how to [suppress unwanted instrumentation](docs/suppressing-instrumentation.md).
+and how to [suppress unwanted instrumentation][suppress].
 
 ## Creating agent extensions
 
@@ -165,3 +165,4 @@ Thanks to all the people who already contributed!
 
 [config]: https://opentelemetry.io/docs/instrumentation/java/automatic/agent-config/
 [manual]: https://opentelemetry.io/docs/instrumentation/java/manual/
+[suppress]: https://opentelemetry.io/docs/instrumentation/java/automatic/agent-config/#suppressing-specific-auto-instrumentation

+ 0 - 22
buildscripts/check-suppressing-instrumentation-docs.sh

@@ -1,22 +0,0 @@
-#!/usr/bin/env bash
-
-# this script reports some false positives, but is helpful in comparing the keys listed
-# in suppressing-instrumentation.md with the keys listed in the source files
-
-set -e -u -o pipefail
-
-comm -3 \
-  <(
-    sed -n '/----------------------/,${p;/^$/q}' \
-        docs/suppressing-instrumentation.md \
-      | sed '1d;$d' \
-      | cut -d '|' -f 3 \
-      | tr -d ' ' \
-      | sort -u
-  ) \
-  <(
-    git ls-files '*Module.java' \
-      | xargs grep super \
-      | grep -oP '(?<=super\(")[^"]+' \
-      | sort -u
-  )

+ 3 - 1
docs/contributing/writing-instrumentation-module.md

@@ -29,7 +29,7 @@ class MyLibraryInstrumentationModule extends InstrumentationModule {
 ```
 
 An `InstrumentationModule` needs to have at least one name. The user of the javaagent can
-[suppress a chosen instrumentation](../suppressing-instrumentation.md) by referring to it by one of
+[suppress a chosen instrumentation][suppress] by referring to it by one of
 its names. The instrumentation module names use `kebab-case`. The main instrumentation name, which
 is the first one, must be the same as the gradle module name, excluding the version suffix if present.
 
@@ -355,3 +355,5 @@ we prefer to use
 because the former inserts a call to `Class.getMethod(...)` in transformed method. In contrast,
 getting the declaring class and method name is just loading constants from constant pool, which is
 a much simpler operation.
+
+[suppress]: https://opentelemetry.io/docs/instrumentation/java/automatic/agent-config/#suppressing-specific-auto-instrumentation