Add ability to set custom sdkDelete functions (#486)

Description of changes:
Add custom delete_operation support

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Author: ndbhat

pkg/config/resource.go

@@ -75,6 +75,9 @@ type ResourceConfig struct {
 	// operation `notImplemented`. Custom read function allows the
 	// reconciler's progression.
 	ReadOperation *ReadOperationsConfig `json:"find_operation,omitempty"`
+	//DeleteOperation contains instructions for the code generator to generate
+	// Go code for the delete operation for the resource.
+	DeleteOperation *DeleteOperationsConfig `json:"delete_operation,omitempty"`
 	// Reconcile describes options for controlling the reconciliation
 	// logic for a particular resource.
 	Reconcile *ReconcileConfig `json:"reconcile,omitempty"`
@@ -347,6 +350,15 @@ type ReadOperationsConfig struct {
 	CustomMethodName string `json:"custom_method_name"`
 }
 
+// DeleteOperationsConfig contains instructions for the code generator to handle
+// custom delete operations for service APIs that have resources that have
+// difficult-to-standardize delete operations.
+type DeleteOperationsConfig struct {
+	// CustomMethodName is a string for the method name to replace the
+	// sdkDelete() method implementation for this resource
+	CustomMethodName string `json:"custom_method_name"`
+}
+
 // AdditionalColumnConfig can be used to specify additional printer columns to be included
 // in a Resource's output from kubectl.
 type AdditionalColumnConfig struct {
@@ -685,6 +697,19 @@ func (c *Config) GetCustomFindMethodName(resourceName string) string {
 	return ""
 }
 
+func (c *Config) GetCustomDeleteMethodName(resourceName string) string {
+	if c == nil {
+		return ""
+	}
+	rConfig, found := c.Resources[resourceName]
+	if found {
+		if rConfig.DeleteOperation != nil {
+			return rConfig.DeleteOperation.CustomMethodName
+		}
+	}
+	return ""
+}
+
 // GetAllRenames returns all of the CRD's field renames observed in the generator config
 // for a given map of operations.
 func (c *Config) GetAllRenames(

pkg/generate/ack/controller.go

@@ -46,6 +46,7 @@ var (
 	controllerIncludePaths = []string{
 		"boilerplate.go.tpl",
 		"pkg/resource/references_read_referenced_resource.go.tpl",
+		"pkg/resource/sdk_delete_custom.go.tpl",
 		"pkg/resource/sdk_find_custom.go.tpl",
 		"pkg/resource/sdk_find_read_one.go.tpl",
 		"pkg/resource/sdk_find_get_attributes.go.tpl",

pkg/model/crd.go

@@ -724,6 +724,10 @@ func (r *CRD) CustomFindMethodName() string {
 	return r.cfg.GetCustomFindMethodName(r.Names.Original)
 }
 
+func (r *CRD) CustomDeleteMethodName() string {
+	return r.cfg.GetCustomDeleteMethodName(r.Names.Original)
+}
+
 // ListOpMatchFieldNames returns a slice of strings representing the field
 // names in the List operation's Output shape's element Shape that we should
 // check a corresponding value in the target Spec exists.

templates/pkg/resource/sdk.go.tpl

@@ -145,7 +145,9 @@ func (rm *resourceManager) sdkDelete(
 		exit(err)
 	}()
 
-{{- if .CRD.Ops.Delete }}
+{{ if .CRD.CustomDeleteMethodName }}
+	{{- template "sdk_delete_custom" . }}
+{{- else if .CRD.Ops.Delete }}
 {{- if $hookCode := Hook .CRD "sdk_delete_pre_build_request" }}
 {{ $hookCode }}
 {{- end }}

templates/pkg/resource/sdk_delete_custom.go.tpl

@@ -0,0 +1,3 @@
+{{- define "sdk_delete_custom" -}}
+	return rm.{{ .CRD.CustomDeleteMethodName }}(ctx, r)
+{{- end -}}