Skip to content

Commit

Permalink
Merge pull request #32007 from geoand/k8s-config-logging
Browse files Browse the repository at this point in the history
Remove now unnecessary logging configuration for k8s config
  • Loading branch information
geoand authored Mar 21, 2023
2 parents 269c187 + 0186f24 commit 8099f14
Showing 1 changed file with 0 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,10 @@

import java.util.List;

import org.jboss.logmanager.Level;

import io.quarkus.deployment.annotations.BuildProducer;
import io.quarkus.deployment.annotations.BuildStep;
import io.quarkus.deployment.annotations.ExecutionTime;
import io.quarkus.deployment.annotations.Record;
import io.quarkus.deployment.builditem.LogCategoryBuildItem;
import io.quarkus.deployment.builditem.RunTimeConfigurationSourceValueBuildItem;
import io.quarkus.kubernetes.client.runtime.KubernetesClientBuildConfig;
import io.quarkus.kubernetes.config.runtime.KubernetesConfigBuildTimeConfig;
Expand Down Expand Up @@ -71,12 +68,4 @@ public void handleAccessToSecrets(KubernetesConfigSourceConfig config,

recorder.warnAboutSecrets(config, buildTimeConfig);
}

// done in order to ensure that http logs aren't shown by default which happens because of the interplay between
// not yet setup logging (as the bootstrap config runs before logging is set up) and the configuration
// of the okhttp3.logging.HttpLoggingInterceptor by io.fabric8.kubernetes.client.utils.HttpClientUtils
@BuildStep
public void produceLoggingCategories(BuildProducer<LogCategoryBuildItem> categories) {
categories.produce(new LogCategoryBuildItem("okhttp3.OkHttpClient", Level.WARN));
}
}

0 comments on commit 8099f14

Please sign in to comment.