Merge pull request #131 from michaelhtm/release-v1.4.2

Release artifacts for release v1.4.2

Author: a-hilaly

config/controller/kustomization.yaml

@@ -6,4 +6,4 @@ kind: Kustomization
 images:
 - name: controller
   newName: public.ecr.aws/aws-controllers-k8s/dynamodb-controller
-  newTag: 1.4.1
+  newTag: 1.4.2

helm/Chart.yaml

@@ -1,8 +1,8 @@
 apiVersion: v1
 name: dynamodb-chart
 description: A Helm chart for the ACK service controller for Amazon DynamoDB (DynamoDB)
-version: 1.4.1
-appVersion: 1.4.1
+version: 1.4.2
+appVersion: 1.4.2
 home: https://github.com/aws-controllers-k8s/dynamodb-controller
 icon: https://raw.githubusercontent.com/aws/eks-charts/master/docs/logo/aws.png
 sources:

helm/templates/NOTES.txt

@@ -1,5 +1,5 @@
 {{ .Chart.Name }} has been installed.
-This chart deploys "public.ecr.aws/aws-controllers-k8s/dynamodb-controller:1.4.1".
+This chart deploys "public.ecr.aws/aws-controllers-k8s/dynamodb-controller:1.4.2".
 
 Check its status by running:
   kubectl --namespace {{ .Release.Namespace }} get pods -l "app.kubernetes.io/instance={{ .Release.Name }}"

helm/values.yaml

@@ -4,7 +4,7 @@
 
 image:
   repository: public.ecr.aws/aws-controllers-k8s/dynamodb-controller
-  tag: 1.4.1
+  tag: 1.4.2
   pullPolicy: IfNotPresent
   pullSecrets: []
 

pkg/resource/table/hooks.go

@@ -614,15 +614,14 @@ func customPreCompare(
 	// Making this field a no-op if user does not set it.
 	// This will ensure controller does not act on this field
 	// if user is unaware of it.
-	if a.ko.Spec.ContributorInsights == nil {
-		return
-	}
-	// latestInsight will always be either ENABLED or DISABLED, since we requeue at readOne if its not 
-	// either
-	desiredInsight, _ := ensureContibutorInsight(a)
-	latestInsight, _ := ensureContibutorInsight(b)
-	if desiredInsight != latestInsight {
-		delta.Add("Spec.ContributorInsights", a.ko.Spec.ContributorInsights, b.ko.Spec.ContributorInsights)
+	if a.ko.Spec.ContributorInsights != nil {
+		// latestInsight will always be either ENABLED or DISABLED, since we requeue at readOne if its not 
+		// either
+		desiredInsight, _ := ensureContibutorInsight(a)
+		latestInsight, _ := ensureContibutorInsight(b)
+		if desiredInsight != latestInsight {
+			delta.Add("Spec.ContributorInsights", a.ko.Spec.ContributorInsights, b.ko.Spec.ContributorInsights)
+		}
 	}
 
 }