Add comparison support for blob type fields

Currently the code-generator panics whenever a CRD/type field is of type blob (array of bytes). This PR adds support for blob type comparison.

Author: a-hilaly

pkg/generate/code/compare.go

@@ -323,6 +323,11 @@ func compareScalar(
 			"%sif *%s != *%s {\n",
 			indent, firstResVarName, secondResVarName,
 		)
+	case "blob":
+		out += fmt.Sprintf(
+			"%sif !bytes.Equal(%s, %s) {\n",
+			indent, firstResVarName, secondResVarName,
+		)
 	case "timestamp":
 		// if !a.ko.Spec.CreatedAt.Equal(b.ko.Spec.CreatedAt) {
 		out += fmt.Sprintf(

pkg/model/model.go

@@ -346,7 +346,7 @@ func (m *Model) getShapeCleanGoType(shape *awssdkmodel.Shape) string {
 		// If it's a map type we need to set the GoType to the cleaned-up
 		// Camel-cased name
 		return "map[string]" + m.getShapeCleanGoType(shape.ValueRef.Shape)
-	case "list", "array", "blob":
+	case "list", "array":
 		// If it's a list type, we need to set the GoType to the cleaned-up
 		// Camel-cased name
 		return "[]" + m.getShapeCleanGoType(shape.MemberRef.Shape)

templates/pkg/resource/delta.go.tpl

@@ -3,13 +3,15 @@
 package {{ .CRD.Names.Snake }}
 
 import (
+	"bytes"
 	"reflect"
 
 	ackcompare "github.com/aws-controllers-k8s/runtime/pkg/compare"
 )
 
 // Hack to avoid import errors during build...
 var (
+	_ = &bytes.Buffer{}
 	_ = &reflect.Method{}
 )