Tries to hone OSGi imports according to use (#1252)

It is my understanding that unless we shade something, we have to add internal import for the few utilities we use. Fixes #1248

Author: adriancole

brave-bom/pom.xml

@@ -35,7 +35,7 @@
 
  <!-- use the same values in ../pom.xml -->
  <zipkin.version>2.21.7</zipkin.version>
- <zipkin-reporter.version>2.15.1</zipkin-reporter.version>
+ <zipkin-reporter.version>2.15.2</zipkin-reporter.version>
  </properties>
 
  <organization>

context/jfr/bnd.bnd

@@ -1,4 +1,4 @@
-# We use brave.internal.Nullable, but nothing else internal.
+# We use brave.internal.Nullable, but it is not used at runtime.
 Import-Package: \
  !brave.internal*,\
  *

context/log4j12/bnd.bnd

@@ -1,6 +1,7 @@
-# We use brave.internal.Nullable and brave.internal.CorrelationContext
+# We use need to import to support brave.internal.CorrelationContext
+# brave.internal.Nullable is not used at runtime.
 Import-Package: \
- !brave.internal*,\
+ brave.internal;braveinternal=true,\
  *
 Export-Package: \
  brave.context.log4j12

context/log4j2/bnd.bnd

@@ -1,6 +1,7 @@
-# We use brave.internal.Nullable and brave.internal.CorrelationContext
+# We use need to import to support brave.internal.CorrelationContext
+# brave.internal.Nullable is not used at runtime.
 Import-Package: \
- !brave.internal*,\
+ brave.internal;braveinternal=true,\
  *
 Export-Package: \
  brave.context.log4j2

context/slf4j/bnd.bnd

@@ -1,6 +1,7 @@
-# We use brave.internal.Nullable and brave.internal.CorrelationContext
+# We use need to import to support brave.internal.CorrelationContext
+# brave.internal.Nullable is not used at runtime.
 Import-Package: \
- !brave.internal*,\
+ brave.internal;braveinternal=true,\
  *
 Export-Package: \
  brave.context.slf4j

instrumentation/dubbo-rpc/bnd.bnd

@@ -1,6 +1,7 @@
-# We use brave.internal.Nullable and brave.internal.Platform
+# We use need to import to support brave.internal.Platform
+# brave.internal.Nullable is not used at runtime.
 Import-Package: \
- !brave.internal*,\
+ brave.internal;braveinternal=true,\
  *
 Export-Package: \
  brave.dubbo.rpc

instrumentation/dubbo/bnd.bnd

@@ -1,6 +1,7 @@
-# We use brave.internal.Nullable and brave.internal.Platform
+# We use need to import to support brave.internal.Platform
+# brave.internal.Nullable is not used at runtime.
 Import-Package: \
- !brave.internal*,\
+ brave.internal;braveinternal=true,\
  *
 Export-Package: \
  brave.dubbo

instrumentation/grpc/bnd.bnd

@@ -1,6 +1,7 @@
-# We use brave.internal.Nullable,Platform,MapPropagationFields,PropagationFieldsFactory
+# We use need to import to support brave.internal.Platform,MapPropagationFields,PropagationFieldsFactory
+# brave.internal.Nullable is not used at runtime.
 Import-Package: \
- !brave.internal*,\
+ brave.internal;braveinternal=true,\
  *
 Export-Package: \
  brave.grpc

instrumentation/http/bnd.bnd

@@ -1,4 +1,4 @@
-# We use brave.internal.Nullable, but nothing else internal.
+# We use brave.internal.Nullable, but it is not used at runtime.
 Import-Package: \
  !brave.internal*,\
  *

instrumentation/httpasyncclient/bnd.bnd

@@ -1,4 +1,4 @@
-# We use brave.internal.Nullable, but nothing else internal.
+# We use brave.internal.Nullable, but it is not used at runtime.
 Import-Package: \
  !brave.internal*,\
  *