build(deps): bump io.cryostat:cryostat-core from 2.28.0 to 2.29.0 (#1880

)

* build(deps): bump io.cryostat:cryostat-core from 2.28.0 to 2.29.0

Bumps [io.cryostat:cryostat-core](https://github.com/cryostatio/cryostat-core) from 2.28.0 to 2.29.0.
- [Release notes](https://github.com/cryostatio/cryostat-core/releases)
- [Commits](cryostatio/cryostat-core@v2.28.0...v2.29.0)

---
updated-dependencies:
- dependency-name: io.cryostat:cryostat-core
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* fixup! build(deps): bump io.cryostat:cryostat-core from 2.28.0 to 2.29.0

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Andrew Azores <aazores@redhat.com>

pom.xml

@@ -78,7 +78,7 @@
   <com.google.dagger.version>2.47</com.google.dagger.version>
   <com.google.dagger.compiler.version>${com.google.dagger.version}</com.google.dagger.compiler.version>
 
-  <io.cryostat.core.version>2.28.0</io.cryostat.core.version>
+  <io.cryostat.core.version>2.29.0</io.cryostat.core.version>
 
   <org.openjdk.nashorn.core.version>15.4</org.openjdk.nashorn.core.version>
   <org.apache.commons.lang3.version>3.13.0</org.apache.commons.lang3.version>

src/main/java/io/cryostat/ApplicationVersion.java

@@ -19,17 +19,14 @@
 import java.io.InputStreamReader;
 import java.nio.charset.StandardCharsets;
 
-import io.cryostat.core.log.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class ApplicationVersion {
 
     private static final String RESOURCE_LOCATION = "/io/cryostat/version";
     private volatile String version;
-    private final Logger logger;
-
-    ApplicationVersion(Logger logger) {
-        this.logger = logger;
-    }
+    private final Logger logger = LoggerFactory.getLogger(getClass());
 
     public synchronized String getVersionString() {
         if (version == null) {
@@ -49,7 +46,7 @@ public synchronized String getVersionString() {
                                                                 RESOURCE_LOCATION)))
                                 .trim();
             } catch (Exception e) {
-                logger.error(e);
+                logger.error("Version retrieval exception", e);
                 version = "unknown";
             }
         }

src/main/java/io/cryostat/Cryostat.java

@@ -24,7 +24,6 @@
 
 import io.cryostat.configuration.CredentialsManager;
 import io.cryostat.core.CryostatCore;
-import io.cryostat.core.log.Logger;
 import io.cryostat.core.sys.Environment;
 import io.cryostat.discovery.DiscoveryStorage;
 import io.cryostat.messaging.MessagingServer;
@@ -39,12 +38,14 @@
 import io.vertx.core.AbstractVerticle;
 import io.vertx.core.Promise;
 import io.vertx.core.Vertx;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 class Cryostat extends AbstractVerticle {
 
     private final Environment environment = new Environment();
     private final Client client;
-    private final Logger logger = Logger.INSTANCE;
+    private final Logger logger = LoggerFactory.getLogger(getClass());
 
     private Cryostat(Client client) {
         this.client = client;
@@ -58,7 +59,7 @@ public void start(Promise<Void> future) {
             client.credentialsManager().migrate();
             client.ruleRegistry().loadRules();
         } catch (Exception e) {
-            logger.error(e);
+            logger.error("Startup exception", e);
             future.fail(e);
             return;
         }
@@ -113,7 +114,7 @@ private void shutdown(Throwable cause) {
             if (!(cause instanceof Exception)) {
                 cause = new RuntimeException(cause);
             }
-            logger.error((Exception) cause);
+            logger.error("Shutdown exception", (Exception) cause);
         }
         logger.info("{} shutting down...", instanceName());
         client.vertx().close().onComplete(n -> logger.info("Shutdown complete"));

src/main/java/io/cryostat/MainModule.java

@@ -31,7 +31,6 @@
 import io.cryostat.configuration.ConfigurationModule;
 import io.cryostat.configuration.Variables;
 import io.cryostat.core.agent.ProbeTemplate;
-import io.cryostat.core.log.Logger;
 import io.cryostat.core.sys.Environment;
 import io.cryostat.core.tui.ClientWriter;
 import io.cryostat.discovery.DiscoveryModule;
@@ -61,6 +60,8 @@
 import dagger.Provides;
 import io.vertx.core.Vertx;
 import org.apache.commons.codec.binary.Base32;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 @Module(
         includes = {
@@ -81,16 +82,12 @@ public abstract class MainModule {
     public static final String CONF_DIR = "CONF_DIR";
     public static final String UUID_FROM_STRING = "UUID_FROM_STRING";
 
-    @Provides
-    @Singleton
-    static ApplicationVersion provideApplicationVersion(Logger logger) {
-        return new ApplicationVersion(logger);
-    }
+    private static final Logger logger = LoggerFactory.getLogger(MainModule.class);
 
     @Provides
     @Singleton
-    static Logger provideLogger() {
-        return Logger.INSTANCE;
+    static ApplicationVersion provideApplicationVersion() {
+        return new ApplicationVersion();
     }
 
     @Provides
@@ -102,7 +99,8 @@ static Base32 provideBase32() {
     @Singleton
     // FIXME remove this outdated ClientWriter abstraction and simply replace with an injected
     // Logger at all ClientWriter injection sites
-    static ClientWriter provideClientWriter(Logger logger) {
+    static ClientWriter provideClientWriter() {
+        Logger logger = LoggerFactory.getLogger(ClientWriter.class);
         return new ClientWriter() {
             @Override
             public void print(String s) {
@@ -111,29 +109,27 @@ public void print(String s) {
 
             @Override
             public void println(Exception e) {
-                logger.warn(e);
+                logger.warn("Write exception", e);
             }
         };
     }
 
     // testing-only when extra adapters aren't needed
-    public static Gson provideGson(Logger logger) {
-        return provideGson(Set.of(), Set.of(), logger);
+    public static Gson provideGson() {
+        return provideGson(Set.of(), Set.of());
     }
 
     // public since this is useful to use directly in tests
     @Provides
     @Singleton
     public static Gson provideGson(
             Set<PluggableTypeAdapter<?>> extraAdapters,
-            Set<PluggableJsonDeserializer<?>> deserializers,
-            Logger logger) {
+            Set<PluggableJsonDeserializer<?>> deserializers) {
         GsonBuilder builder =
                 new GsonBuilder()
                         .serializeNulls()
                         .disableHtmlEscaping()
-                        .registerTypeAdapter(
-                                JMXServiceURL.class, new GsonJmxServiceUrlAdapter(logger))
+                        .registerTypeAdapter(JMXServiceURL.class, new GsonJmxServiceUrlAdapter())
                         .registerTypeAdapter(HttpMimeType.class, new HttpMimeTypeAdapter())
                         .registerTypeHierarchyAdapter(Path.class, new PathTypeAdapter())
                         .registerTypeAdapter(Rule.class, new RuleDeserializer())
@@ -151,7 +147,7 @@ JMXServiceURL.class, new GsonJmxServiceUrlAdapter(logger))
     @Provides
     @Singleton
     @Named(RECORDINGS_PATH)
-    static Path provideSavedRecordingsPath(Logger logger, Environment env) {
+    static Path provideSavedRecordingsPath(Environment env) {
         String archivePath = env.getEnv(Variables.ARCHIVE_PATH, "/flightrecordings");
         logger.info("Local save path for flight recordings set as {}", archivePath);
         return Paths.get(archivePath);
@@ -180,7 +176,7 @@ public static int provideVertxPoolSize(Environment env) {
     @Provides
     @Singleton
     public static VerticleDeployer provideVerticleDeployer(
-            Vertx vertx, @Named(Variables.VERTX_POOL_SIZE) int poolSize, Logger logger) {
-        return new VerticleDeployer(vertx, poolSize, logger);
+            Vertx vertx, @Named(Variables.VERTX_POOL_SIZE) int poolSize) {
+        return new VerticleDeployer(vertx, poolSize);
     }
 }

src/main/java/io/cryostat/VerticleDeployer.java

@@ -15,27 +15,26 @@
  */
 package io.cryostat;
 
-import io.cryostat.core.log.Logger;
-
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import io.vertx.core.DeploymentOptions;
 import io.vertx.core.Future;
 import io.vertx.core.Verticle;
 import io.vertx.core.Vertx;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class VerticleDeployer {
 
     private final Vertx vertx;
     private final int poolSize;
-    private final Logger logger;
+    private final Logger logger = LoggerFactory.getLogger(getClass());
 
     @SuppressFBWarnings(
             value = "EI_EXPOSE_REP2",
             justification = "vertx is externally mutable and that's fine")
-    public VerticleDeployer(Vertx vertx, int poolSize, Logger logger) {
+    public VerticleDeployer(Vertx vertx, int poolSize) {
         this.vertx = vertx;
         this.poolSize = poolSize;
-        this.logger = logger;
     }
 
     public Future deploy(Verticle verticle, boolean worker) {

src/main/java/io/cryostat/configuration/ConfigurationModule.java

@@ -22,7 +22,6 @@
 import javax.inject.Singleton;
 import javax.persistence.EntityManager;
 
-import io.cryostat.core.log.Logger;
 import io.cryostat.core.sys.Environment;
 import io.cryostat.core.sys.FileSystem;
 import io.cryostat.discovery.DiscoveryStorage;
@@ -33,16 +32,19 @@
 import dagger.Lazy;
 import dagger.Module;
 import dagger.Provides;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 @Module
 public abstract class ConfigurationModule {
     public static final String CONFIGURATION_PATH = "CONFIGURATION_PATH";
     public static final String CREDENTIALS_SUBDIRECTORY = "credentials";
+    private static final Logger logger = LoggerFactory.getLogger(ConfigurationModule.class);
 
     @Provides
     @Singleton
     @Named(CONFIGURATION_PATH)
-    static Path provideConfigurationPath(Logger logger, Environment env) {
+    static Path provideConfigurationPath(Environment env) {
         String path = env.getEnv(Variables.CONFIG_PATH, "/opt/cryostat.d/conf.d");
         logger.info(String.format("Local config path set as %s", path));
         return Paths.get(path);
@@ -57,8 +59,7 @@ static CredentialsManager provideCredentialsManager(
             DiscoveryStorage discovery,
             StoredCredentialsDao dao,
             FileSystem fs,
-            Gson gson,
-            Logger logger) {
+            Gson gson) {
         Path credentialsDir = confDir.resolve(CREDENTIALS_SUBDIRECTORY);
         return new CredentialsManager(
                 credentialsDir,
@@ -67,13 +68,12 @@ static CredentialsManager provideCredentialsManager(
                 discovery,
                 dao,
                 fs,
-                gson,
-                logger);
+                gson);
     }
 
     @Provides
     @Singleton
-    static StoredCredentialsDao provideStoredCredentialsDao(EntityManager em, Logger logger) {
-        return new StoredCredentialsDao(em, logger);
+    static StoredCredentialsDao provideStoredCredentialsDao(EntityManager em) {
+        return new StoredCredentialsDao(em);
     }
 }

src/main/java/io/cryostat/configuration/CredentialsManager.java

@@ -32,7 +32,6 @@
 
 import javax.script.ScriptException;
 
-import io.cryostat.core.log.Logger;
 import io.cryostat.core.net.Credentials;
 import io.cryostat.core.sys.FileSystem;
 import io.cryostat.platform.PlatformClient;
@@ -49,6 +48,8 @@
 import dagger.Lazy;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.http.client.utils.URIBuilder;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class CredentialsManager
         extends AbstractEventEmitter<CredentialsManager.CredentialsEvent, String> {
@@ -60,7 +61,7 @@ public class CredentialsManager
     private final StoredCredentialsDao dao;
     private final FileSystem fs;
     private final Gson gson;
-    private final Logger logger;
+    private final Logger logger = LoggerFactory.getLogger(getClass());
 
     CredentialsManager(
             Path credentialsDir,
@@ -69,16 +70,14 @@ public class CredentialsManager
             PlatformClient platformClient,
             StoredCredentialsDao dao,
             FileSystem fs,
-            Gson gson,
-            Logger logger) {
+            Gson gson) {
         this.credentialsDir = credentialsDir;
         this.matchExpressionValidator = matchExpressionValidator;
         this.matchExpressionEvaluator = matchExpressionEvaluator;
         this.platformClient = platformClient;
         this.dao = dao;
         this.fs = fs;
         this.gson = gson;
-        this.logger = logger;
     }
 
     // TODO remove after 2.2 release
@@ -126,7 +125,7 @@ public void migrate() throws Exception {
                     }
                 }
             } catch (IOException | IllegalStateException e) {
-                logger.warn(e);
+                logger.warn("Migration exception", e);
                 continue;
             }
         }
@@ -231,7 +230,7 @@ public Set<ServiceRef> resolveMatchingTargets(String matchExpression) {
                     matchedTargets.add(target);
                 }
             } catch (ScriptException e) {
-                logger.error(e);
+                logger.error("Script execution exception", e);
                 break;
             }
         }

src/main/java/io/cryostat/configuration/StoredCredentialsDao.java

@@ -17,11 +17,10 @@
 
 import javax.persistence.EntityManager;
 
-import io.cryostat.core.log.Logger;
 import io.cryostat.storage.AbstractDao;
 
 class StoredCredentialsDao extends AbstractDao<Integer, StoredCredentials> {
-    StoredCredentialsDao(EntityManager em, Logger logger) {
-        super(StoredCredentials.class, em, logger);
+    StoredCredentialsDao(EntityManager em) {
+        super(StoredCredentials.class, em);
     }
 }

src/main/java/io/cryostat/discovery/AbstractNodeTypeAdapter.java

@@ -25,7 +25,6 @@
 import java.util.Map;
 import java.util.Set;
 
-import io.cryostat.core.log.Logger;
 import io.cryostat.platform.ServiceRef;
 import io.cryostat.platform.ServiceRef.AnnotationKey;
 import io.cryostat.platform.discovery.AbstractNode;
@@ -39,17 +38,18 @@
 import com.google.gson.stream.JsonToken;
 import com.google.gson.stream.JsonWriter;
 import dagger.Lazy;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class AbstractNodeTypeAdapter extends PluggableTypeAdapter<AbstractNode> {
 
     private final Lazy<Set<PluggableTypeAdapter<?>>> adapters;
-    private final Logger logger;
+    private final Logger logger = LoggerFactory.getLogger(getClass());
 
     public AbstractNodeTypeAdapter(
-            Class<AbstractNode> klazz, Lazy<Set<PluggableTypeAdapter<?>>> adapters, Logger logger) {
+            Class<AbstractNode> klazz, Lazy<Set<PluggableTypeAdapter<?>>> adapters) {
         super(klazz);
         this.adapters = adapters;
-        this.logger = logger;
     }
 
     @Override