fix: evaluate controller-added tags (#314)

Signed-off-by: Eytan Avisror <[email protected]>

Author: eytan-avisror

controllers/provisioners/eks/update.go

@@ -235,12 +235,11 @@ func (ctx *EksInstanceGroupContext) UpdateScalingGroup(configName string, scalin
 
 func (ctx *EksInstanceGroupContext) TagsUpdateNeeded() bool {
 	var (
-		instanceGroup = ctx.GetInstanceGroup()
-		configuration = instanceGroup.GetEKSConfiguration()
-		state         = ctx.GetDiscoveredState()
-		scalingGroup  = state.GetScalingGroup()
-		asgName       = aws.StringValue(scalingGroup.AutoScalingGroupName)
-		rmTags        = ctx.GetRemovedTags(asgName)
+		state        = ctx.GetDiscoveredState()
+		scalingGroup = state.GetScalingGroup()
+		asgName      = aws.StringValue(scalingGroup.AutoScalingGroupName)
+		rmTags       = ctx.GetRemovedTags(asgName)
+		addedTags    = ctx.GetAddedTags(asgName)
 	)
 
 	if len(rmTags) > 0 {
@@ -256,7 +255,11 @@ func (ctx *EksInstanceGroupContext) TagsUpdateNeeded() bool {
 		existingTags = append(existingTags, tagSet)
 	}
 
-	for _, tag := range configuration.GetTags() {
+	for _, tag := range addedTags {
+		tag := map[string]string{
+			"key":   aws.StringValue(tag.Key),
+			"value": aws.StringValue(tag.Value),
+		}
 		if !common.StringMapSliceContains(existingTags, tag) {
 			return true
 		}