Merge PR #376: Update cloudfoundry-resources module to follow style guide

Author: trask

cloudfoundry-resources/build.gradle.kts

@@ -8,6 +8,9 @@ description = "OpenTelemetry CloudFoundry Resources"
 otelJava.moduleName.set("io.opentelemetry.contrib.cloudfoundry.resources")
 
 dependencies {
+  annotationProcessor("com.google.auto.service:auto-service")
+  compileOnly("com.google.auto.service:auto-service")
+
   api("io.opentelemetry:opentelemetry-api")
   compileOnly("io.opentelemetry:opentelemetry-api-incubator")
   api("io.opentelemetry:opentelemetry-sdk")

cloudfoundry-resources/src/main/java/io/opentelemetry/contrib/cloudfoundry/resources/CloudFoundryResource.java

@@ -5,6 +5,8 @@
 
 package io.opentelemetry.contrib.cloudfoundry.resources;
 
+import static io.opentelemetry.api.common.AttributeKey.stringKey;
+
 import com.fasterxml.jackson.core.JsonFactory;
 import com.fasterxml.jackson.core.JsonParser;
 import com.fasterxml.jackson.core.JsonToken;
@@ -22,24 +24,22 @@ public final class CloudFoundryResource {
   private static final String ENV_VCAP_APPLICATION = "VCAP_APPLICATION";
 
   // copied from CloudfoundryIncubatingAttributes
-  private static final AttributeKey<String> CLOUDFOUNDRY_APP_ID =
-      AttributeKey.stringKey("cloudfoundry.app.id");
+  private static final AttributeKey<String> CLOUDFOUNDRY_APP_ID = stringKey("cloudfoundry.app.id");
   private static final AttributeKey<String> CLOUDFOUNDRY_APP_INSTANCE_ID =
-      AttributeKey.stringKey("cloudfoundry.app.instance.id");
+      stringKey("cloudfoundry.app.instance.id");
   private static final AttributeKey<String> CLOUDFOUNDRY_APP_NAME =
-      AttributeKey.stringKey("cloudfoundry.app.name");
-  private static final AttributeKey<String> CLOUDFOUNDRY_ORG_ID =
-      AttributeKey.stringKey("cloudfoundry.org.id");
+      stringKey("cloudfoundry.app.name");
+  private static final AttributeKey<String> CLOUDFOUNDRY_ORG_ID = stringKey("cloudfoundry.org.id");
   private static final AttributeKey<String> CLOUDFOUNDRY_ORG_NAME =
-      AttributeKey.stringKey("cloudfoundry.org.name");
+      stringKey("cloudfoundry.org.name");
   private static final AttributeKey<String> CLOUDFOUNDRY_PROCESS_ID =
-      AttributeKey.stringKey("cloudfoundry.process.id");
+      stringKey("cloudfoundry.process.id");
   private static final AttributeKey<String> CLOUDFOUNDRY_PROCESS_TYPE =
-      AttributeKey.stringKey("cloudfoundry.process.type");
+      stringKey("cloudfoundry.process.type");
   private static final AttributeKey<String> CLOUDFOUNDRY_SPACE_ID =
-      AttributeKey.stringKey("cloudfoundry.space.id");
+      stringKey("cloudfoundry.space.id");
   private static final AttributeKey<String> CLOUDFOUNDRY_SPACE_NAME =
-      AttributeKey.stringKey("cloudfoundry.space.name");
+      stringKey("cloudfoundry.space.name");
   private static final Logger LOG = Logger.getLogger(CloudFoundryResource.class.getName());
   private static final JsonFactory JSON_FACTORY = new JsonFactory();
   private static final Resource INSTANCE = buildResource(System::getenv);

cloudfoundry-resources/src/main/java/io/opentelemetry/contrib/cloudfoundry/resources/CloudFoundryResourceDetector.java

@@ -5,11 +5,14 @@
 
 package io.opentelemetry.contrib.cloudfoundry.resources;
 
+import com.google.auto.service.AutoService;
 import io.opentelemetry.api.incubator.config.DeclarativeConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.internal.ComponentProvider;
 import io.opentelemetry.sdk.resources.Resource;
 
-public class CloudFoundryResourceDetector implements ComponentProvider<Resource> {
+@SuppressWarnings("rawtypes")
+@AutoService(ComponentProvider.class)
+public final class CloudFoundryResourceDetector implements ComponentProvider<Resource> {
 
   @Override
   public Class<Resource> getType() {

cloudfoundry-resources/src/main/java/io/opentelemetry/contrib/cloudfoundry/resources/CloudFoundryResourceProvider.java

@@ -5,11 +5,13 @@
 
 package io.opentelemetry.contrib.cloudfoundry.resources;
 
+import com.google.auto.service.AutoService;
 import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.ResourceProvider;
 import io.opentelemetry.sdk.resources.Resource;
 
-public class CloudFoundryResourceProvider implements ResourceProvider {
+@AutoService(ResourceProvider.class)
+public final class CloudFoundryResourceProvider implements ResourceProvider {
 
   @Override
   public Resource createResource(ConfigProperties configProperties) {

cloudfoundry-resources/src/main/resources/META-INF/services/io.opentelemetry.sdk.autoconfigure.spi.ResourceProvider

@@ -5,7 +5,9 @@
 
 package io.opentelemetry.contrib.cloudfoundry.resources;
 
+import static java.util.stream.Collectors.joining;
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.fail;
 
 import io.opentelemetry.sdk.resources.Resource;
 import io.opentelemetry.semconv.SchemaUrls;
@@ -18,8 +20,6 @@
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.Map;
-import java.util.stream.Collectors;
-import org.assertj.core.api.Assertions;
 import org.junit.jupiter.api.Test;
 
 class CloudFoundryResourceTest {
@@ -36,11 +36,11 @@ private static String loadVcapApplicationSample(String filename) {
       if (is != null) {
         return new BufferedReader(new InputStreamReader(is, StandardCharsets.UTF_8))
             .lines()
-            .collect(Collectors.joining());
+            .collect(joining());
       }
-      Assertions.fail("Cannot load resource " + filename);
+      fail("Cannot load resource " + filename);
     } catch (IOException e) {
-      Assertions.fail("Error reading " + filename);
+      fail("Error reading " + filename);
     }
     return "";
   }