In order to automatically test the new variables, the simplest/only solution is to add them to the orchestrator-ci-system.conf.json file that is used by the docker subsystem and by jenkins.

Author: Yves Trudeau

go/db/db.go

@@ -72,7 +72,7 @@ func getMySQLURI() string {
 		mysqlURI, _ = SetupMySQLOrchestratorTLS(mysqlURI)
 	}
 	if config.Config.MySQLOrchestratorMaxAllowedPacket >= 0 {
-		mysqlURI = fmt.Sprintf("%s&maxAllowedPacket=%d",mysqlURI,config.Config.MySQLOrchestratorMaxAllowedPacket)
+		mysqlURI = fmt.Sprintf("%s&maxAllowedPacket=%d", mysqlURI, config.Config.MySQLOrchestratorMaxAllowedPacket)
 	}
 	return mysqlURI
 }
@@ -99,7 +99,7 @@ func openTopology(host string, port int, readTimeout int) (db *sql.DB, err error
 	)
 
 	if config.Config.MySQLTopologyMaxAllowedPacket >= 0 {
-		mysqlURI = fmt.Sprintf("%s&maxAllowedPacket=%d",mysqlURI,config.Config.MySQLTopologyMaxAllowedPacket)
+		mysqlURI = fmt.Sprintf("%s&maxAllowedPacket=%d", mysqlURI, config.Config.MySQLTopologyMaxAllowedPacket)
 	}
 	if config.Config.MySQLTopologyUseMutualTLS ||
 		(config.Config.MySQLTopologyUseMixedTLS && requiresTLS(host, port, mysql_uri)) {

tests/system/orchestrator-ci-system.conf.json

@@ -4,6 +4,7 @@
   "ListenAddress": ":3000",
   "MySQLTopologyUser": "ci",
   "MySQLTopologyPassword": "ci",
+  "MySQLTopologyMaxAllowedPacket": 67108864,
   "BackendDB": "sqlite",
   "SQLite3DataFile": "/tmp/orchestrator.sqlite3",
   "MySQLConnectTimeoutSeconds": 1,
@@ -12,6 +13,7 @@
   "InstancePollSeconds": 1,
   "HostnameResolveMethod": "none",
   "MySQLHostnameResolveMethod": "@@report_host",
+  "MySQLMaxAllowedPacket": 67108864,
   "SkipBinlogServerUnresolveCheck": true,
   "ExpiryHostnameResolvesMinutes": 60,
   "VerifyReplicationFilters": false,