Fix: handle openpititx config

Signed-off-by: NLiHui <andrewli@yunify.com>

cmd/controller-manager/app/server.go

@@ -228,7 +228,7 @@ func run(s *options.KubeSphereControllerManagerOptions, stopCh <-chan struct{})
 		klog.Fatal("Unable to create helm category controller")
 	}
 
-	if !s.OpenPitrixOptions.IsEmpty() {
+	if !s.OpenPitrixOptions.AppStoreConfIsEmpty() {
 		storageClient, err := s3.NewS3Client(s.OpenPitrixOptions.S3Options)
 		if err != nil {
 			klog.Fatalf("failed to connect to s3, please check openpitrix s3 service status, error: %v", err)

pkg/apiserver/config/config.go

@@ -200,11 +200,14 @@ func (conf *Config) ToMap() map[string]bool {
 		}
 
 		if name == "openpitrix" {
-			if conf.OpenPitrixOptions.IsEmpty() {
+			// openpitrix is always true
+			result[name] = true
+			if conf.OpenPitrixOptions == nil {
 				result["openpitrix.appstore"] = false
 			} else {
-				result["openpitrix.appstore"] = true
+				result["openpitrix.appstore"] = conf.OpenPitrixOptions.AppStoreConfIsEmpty()
 			}
+			continue
 		}
 
 		if c.Field(i).IsNil() {
@@ -240,10 +243,6 @@ func (conf *Config) stripEmptyOptions() {
 		conf.LdapOptions = nil
 	}
 
-	if conf.OpenPitrixOptions != nil && conf.OpenPitrixOptions.IsEmpty() {
-		conf.OpenPitrixOptions = nil
-	}
-
 	if conf.NetworkOptions != nil && conf.NetworkOptions.IsEmpty() {
 		conf.NetworkOptions = nil
 	}

pkg/simple/client/openpitrix/options.go

@@ -39,7 +39,7 @@ func (s *Options) Validate() []error {
 	return errors
 }
 
-func (s *Options) IsEmpty() bool {
+func (s *Options) AppStoreConfIsEmpty() bool {
 	return s.S3Options == nil || s.S3Options.Endpoint == ""
 }