Fix warning at startup regarding JVmStateSection

Two same instances of JvmStateSection are defined in pico. It generates
the warning log:
"WARN  ce[][o.s.c.p.ComponentKeys] Bad component key: org.sonar.process.systeminfo.JvmStateSection@13078cb2. Please implement toString() method on class org.sonar.process.systeminfo.JvmStateSection"
This commit is contained in:
Simon Brandhof 2017-10-16 12:01:26 +02:00
parent 2bde31ecbf
commit 67092c5570
3 changed files with 9 additions and 11 deletions

View File

@ -21,9 +21,11 @@ package org.sonar.ce;
import org.sonar.ce.configuration.CeConfigurationImpl;
import org.sonar.ce.log.CeLogging;
import org.sonar.core.platform.Module;
import org.sonar.process.systeminfo.JvmStateSection;
import org.sonar.ce.monitoring.CeDatabaseMBeanImpl;
import org.sonar.core.platform.Module;
import org.sonar.process.systeminfo.JvmPropertiesSection;
import org.sonar.process.systeminfo.JvmStateSection;
import org.sonar.server.platform.monitoring.cluster.LoggingSection;
public class CeConfigurationModule extends Module {
@Override
@ -32,6 +34,8 @@ public class CeConfigurationModule extends Module {
CeConfigurationImpl.class,
CeLogging.class,
CeDatabaseMBeanImpl.class,
new JvmStateSection("Compute Engine JVM State"));
new JvmStateSection("Compute Engine JVM State"),
new JvmPropertiesSection("Compute Engine JVM Properties"),
LoggingSection.class);
}
}

View File

@ -24,9 +24,6 @@ import org.sonar.ce.monitoring.CeTasksMBeanImpl;
import org.sonar.ce.queue.CeQueueInitializer;
import org.sonar.ce.queue.InternalCeQueueImpl;
import org.sonar.core.platform.Module;
import org.sonar.process.systeminfo.JvmPropertiesSection;
import org.sonar.process.systeminfo.JvmStateSection;
import org.sonar.server.platform.monitoring.cluster.LoggingSection;
public class CeQueueModule extends Module {
@Override
@ -38,9 +35,6 @@ public class CeQueueModule extends Module {
// queue monitoring
CEQueueStatusImpl.class,
CeTasksMBeanImpl.class,
new JvmStateSection("Compute Engine JVM State"),
new JvmPropertiesSection("Compute Engine JVM Properties"),
LoggingSection.class,
// init queue state and queue processing
CeQueueInitializer.class);

View File

@ -130,8 +130,8 @@ public class ComputeEngineContainerImplTest {
.hasSize(
CONTAINER_ITSELF
+ 75 // level 4
+ 4 // content of CeConfigurationModule
+ 7 // content of CeQueueModule
+ 6 // content of CeConfigurationModule
+ 4 // content of CeQueueModule
+ 4 // content of CeHttpModule
+ 3 // content of CeTaskCommonsModule
+ 4 // content of ProjectAnalysisTaskModule