refactor: remove built-in daemon manager (#2183)

Author: bzp2010

.github/actions/tmate-action

@@ -29,6 +29,8 @@ jobs:
 
     steps:
       - uses: actions/checkout@v2
+        with:
+          submodules: recursive
 
       - name: cache etcd
         id: cache-etcd
@@ -45,6 +47,15 @@ jobs:
           wget https://github.com/etcd-io/etcd/releases/download/v${{ matrix.etcd }}/etcd-v${{ matrix.etcd }}-linux-amd64.tar.gz
           tar zxvf etcd-v${{ matrix.etcd }}-linux-amd64.tar.gz
 
+      - name: install bats
+        run: |
+          git clone https://github.com/sstephenson/bats.git && cd bats
+          sudo ./install.sh /usr/local
+
       - name: run test
         working-directory: ./api
-        run: sudo ./test/shell/cli_test.sh
+        run: chmod +x ./test/shell/cli_test.sh && sudo ./test/shell/cli_test.sh
+
+      - name: tmate debugger
+        if: ${{ failure() }}
+        uses: ./.github/actions/tmate-action

.github/workflows/backend-cli-test.yml

@@ -4,3 +4,6 @@
 [submodule ".github/actions/paths-filter"]
 	path = .github/actions/paths-filter
 	url = https://github.com/dorny/paths-filter.git
+[submodule ".github/actions/tmate-action"]
+	path = .github/actions/tmate-action
+	url = https://github.com/mxschmitt/action-tmate

.gitmodules

@@ -29,12 +29,8 @@ import (
 	"github.com/apisix/manager-api/internal/log"
 )
 
-var (
-	forceStart bool
-)
-
 var rootCmd = &cobra.Command{
-	Use:   "manager-api [flags]",
+	Use:   "manager-api",
 	Short: "Apache APISIX Manager API",
 	RunE: func(cmd *cobra.Command, args []string) error {
 		err := manageAPI()
@@ -43,41 +39,25 @@ var rootCmd = &cobra.Command{
 }
 
 func init() {
-	cobra.OnInitialize(func() {
-		var err error
-		service, err = createService()
-		if err != nil {
-			fmt.Fprintf(os.Stderr, "error occurred while initializing service: %s", err)
-		}
-	})
-
 	rootCmd.PersistentFlags().StringVarP(&conf.ConfigFile, "config", "c", "", "config file")
 	rootCmd.PersistentFlags().StringVarP(&conf.WorkDir, "work-dir", "p", ".", "current work directory")
-	rootCmd.PersistentFlags().BoolVarP(&forceStart, "force", "f", false, "force start manager-api")
 
 	rootCmd.AddCommand(
 		newVersionCommand(),
-		newInstallCommand(),
-		newRemoveCommand(),
-		newStartCommand(),
-		newStopCommand(),
-		newStatusCommand(),
 	)
 }
 
 func Execute() {
 	if err := rootCmd.Execute(); err != nil {
-		_, _ = fmt.Fprintln(os.Stderr, err.Error())
+		_, _ = fmt.Fprintln(os.Stderr, err)
 	}
 }
 
 func manageAPI() error {
 	conf.InitConf()
 	log.InitLogger()
 
-	s, err := server.NewServer(&server.Options{
-		ForceStart: forceStart,
-	})
+	s, err := server.NewServer(&server.Options{})
 	if err != nil {
 		return err
 	}
@@ -94,7 +74,7 @@ func manageAPI() error {
 	case sig := <-quit:
 		log.Infof("The Manager API server receive %s and start shutting down", sig.String())
 		s.Stop()
-		log.Infof("The Manager API server exited")
+		log.Infof("See you next time!")
 	case err := <-errSig:
 		log.Errorf("The Manager API server start failed: %s", err.Error())
 		return err