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

webdevops · Feb 17, 2024 · 8ec4deb · 8ec4deb
1 parent 8749132
commit 8ec4deb
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
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 {