pdutil: use temp pause config (#592) (#596)

* pdutil: use temp pause config * refactor pdutil * fix ci * add pd version fail back * fix ci * fix ci * fix unmarshal bool * address comment * fi * add test case * Apply suggestions from code review Co-authored-by: kennytm <kennytm@gmail.com> * address comment * fix ci * update comment * update comment * Update pkg/pdutil/pd.go Co-authored-by: kennytm <kennytm@gmail.com> * address comment * change convoluted logic * resume config by set ttlSecond to 0 * test pause config * fix failpoint * update test * fix test Co-authored-by: luancheng <luancheng@pingcap.com> Co-authored-by: kennytm <kennytm@gmail.com>
pingcap · Nov 25, 2020 · b5c73bb · b5c73bb
1 parent 6001023
commit b5c73bb
Show file tree

Hide file tree

Showing 4 changed files with 188 additions and 116 deletions.
diff --git a/cmd/debug.go b/cmd/debug.go
@@ -22,7 +22,6 @@ import (
 	"go.uber.org/zap"
 
 	berrors "github.com/pingcap/br/pkg/errors"
-	"github.com/pingcap/br/pkg/pdutil"
 	"github.com/pingcap/br/pkg/restore"
 	"github.com/pingcap/br/pkg/rtree"
 	"github.com/pingcap/br/pkg/task"
@@ -367,10 +366,10 @@ func setPDConfigCommand() *cobra.Command {
 			}
 			defer mgr.Close()
 
-			if err := mgr.UpdatePDScheduleConfig(ctx, pdutil.DefaultPDCfg); err != nil {
+			if err := mgr.UpdatePDScheduleConfig(ctx); err != nil {
 				return errors.Annotate(err, "fail to update PD merge config")
 			}
-			log.Info("add pd configs succeed", zap.Any("config", pdutil.DefaultPDCfg))
+			log.Info("add pd configs succeed")
 			return nil
 		},
 	}