Skip to content

Commit 91a3e6b

Browse files
committed
fix: properly log when helm generation is disabled
Signed-off-by: Chris Laprun <[email protected]>
1 parent 68979e6 commit 91a3e6b

File tree

2 files changed

+18
-6
lines changed

2 files changed

+18
-6
lines changed
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
package io.quarkiverse.operatorsdk.deployment.helm;
2+
3+
import org.jboss.logging.Logger;
4+
5+
import io.quarkus.deployment.annotations.BuildStep;
6+
import io.quarkus.deployment.annotations.Produce;
7+
import io.quarkus.deployment.pkg.builditem.ArtifactResultBuildItem;
8+
9+
// Note that steps of this processor won't be run in dev mode because ArtifactResultBuildItems are only considered in NORMAL mode
10+
public class HelmChartDisabledLogger {
11+
private static final Logger log = Logger.getLogger(HelmChartDisabledLogger.class);
12+
13+
@BuildStep(onlyIfNot = HelmGenerationEnabled.class)
14+
@Produce(ArtifactResultBuildItem.class)
15+
void outputHelmGenerationDisabled() {
16+
log.debug("Generating Helm chart is disabled");
17+
}
18+
}

core/deployment/src/main/java/io/quarkiverse/operatorsdk/deployment/helm/HelmChartProcessor.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -58,12 +58,6 @@ public class HelmChartProcessor {
5858
public static final String CRD_DIR = "crds";
5959
public static final String CRD_ROLE_BINDING_TEMPLATE_PATH = "/helm/crd-role-binding-template.yaml";
6060

61-
@BuildStep(onlyIfNot = HelmGenerationEnabled.class)
62-
@Produce(ArtifactResultBuildItem.class)
63-
void outputHelmGenerationDisabled() {
64-
log.debug("Generating Helm chart is disabled");
65-
}
66-
6761
@BuildStep
6862
HelmTargetDirectoryBuildItem createRelatedDirectories(OutputTargetBuildItem outputTarget) {
6963
final var helmDir = outputTarget.getOutputDirectory().resolve("helm").toFile();

0 commit comments

Comments
 (0)