bugfix: error in Do will never return to outer scope

former bugfix is not correct, this correct it

pass nil to errChan if no err

format code, directly pass err to errChan

Signed-off-by: Guangming Wang <guangming.wang@daocloud.io>

internal/olm/manager.go

@@ -40,17 +40,19 @@ type Manager struct {
 }
 
 func (m *Manager) initialize() (err error) {
+	errChan := make(chan error, 1)
+	defer close(errChan)
 	m.once.Do(func() {
 		if m.Client == nil {
 			cfg, err := config.GetConfig()
 			if err != nil {
-				err = errors.Wrapf(err, "failed to get Kubernetes config")
+				errChan <- errors.Wrapf(err, "failed to get Kubernetes config")
 				return
 			}
 
 			client, err := ClientForConfig(cfg)
 			if err != nil {
-				err = errors.Wrapf(err, "failed to create manager client")
+				errChan <- errors.Wrapf(err, "failed to create manager client")
 				return
 			}
 			m.Client = client
@@ -61,7 +63,9 @@ func (m *Manager) initialize() (err error) {
 		if m.Timeout <= 0 {
 			m.Timeout = DefaultTimeout
 		}
+		errChan <- nil
 	})
+	err = <-errChan
 	return err
 }