No longer need to disable JBoss Threads logging of version

Author: Sanne

core/deployment/src/main/java/io/quarkus/deployment/dev/DevModeCommandLineBuilder.java

@@ -37,7 +37,6 @@
 import io.quarkus.bootstrap.model.JvmOptionsBuilder;
 import io.quarkus.deployment.util.CommandLineUtil;
 import io.quarkus.maven.dependency.ArtifactKey;
-import io.quarkus.runtime.logging.JBossVersion;
 import io.smallrye.common.process.ProcessUtil;
 
 public class DevModeCommandLineBuilder {
@@ -348,8 +347,6 @@ public DevModeCommandLineBuilder extensionDevModeJvmOptionFilter(
     }
 
     public DevModeCommandLine build() throws Exception {
-        JBossVersion.disableVersionLogging();
-
         //build a class-path string for the base platform
         //this stuff does not change
         // Do not include URIs in the manifest, because some JVMs do not like that

core/runtime/src/main/java/io/quarkus/runtime/Quarkus.java

@@ -15,7 +15,6 @@
 import io.quarkus.bootstrap.logging.InitialConfigurator;
 import io.quarkus.bootstrap.logging.QuarkusDelayedHandler;
 import io.quarkus.launcher.QuarkusLauncher;
-import io.quarkus.runtime.logging.JBossVersion;
 import io.quarkus.runtime.shutdown.ShutdownRecorder;
 
 /**
@@ -67,7 +66,6 @@ public static void run(Class<? extends QuarkusApplication> quarkusApplication, S
     public static void run(Class<? extends QuarkusApplication> quarkusApplication, BiConsumer<Integer, Throwable> exitHandler,
             String... args) {
         try {
-            JBossVersion.disableVersionLogging();
             System.setProperty("java.util.logging.manager", "org.jboss.logmanager.LogManager");
             System.setProperty("java.util.concurrent.ForkJoinPool.common.threadFactory",
                     "io.quarkus.bootstrap.forkjoin.QuarkusForkJoinWorkerThreadFactory");

core/runtime/src/main/java/io/quarkus/runtime/logging/JBossVersion.java

@@ -67,7 +67,6 @@
 import io.quarkus.runner.bootstrap.AugmentActionImpl;
 import io.quarkus.runner.bootstrap.StartupActionImpl;
 import io.quarkus.runtime.LaunchMode;
-import io.quarkus.runtime.logging.JBossVersion;
 import io.quarkus.test.common.GroovyClassValue;
 import io.quarkus.test.common.PathTestHelper;
 import io.quarkus.test.common.PropertyTestUtil;
@@ -90,7 +89,6 @@ public class QuarkusUnitTest
     private Handler[] originalHandlers;
 
     static {
-        JBossVersion.disableVersionLogging();
         System.setProperty("java.util.logging.manager", "org.jboss.logmanager.LogManager");
         rootLogger = (Logger) LogManager.getLogManager().getLogger("");
     }

test-framework/junit5-internal/src/main/java/io/quarkus/test/QuarkusUnitTest.java

@@ -43,7 +43,6 @@
 import io.quarkus.bootstrap.app.CuratedApplication;
 import io.quarkus.bootstrap.logging.InitialConfigurator;
 import io.quarkus.deployment.dev.testing.TestConfig;
-import io.quarkus.runtime.logging.JBossVersion;
 import io.quarkus.runtime.logging.LogRuntimeConfig;
 import io.quarkus.runtime.test.TestHttpEndpointProvider;
 import io.quarkus.test.common.ArtifactLauncher;
@@ -189,7 +188,6 @@ private QuarkusTestExtensionState ensureStarted(ExtensionContext extensionContex
     private QuarkusTestExtensionState doProcessStart(Properties quarkusArtifactProperties,
             Class<? extends QuarkusTestProfile> profile, ExtensionContext context)
             throws Throwable {
-        JBossVersion.disableVersionLogging();
 
         SmallRyeConfig config = ConfigProvider.getConfig().unwrap(SmallRyeConfig.class);
         String artifactType = getEffectiveArtifactType(quarkusArtifactProperties, config);

test-framework/junit5/src/main/java/io/quarkus/test/junit/QuarkusIntegrationTestExtension.java

@@ -31,7 +31,6 @@
 import org.junit.jupiter.api.extension.ParameterResolver;
 
 import io.quarkus.deployment.dev.testing.TestConfig;
-import io.quarkus.runtime.logging.JBossVersion;
 import io.quarkus.test.common.ArtifactLauncher;
 import io.quarkus.test.common.TestResourceManager;
 import io.quarkus.test.junit.launcher.ArtifactLauncherProvider;
@@ -65,8 +64,6 @@ public void beforeEach(ExtensionContext context) throws Exception {
     }
 
     private LaunchResult doLaunch(ExtensionContext context, String[] arguments) throws Exception {
-        JBossVersion.disableVersionLogging();
-
         var result = doProcessStart(context, arguments);
         List<String> out = Arrays.asList(new String(result.getOutput(), StandardCharsets.UTF_8).split("\n"));
         List<String> err = Arrays.asList(new String(result.getStderror(), StandardCharsets.UTF_8).split("\n"));

test-framework/junit5/src/main/java/io/quarkus/test/junit/QuarkusMainIntegrationTestExtension.java

@@ -42,7 +42,6 @@
 import io.quarkus.deployment.dev.testing.LogCapturingOutputFilter;
 import io.quarkus.dev.console.QuarkusConsole;
 import io.quarkus.dev.testing.TracingHandler;
-import io.quarkus.runtime.logging.JBossVersion;
 import io.quarkus.test.common.TestResourceManager;
 import io.quarkus.test.junit.main.Launch;
 import io.quarkus.test.junit.main.LaunchResult;
@@ -74,7 +73,6 @@ public void beforeEach(ExtensionContext context) throws Exception {
 
     private void ensurePrepared(ExtensionContext extensionContext, Class<? extends QuarkusTestProfile> profile)
             throws Exception {
-        JBossVersion.disableVersionLogging();
         QuarkusTestExtensionState state = getState(extensionContext);
         boolean wrongProfile = !Objects.equals(profile, quarkusTestProfile);
         // we reload the test resources if we changed test class and if we had or will have per-test test resources
@@ -224,7 +222,6 @@ private void flushAllLoggers() {
 
     private int doJavaStart(ExtensionContext context, Class<? extends QuarkusTestProfile> profile, String[] arguments)
             throws Exception {
-        JBossVersion.disableVersionLogging();
 
         TracingHandler.quarkusStarting();
         Closeable testResourceManager = null;

test-framework/junit5/src/main/java/io/quarkus/test/junit/QuarkusMainTestExtension.java

@@ -65,7 +65,6 @@
 import io.quarkus.runtime.ApplicationLifecycleManager;
 import io.quarkus.runtime.LaunchMode;
 import io.quarkus.runtime.configuration.DurationConverter;
-import io.quarkus.runtime.logging.JBossVersion;
 import io.quarkus.runtime.test.TestHttpEndpointProvider;
 import io.quarkus.test.TestMethodInvoker;
 import io.quarkus.test.common.GroovyClassValue;
@@ -161,8 +160,6 @@ public void run() {
     }
 
     private ExtensionState doJavaStart(ExtensionContext context, Class<? extends QuarkusTestProfile> profile) throws Throwable {
-        JBossVersion.disableVersionLogging();
-
         // TODO we should do much less of this, because it's being done upfront by the interceptor
         TracingHandler.quarkusStarting();
         hangDetectionExecutor = Executors.newSingleThreadScheduledExecutor(new ThreadFactory() {