fix: remap dry-run config field (#12)

h4wkmoon · web-flow · commit 8ec4deb38ffa · 2024-02-17T13:51:53.000+01:00
diff --git a/autopilot/main.go b/autopilot/main.go
@@ -29,8 +29,6 @@ import (
 
 type (
 	AzureK8sAutopilot struct {
-		DryRun bool
-
 		ctx    context.Context
 		Config config.Opts
 
diff --git a/autopilot/task.repair.go b/autopilot/task.repair.go
@@ -65,7 +65,7 @@ func (r *AzureK8sAutopilot) repairRun(contextLogger *zap.SugaredLogger) {
 				continue
 			}
 
-			if r.DryRun {
+			if r.Config.DryRun {
 				nodeContextLogger.Infof("node %s repair skipped, dry run", node.Name)
 				if err := r.repair.nodeLock.Add(node.Name, true, r.Config.Repair.LockDuration); err != nil {
 					nodeContextLogger.Error(err)
diff --git a/autopilot/task.update.go b/autopilot/task.update.go
@@ -31,7 +31,7 @@ func (r *AzureK8sAutopilot) updateRun(contextLogger *zap.SugaredLogger) {
 		return
 	}
 
-	if !r.DryRun {
+	if !r.Config.DryRun {
 		for _, node := range candidateList {
 			// concurrency update limit
 			if r.Config.Update.Limit > 0 && r.update.nodeLock.ItemCount() >= r.Config.Update.Limit {

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ func (r AzureK8sAutopilot) repairRun(contextLogger zap.SugaredLogger) {`
`65`	`65`	`continue`
`66`	`66`	`}`
`67`	`67`
`68`		`- if r.DryRun {`
	`68`	`+ if r.Config.DryRun {`
`69`	`69`	`nodeContextLogger.Infof("node %s repair skipped, dry run", node.Name)`
`70`	`70`	`if err := r.repair.nodeLock.Add(node.Name, true, r.Config.Repair.LockDuration); err != nil {`
`71`	`71`	`nodeContextLogger.Error(err)`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ func (r AzureK8sAutopilot) updateRun(contextLogger zap.SugaredLogger) {`
`31`	`31`	`return`
`32`	`32`	`}`
`33`	`33`
`34`		`- if !r.DryRun {`
	`34`	`+ if !r.Config.DryRun {`
`35`	`35`	`for _, node := range candidateList {`
`36`	`36`	`// concurrency update limit`
`37`	`37`	`if r.Config.Update.Limit > 0 && r.update.nodeLock.ItemCount() >= r.Config.Update.Limit {`