From 467ddfffdef8a01f746ace76bdf3f3f7f09e1322 Mon Sep 17 00:00:00 2001
From: Marko Urh <urh.marko@gmail.com>
Date: Fri, 10 May 2024 15:58:16 +0200
Subject: [PATCH 1/2] fix: adjust env var name, default in
 application.properties vs code and redefine log

---
 .../apache/pulsar/manager/PulsarApplicationListener.java  | 8 ++++----
 src/main/resources/application.properties                 | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/apache/pulsar/manager/PulsarApplicationListener.java b/src/main/java/org/apache/pulsar/manager/PulsarApplicationListener.java
index 912061e3..de0fac9f 100644
--- a/src/main/java/org/apache/pulsar/manager/PulsarApplicationListener.java
+++ b/src/main/java/org/apache/pulsar/manager/PulsarApplicationListener.java
@@ -60,8 +60,8 @@ public class PulsarApplicationListener implements ApplicationListener<ContextRef
     @Value("${default.environment.bookie_url}")
     private String defaultEnvironmentBookieUrl;
 
-    @Value("${default.superuser.enable}")
-    private Boolean defaultSuperuserEnable = false;
+    @Value("${default.superuser.enabled}")
+    private Boolean defaultSuperuserEnabled;
 
     @Value("${default.superuser.name}")
     private String defaultSuperuserName;
@@ -94,8 +94,8 @@ public void onApplicationEvent(ContextRefreshedEvent event) {
     }
 
     private void seedDefaultSuperuser() {
-        if(defaultSuperuserEnable == false) {
-            log.debug("Superuser seed disabled");
+        if(defaultSuperuserEnabled == false) {
+            log.info("Superuser seed via application.properties is disabled");
             return;
         }
         
diff --git a/src/main/resources/application.properties b/src/main/resources/application.properties
index 1ae3a966..d2c90488 100644
--- a/src/main/resources/application.properties
+++ b/src/main/resources/application.properties
@@ -134,7 +134,7 @@ default.environment.service_url=
 default.environment.bookie_url=
 
 # default superuser configuration
-default.superuser.enable=
+default.superuser.enabled=false
 default.superuser.name=
 default.superuser.password=
 default.superuser.email=

From d6dc1f0ddcccf0f031b55037e10dc41700865f9c Mon Sep 17 00:00:00 2001
From: Marko Urh <urh.marko@gmail.com>
Date: Fri, 10 May 2024 16:11:36 +0200
Subject: [PATCH 2/2] fix: remove circular ref

---
 .../pulsar/manager/service/impl/PulsarAdminServiceImpl.java    | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/main/java/org/apache/pulsar/manager/service/impl/PulsarAdminServiceImpl.java b/src/main/java/org/apache/pulsar/manager/service/impl/PulsarAdminServiceImpl.java
index da9ad9f9..458bdf08 100644
--- a/src/main/java/org/apache/pulsar/manager/service/impl/PulsarAdminServiceImpl.java
+++ b/src/main/java/org/apache/pulsar/manager/service/impl/PulsarAdminServiceImpl.java
@@ -71,9 +71,6 @@ public class PulsarAdminServiceImpl implements PulsarAdminService {
     @Autowired
     private EnvironmentsRepository environmentsRepository;
 
-    @Autowired
-    private EnvironmentCacheService environmentCacheService;
-
     @PreDestroy
     public void destroy() {
         pulsarAdmins.values().forEach(value -> value.close());