Return a terminal error when sdkUpdate is not implemented

a-hilaly · a-hilaly · commit 1eb464ccf008 · 2022-11-03T17:39:27.000-07:00
Currently the generated sdk code throws a simple error when `sdkUpdate` is not implemented. This is only observable in the controller logs and users are not able to read this information from their resource's status/conditions. This patch forces `sdkUpdate` to return a terminal error instead of a normal error. Fixes aws-controllers-k8s/community#1521
diff --git a/templates/pkg/resource/sdk_update_not_implemented.go.tpl b/templates/pkg/resource/sdk_update_not_implemented.go.tpl
@@ -6,6 +6,6 @@ func (rm *resourceManager) sdkUpdate(
 	delta *ackcompare.Delta,
 ) (*resource, error) {
 	// TODO(jaypipes): Figure this out...
-	return nil, ackerr.NotImplemented
+	return nil, ackerr.NewTerminal(ackerr.NotImplemented)
 }
 {{- end -}}

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,6 @@ func (rm *resourceManager) sdkUpdate(`
`6`	`6`	`delta *ackcompare.Delta,`
`7`	`7`	`) (*resource, error) {`
`8`	`8`	`// TODO(jaypipes): Figure this out...`
`9`		`- return nil, ackerr.NotImplemented`
	`9`	`+ return nil, ackerr.NewTerminal(ackerr.NotImplemented)`
`10`	`10`	`}`
`11`	`11`	`{{- end -}}`