openapi(dm): add openapi conifg in dm-master config file

dm/dm/master/config.go

@@ -91,7 +91,7 @@ func NewConfig() *Config {
 }
 
 type ExperimentalFeatures struct {
-	OpenAPI bool `toml:"openapi"`
+	OpenAPI bool `toml:"openapi,omitempty"` // OpenAPI is available in v5.4 as default.
 }
 
 // Config is the configuration for dm-master.
@@ -128,6 +128,7 @@ type Config struct {
 	AutoCompactionMode      string `toml:"auto-compaction-mode" json:"auto-compaction-mode"`
 	AutoCompactionRetention string `toml:"auto-compaction-retention" json:"auto-compaction-retention"`
 	QuotaBackendBytes       int64  `toml:"quota-backend-bytes" json:"quota-backend-bytes"`
+	OpenAPI                 bool   `toml:"openapi"`
 
 	// directory path used to store source config files when upgrading from v1.0.x.
 	// if this path set, DM-master leader will try to upgrade from v1.0.x to the current version.
@@ -313,6 +314,10 @@ func (c *Config) adjust() error {
 		c.QuotaBackendBytes = quotaBackendBytesLowerBound
 	}
 
+	if c.ExperimentalFeatures.OpenAPI {
+		log.L().Warn("openapi is a GA feature and removed from experimental features, so this configuration may have no affect in feature release, please set openapi=true in dm-master config file")
+	}
+
 	return err
 }
 

dm/dm/master/config_test.go

@@ -115,6 +115,7 @@ func (t *testConfigSuite) TestConfig(c *check.C) {
 			c.Assert(cfg.Join, check.Equals, "")
 			c.Assert(cfg.String(), check.Matches, fmt.Sprintf("{.*master-addr\":\"%s\".*}", masterAddr))
 			c.Assert(cfg.ExperimentalFeatures.OpenAPI, check.Equals, false)
+			c.Assert(cfg.OpenAPI, check.Equals, false)
 		}
 	}
 }

dm/dm/master/dm-master.toml

@@ -42,6 +42,9 @@ rpc-timeout = "30s"
 rpc-rate-burst = 40
 rpc-rate-limit = 10.0
 
+# openapi feature
+openapi = false
+
 # some experimental features
 [experimental]
 openapi = false

dm/dm/master/openapi_test.go

@@ -88,7 +88,7 @@ func (t *openAPISuite) TestRedirectRequestToLeader(c *check.C) {
 	cfg1.PeerUrls = tempurl.Alloc()
 	cfg1.AdvertisePeerUrls = cfg1.PeerUrls
 	cfg1.InitialCluster = fmt.Sprintf("%s=%s", cfg1.Name, cfg1.AdvertisePeerUrls)
-	cfg1.ExperimentalFeatures.OpenAPI = true
+	cfg1.OpenAPI = true
 
 	s1 := NewServer(cfg1)
 	c.Assert(s1.Start(ctx), check.IsNil)
@@ -108,7 +108,7 @@ func (t *openAPISuite) TestRedirectRequestToLeader(c *check.C) {
 	cfg2.PeerUrls = tempurl.Alloc()
 	cfg2.AdvertisePeerUrls = cfg2.PeerUrls
 	cfg2.Join = cfg1.MasterAddr // join to an existing cluster
-	cfg2.ExperimentalFeatures.OpenAPI = true
+	cfg2.OpenAPI = true
 
 	s2 := NewServer(cfg2)
 	c.Assert(s2.Start(ctx), check.IsNil)
@@ -778,7 +778,7 @@ func setupServer(ctx context.Context, c *check.C) *Server {
 	cfg1.AdvertisePeerUrls = cfg1.PeerUrls
 	cfg1.AdvertiseAddr = cfg1.MasterAddr
 	cfg1.InitialCluster = fmt.Sprintf("%s=%s", cfg1.Name, cfg1.AdvertisePeerUrls)
-	cfg1.ExperimentalFeatures.OpenAPI = true
+	cfg1.OpenAPI = true
 
 	s1 := NewServer(cfg1)
 	c.Assert(s1.Start(ctx), check.IsNil)

dm/dm/master/server.go

@@ -191,7 +191,7 @@ func (s *Server) Start(ctx context.Context) (err error) {
 		"/status": getStatusHandle(),
 		"/debug/": getDebugHandler(),
 	}
-	if s.cfg.ExperimentalFeatures.OpenAPI {
+	if s.cfg.OpenAPI || s.cfg.ExperimentalFeatures.OpenAPI {
 		if initOpenAPIErr := s.InitOpenAPIHandles(); initOpenAPIErr != nil {
 			return terror.ErrOpenAPICommonError.Delegate(initOpenAPIErr)
 		}

dm/tests/dmctl_basic/conf/get_master1.toml

@@ -20,10 +20,10 @@ max-request-bytes = 1572864
 auto-compaction-mode = "periodic"
 auto-compaction-retention = "1h"
 quota-backend-bytes = 2147483648
+openapi = false
 v1-sources-path = ""
 ssl-ca = ""
 ssl-cert = ""
 ssl-key = ""
 
 [experimental]
-  openapi = false

dm/tests/openapi/conf/dm-master1.toml

@@ -4,6 +4,4 @@ initial-cluster = "master1=http://127.0.0.1:8291"
 master-addr = ":8261"
 name = "master1"
 peer-urls = "127.0.0.1:8291"
-
-[experimental]
 openapi = true

dm/tests/openapi/conf/dm-master2.toml

@@ -4,6 +4,4 @@ join = "127.0.0.1:8261"
 master-addr = ":8361"
 name = "master2"
 peer-urls = "127.0.0.1:8292"
-
-[experimental]
 openapi = true