Improves telemetry bean exception handling (#488)

* Improves telemetry bean exception handling * Updates log lines for lint length and clarity
DataDog · Oct 19, 2023 · 43c6494 · 43c6494
1 parent 195c704
commit 43c6494
Showing 1 changed file with 17 additions and 8 deletions.
diff --git a/src/main/java/org/datadog/jmxfetch/Instance.java b/src/main/java/org/datadog/jmxfetch/Instance.java
@@ -279,29 +279,38 @@ public Instance(
             log.info("collect_default_jvm_metrics is false - not collecting default JVM metrics");
         }
 
-        instanceTelemetryBean = createJmxBean();
+        instanceTelemetryBean = createInstanceTelemetryBean();
     }
 
     private ObjectName getObjName(String domain,String instance)
             throws MalformedObjectNameException {
         return new ObjectName(domain + ":target_instance=" + ObjectName.quote(instance));
     }
 
-    private InstanceTelemetry createJmxBean() {
+    private InstanceTelemetry createInstanceTelemetryBean() {
         mbs =  ManagementFactory.getPlatformMBeanServer();
         InstanceTelemetry bean = new InstanceTelemetry();
         log.debug("Created jmx bean for instance: " + this.getCheckName());
 
         try {
             instanceTelemetryBeanName = getObjName(appConfig.getJmxfetchTelemetryDomain(),
                      this.getName());
-            mbs.registerMBean(bean,instanceTelemetryBeanName);
-            log.debug("Succesfully registered jmx bean for instance: " + this.getCheckName()
-                    + " with ObjectName = " + instanceTelemetryBeanName);
+        } catch (MalformedObjectNameException e) {
+            log.warn(
+                "Could not construct bean name for jmxfetch_telemetry_domain '{}' and name '{}'",
+                appConfig.getJmxfetchTelemetryDomain(), this.getName());
+            return bean;
+        }
 
-        } catch (MalformedObjectNameException | InstanceAlreadyExistsException
-                | MBeanRegistrationException | NotCompliantMBeanException e) {
-            log.warn("Could not register bean for instance: " + this.getCheckName(),e);
+        try {
+            mbs.registerMBean(bean,instanceTelemetryBeanName);
+            log.debug("Succesfully registered jmx bean for instance {} with ObjectName = {}",
+                this.getName(), instanceTelemetryBeanName);
+        } catch (InstanceAlreadyExistsException
+         | MBeanRegistrationException
+         | NotCompliantMBeanException e) {
+            log.warn("Could not register bean named '{}' for instance: ",
+                instanceTelemetryBeanName.toString(), e);
         }
 
         return bean;