sidecar: fix the bug of start index of serverid

acekingke · acekingke · commit ec6a085203d9 · 2024-11-19T16:29:08.000+08:00
diff --git a/sidecar/config.go b/sidecar/config.go
@@ -329,7 +329,7 @@ func (cfg *Config) buildExtraConfig(filePath string) (*ini.File, error) {
 		log.Info("It has remote cluster server-id start offset  +100")
 		startIndex += mysqlServerIDOffsetInc
 	}
-	if len(arr) == 3 && arr[1] == "ro" {
+	if len(arr) >= 3 && arr[len(arr)-2] == "ro" {
 		log.Info("It is readonly pod,  server-id start offset  +100")
 		startIndex += mysqlServerIDOffsetInc
 	}

Original file line number	Diff line number	Diff line change
`@@ -329,7 +329,7 @@ func (cfg Config) buildExtraConfig(filePath string) (ini.File, error) {`
`329`	`329`	`log.Info("It has remote cluster server-id start offset +100")`
`330`	`330`	`startIndex += mysqlServerIDOffsetInc`
`331`	`331`	`}`
`332`		`- if len(arr) == 3 && arr[1] == "ro" {`
	`332`	`+ if len(arr) >= 3 && arr[len(arr)-2] == "ro" {`
`333`	`333`	`log.Info("It is readonly pod, server-id start offset +100")`
`334`	`334`	`startIndex += mysqlServerIDOffsetInc`
`335`	`335`	`}`