Skip to content

Commit fd5aee8

Browse files
committed
fix: exit operator on exception when starting
Fixes #1136
1 parent 4e8f123 commit fd5aee8

File tree

1 file changed

+19
-14
lines changed
  • operator-framework-core/src/main/java/io/javaoperatorsdk/operator

1 file changed

+19
-14
lines changed

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/Operator.java

Lines changed: 19 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -85,20 +85,25 @@ public List<Controller> getControllers() {
8585
* and start the cluster monitoring processes.
8686
*/
8787
public void start() {
88-
controllers.shouldStart();
89-
90-
final var version = ConfigurationServiceProvider.instance().getVersion();
91-
log.info(
92-
"Operator SDK {} (commit: {}) built on {} starting...",
93-
version.getSdkVersion(),
94-
version.getCommit(),
95-
version.getBuiltTime());
96-
97-
final var clientVersion = Version.clientVersion();
98-
log.info("Client version: {}", clientVersion);
99-
100-
ExecutorServiceManager.init();
101-
controllers.start();
88+
try {
89+
controllers.shouldStart();
90+
91+
final var version = ConfigurationServiceProvider.instance().getVersion();
92+
log.info(
93+
"Operator SDK {} (commit: {}) built on {} starting...",
94+
version.getSdkVersion(),
95+
version.getCommit(),
96+
version.getBuiltTime());
97+
98+
final var clientVersion = Version.clientVersion();
99+
log.info("Client version: {}", clientVersion);
100+
101+
ExecutorServiceManager.init();
102+
controllers.start();
103+
} catch (Exception e) {
104+
log.error("Error starting operator", e);
105+
System.exit(1);
106+
}
102107
}
103108

104109
@Override

0 commit comments

Comments
 (0)