[ignore] Made changes to resource template by providing indentations

CiscoDevNet · Apr 23, 2024 · cfdeeaa · cfdeeaa
1 parent 57faee2
commit cfdeeaa
Show file tree

Hide file tree

Showing 12 changed files with 69 additions and 75 deletions.
diff --git a/docs/data-sources/annotation.md b/docs/data-sources/annotation.md
@@ -73,7 +73,6 @@ data "aci_annotation" "example_application_epg" {
   - [aci_route_control_profile](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/route_control_profile) ([rtctrlProfile](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/rtctrlProfile/overview))
   - [aci_contract_interface](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/contract_interface) ([vzCPIf](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/vzCPIf/overview))
   - [aci_out_of_band_contract](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/out_of_band_contract) ([vzOOBBrCP](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/vzOOBBrCP/overview))
-  - Too many classes to display, see model documentation for all possible classes of [tagAnnotation](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/tagAnnotation/overview).
 * `key` (key) - (string) The key used to uniquely identify this configuration object.
 
 ### Read-Only ###

diff --git a/docs/data-sources/certificate_authority.md b/docs/data-sources/certificate_authority.md
@@ -55,7 +55,7 @@ data "aci_certificate_authority" "example_tenant" {
 ### Optional ###
 
 * `parent_dn` - (string) The distinguished name (DN) of the parent object, possible resources:
-      - [aci_tenant](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/fvTenant) ([fvTenant](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/fvTenant/overview))
+  - [aci_tenant](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/fvTenant) ([fvTenant](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/fvTenant/overview))
 
 ### Read-Only ###
 

diff --git a/docs/data-sources/key_ring.md b/docs/data-sources/key_ring.md
@@ -55,7 +55,7 @@ data "aci_key_ring" "example_tenant" {
 ### Optional ###
 
 * `parent_dn` - (string) The distinguished name (DN) of the parent object, possible resources:
-      - [aci_tenant](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/fvTenant) ([fvTenant](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/fvTenant/overview))
+  - [aci_tenant](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/fvTenant) ([fvTenant](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/fvTenant/overview))
 
 ### Read-Only ###
 

diff --git a/docs/data-sources/tag.md b/docs/data-sources/tag.md
@@ -73,7 +73,6 @@ data "aci_tag" "example_application_epg" {
   - [aci_route_control_profile](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/route_control_profile) ([rtctrlProfile](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/rtctrlProfile/overview))
   - [aci_contract_interface](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/contract_interface) ([vzCPIf](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/vzCPIf/overview))
   - [aci_out_of_band_contract](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/out_of_band_contract) ([vzOOBBrCP](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/vzOOBBrCP/overview))
-  - Too many classes to display, see model documentation for all possible classes of [tagTag](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/tagTag/overview).
 * `key` (key) - (string) The key used to uniquely identify this configuration object.
 
 ### Read-Only ###

diff --git a/docs/resources/annotation.md b/docs/resources/annotation.md
@@ -82,7 +82,6 @@ All examples for the Annotation resource can be found in the [examples](https://
   - [aci_route_control_profile](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/route_control_profile) ([rtctrlProfile](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/rtctrlProfile/overview))
   - [aci_contract_interface](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/contract_interface) ([vzCPIf](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/vzCPIf/overview))
   - [aci_out_of_band_contract](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/out_of_band_contract) ([vzOOBBrCP](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/vzOOBBrCP/overview))
-  - Too many classes to display, see model documentation for all possible classes of [tagAnnotation](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/tagAnnotation/overview).
 * `key` (key) - (string) The key used to uniquely identify this configuration object.
 * `value` (value) - (string) The value of the property.
 

diff --git a/docs/resources/certificate_authority.md b/docs/resources/certificate_authority.md
@@ -181,7 +181,7 @@ All examples for the Certificate Authority resource can be found in the [example
 ### Optional ###
 
 * `parent_dn` - (string) The distinguished name (DN) of the parent object, possible resources:
-      - [aci_tenant](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/fvTenant) ([fvTenant](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/fvTenant/overview))
+  - [aci_tenant](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/fvTenant) ([fvTenant](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/fvTenant/overview))
 * `annotation` (annotation) - (string) The annotation of the Certificate Authority object.
   - Default: `orchestrator:terraform`
 * `description` (descr) - (string) The description of the Certificate Authority object.

diff --git a/docs/resources/key_ring.md b/docs/resources/key_ring.md
@@ -196,7 +196,7 @@ All examples for the Key Ring resource can be found in the [examples](https://gi
 ### Optional ###
 
 * `parent_dn` - (string) The distinguished name (DN) of the parent object, possible resources:
-      - [aci_tenant](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/fvTenant) ([fvTenant](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/fvTenant/overview))
+  - [aci_tenant](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/fvTenant) ([fvTenant](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/fvTenant/overview))
 * `admin_state` (adminState) - (string) The current administrative state of the certificate request process.
   - Default: `started`
   - Valid Values: `completed`, `created`, `reqCreated`, `started`, `tpSet`.

diff --git a/docs/resources/tag.md b/docs/resources/tag.md
@@ -82,7 +82,6 @@ All examples for the Tag resource can be found in the [examples](https://github.
   - [aci_route_control_profile](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/route_control_profile) ([rtctrlProfile](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/rtctrlProfile/overview))
   - [aci_contract_interface](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/contract_interface) ([vzCPIf](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/vzCPIf/overview))
   - [aci_out_of_band_contract](https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs/resources/out_of_band_contract) ([vzOOBBrCP](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/vzOOBBrCP/overview))
-  - Too many classes to display, see model documentation for all possible classes of [tagTag](https://pubhub.devnetcloud.com/media/model-doc-latest/docs/app/index.html#/objects/tagTag/overview).
 * `key` (key) - (string) The key used to uniquely identify this configuration object.
 * `value` (value) - (string) The value of the property.
 

diff --git a/gen/generator.go b/gen/generator.go
@@ -39,7 +39,6 @@ import (
 	"go/format"
 	"io"
 	"log"
-	"math"
 	"net/http"
 	"os"
 	"path"
@@ -1765,10 +1764,10 @@ func setDocumentationData(m *Model, definitions Definitions) {
 		}
 	}
 
-	if len(resourcesNotFound) != 0 {
+	if len(resourcesNotFound) != 0 && len(resourcesFound) < docsParentDnAmount {
 		if len(resourcesNotFound) > docsParentDnAmount-len(resourcesFound) {
 			// TODO catch default classes and add to documentation
-			resourcesNotFound = resourcesNotFound[0:int(math.Abs(float64((docsParentDnAmount - len(resourcesFound)))))]
+			resourcesNotFound = resourcesNotFound[0:(docsParentDnAmount - len(resourcesFound))]
 			m.DocumentationParentDns = append(m.DocumentationParentDns, fmt.Sprintf("Too many classes to display, see model documentation for all possible classes of %s.", GetDevnetDocForClass(m.PkgName)))
 		} else {
 			var resourceDetails string

diff --git a/gen/templates/datasource.md.tmpl b/gen/templates/datasource.md.tmpl
@@ -70,7 +70,7 @@ Data source for {{.ResourceNameAsDescription}}
 * `parent_dn` - (string) The distinguished name (DN) of the parent object, possible resources:
   {{- if .MultiParentFormats }}
     {{- range .DocumentationParentDns }}
-      - {{ . }}
+  - {{ . }}
     {{- end }}
   {{- end }}
 {{ end }}

diff --git a/gen/templates/resource.go.tmpl b/gen/templates/resource.go.tmpl
@@ -284,74 +284,74 @@ func (r *{{.ResourceClassName}}Resource) Configure(ctx context.Context, req reso
 func (r *{{.ResourceClassName}}Resource) Create(ctx context.Context, req resource.CreateRequest, resp *resource.CreateResponse) {
 	tflog.Debug(ctx, "Start create of resource: aci_{{.ResourceName}}")
 
-{{- if .HasChild }}
-// On create retrieve information on current state prior to making any changes in order to determine child delete operations
-var stateData *{{ .ResourceClassName }}ResourceModel
-resp.Diagnostics.Append(req.Plan.Get(ctx, &stateData)...)
-set{{ .ResourceClassName }}Id(ctx, stateData)
-getAndSet{{ .ResourceClassName }}Attributes(ctx, &resp.Diagnostics, r.client, stateData)
-{{- end }}
+	{{- if .HasChild }}
+	// On create retrieve information on current state prior to making any changes in order to determine child delete operations
+	var stateData *{{ .ResourceClassName }}ResourceModel
+	resp.Diagnostics.Append(req.Plan.Get(ctx, &stateData)...)
+	set{{ .ResourceClassName }}Id(ctx, stateData)
+	getAndSet{{ .ResourceClassName }}Attributes(ctx, &resp.Diagnostics, r.client, stateData)
+	{{- end }}
 
-var data *{{ .ResourceClassName }}ResourceModel
+	var data *{{ .ResourceClassName }}ResourceModel
 
-// Read Terraform plan data into the model
-resp.Diagnostics.Append(req.Plan.Get(ctx, &data)...)
+	// Read Terraform plan data into the model
+	resp.Diagnostics.Append(req.Plan.Get(ctx, &data)...)
 
-if resp.Diagnostics.HasError() {
-    return
-}
+	if resp.Diagnostics.HasError() {
+		return
+	}
 
-set{{ .ResourceClassName }}Id(ctx, data)
+	set{{ .ResourceClassName }}Id(ctx, data)
 
-tflog.Debug(ctx, fmt.Sprintf("Create of resource aci_{{ .ResourceName }} with id '%s'", data.Id.ValueString()))
+	tflog.Debug(ctx, fmt.Sprintf("Create of resource aci_{{ .ResourceName }} with id '%s'", data.Id.ValueString()))
 
-{{ if .MultiParentFormats }}
-    {{- if .HasChild }}
-        {{- range .Children }}
-var {{ .PkgName }}Plan, {{ .PkgName }}State []{{ .ResourceClassName }}{{ $.ResourceClassName }}ResourceModel
-data.{{ .ResourceClassName }}.ElementsAs(ctx, &{{ .PkgName }}Plan, false)
-stateData.{{ .ResourceClassName }}.ElementsAs(ctx, &{{ .PkgName }}State, false)
-        {{- end }}
-jsonPayload := get{{ .ResourceClassName }}CreateJsonPayload(ctx, &resp.Diagnostics, data, stateData{{- range .Children }}, {{ .PkgName }}Plan, {{ .PkgName }}State{{- end }})    
-    {{- else }}
-jsonPayload := get{{ .ResourceClassName }}CreateJsonPayload(ctx, &resp.Diagnostics, data, stateData)
-    {{- end }}
+	{{ if .MultiParentFormats }}
+		{{- if .HasChild }}
+			{{- range .Children }}
+	var {{ .PkgName }}Plan, {{ .PkgName }}State []{{ .ResourceClassName }}{{ $.ResourceClassName }}ResourceModel
+	data.{{ .ResourceClassName }}.ElementsAs(ctx, &{{ .PkgName }}Plan, false)
+	stateData.{{ .ResourceClassName }}.ElementsAs(ctx, &{{ .PkgName }}State, false)
+			{{- end }}
+	jsonPayload := get{{ .ResourceClassName }}CreateJsonPayload(ctx, &resp.Diagnostics, data, stateData{{- range .Children }}, {{ .PkgName }}Plan, {{ .PkgName }}State{{- end }})    
+		{{- else }}
+	jsonPayload := get{{ .ResourceClassName }}CreateJsonPayload(ctx, &resp.Diagnostics, data, stateData)
+		{{- end }}
 
-if resp.Diagnostics.HasError() {
-    return
-}
+	if resp.Diagnostics.HasError() {
+		return
+	}
 
-wrapperClassMap := map[string]string{ {{- range $key, $value := .MultiParentFormats }} "{{ $value.RnPrepend }}": "{{ $value.WrapperClass }}", {{- end }} }
-for rnPrepend, wrapperClass := range wrapperClassMap {
-    if strings.Contains(data.Id.ValueString(), rnPrepend) && wrapperClass != "" {
-        DoRestRequest(ctx, &resp.Diagnostics, r.client, fmt.Sprintf("api/mo/%s%s.json", strings.Split(data.Id.ValueString(), rnPrepend)[0], rnPrepend), "POST", jsonPayload)
-    } else if strings.Contains(data.Id.ValueString(), rnPrepend) && wrapperClass == "" {
-        DoRestRequest(ctx, &resp.Diagnostics, r.client, fmt.Sprintf("api/mo/%s.json", data.Id.ValueString()), "POST", jsonPayload)
-    }
-}
-{{- else }}
-    {{- if .HasChild }}
-        {{- range .Children }}
-var {{ .PkgName }}Plan, {{ .PkgName }}State []{{ .ResourceClassName }}{{ $.ResourceClassName }}ResourceModel
-data.{{ .ResourceClassName }}.ElementsAs(ctx, &{{ .PkgName }}Plan, false)
-stateData.{{ .ResourceClassName }}.ElementsAs(ctx, &{{ .PkgName }}State, false)
-        {{- end }}
-jsonPayload := get{{ .ResourceClassName }}CreateJsonPayload(ctx, &resp.Diagnostics, data{{- range .Children }}, {{ .PkgName }}Plan, {{ .PkgName }}State{{- end }})    
-    {{- else }}
-jsonPayload := get{{ .ResourceClassName }}CreateJsonPayload(ctx, &resp.Diagnostics, data)
-    {{- end }}
+	wrapperClassMap := map[string]string{ {{- range $key, $value := .MultiParentFormats }} "{{ $value.RnPrepend }}": "{{ $value.WrapperClass }}", {{- end }} }
+	for rnPrepend, wrapperClass := range wrapperClassMap {
+		if strings.Contains(data.Id.ValueString(), rnPrepend) && wrapperClass != "" {
+			DoRestRequest(ctx, &resp.Diagnostics, r.client, fmt.Sprintf("api/mo/%s%s.json", strings.Split(data.Id.ValueString(), rnPrepend)[0], rnPrepend), "POST", jsonPayload)
+		} else if strings.Contains(data.Id.ValueString(), rnPrepend) && wrapperClass == "" {
+			DoRestRequest(ctx, &resp.Diagnostics, r.client, fmt.Sprintf("api/mo/%s.json", data.Id.ValueString()), "POST", jsonPayload)
+		}
+	}
+	{{- else }}
+		{{- if .HasChild }}
+			{{- range .Children }}
+	var {{ .PkgName }}Plan, {{ .PkgName }}State []{{ .ResourceClassName }}{{ $.ResourceClassName }}ResourceModel
+	data.{{ .ResourceClassName }}.ElementsAs(ctx, &{{ .PkgName }}Plan, false)
+	stateData.{{ .ResourceClassName }}.ElementsAs(ctx, &{{ .PkgName }}State, false)
+			{{- end }}
+	jsonPayload := get{{ .ResourceClassName }}CreateJsonPayload(ctx, &resp.Diagnostics, data{{- range .Children }}, {{ .PkgName }}Plan, {{ .PkgName }}State{{- end }})    
+		{{- else }}
+	jsonPayload := get{{ .ResourceClassName }}CreateJsonPayload(ctx, &resp.Diagnostics, data)
+		{{- end }}
 
-if resp.Diagnostics.HasError() {
-    return
-}
+	if resp.Diagnostics.HasError() {
+		return
+	}
 
-DoRestRequest(ctx, &resp.Diagnostics, r.client, fmt.Sprintf("api/mo/%s.json", data.Id.ValueString()), "POST", jsonPayload)
-if resp.Diagnostics.HasError() {
-    return
-}
-{{- end }}
+	DoRestRequest(ctx, &resp.Diagnostics, r.client, fmt.Sprintf("api/mo/%s.json", data.Id.ValueString()), "POST", jsonPayload)
+	if resp.Diagnostics.HasError() {
+		return
+	}
+	{{- end }}
 
-getAndSet{{.ResourceClassName}}Attributes(ctx, &resp.Diagnostics, r.client, data)
+	getAndSet{{.ResourceClassName}}Attributes(ctx, &resp.Diagnostics, r.client, data)
 
 	// Save data into Terraform state
 	resp.Diagnostics.Append(resp.State.Set(ctx, &data)...)
@@ -512,7 +512,7 @@ func (r *{{.ResourceClassName}}Resource) ImportState(ctx context.Context, req re
 
 func getAndSet{{.ResourceClassName}}Attributes(ctx context.Context, diags *diag.Diagnostics, client *client.Client, data *{{.ResourceClassName}}ResourceModel) {
 	{{- if .HasChild}}
-	requestData := DoRestRequest(ctx, diags, client,  fmt.Sprintf("api/mo/%s.json?rsp-subtree=children&rsp-subtree-class=%s", data.Id.ValueString(), "{{- .PkgName}},{{- listToString .ChildClasses}}"), "GET", nil)
+	requestData := DoRestRequest(ctx, diags, client, fmt.Sprintf("api/mo/%s.json?rsp-subtree=children&rsp-subtree-class=%s", data.Id.ValueString(), "{{- .PkgName}},{{- listToString .ChildClasses}}"), "GET", nil)
 	{{- else}}
 	requestData := DoRestRequest(ctx, diags, client, fmt.Sprintf("api/mo/%s.json", data.Id.ValueString()), "GET", nil)
 	{{- end}}
@@ -565,10 +565,9 @@ func getAndSet{{.ResourceClassName}}Attributes(ctx context.Context, diags *diag.
 				{{- end}}
 			}
 
-
 			{{- range .Properties}}
 			if data.{{.Name}}.IsUnknown() {
-			data.{{.Name}} = types.StringNull()
+				data.{{.Name}} = types.StringNull()
 			}
 			{{- end}}
 
@@ -847,13 +846,13 @@ func get{{ .ResourceClassName }}CreateJsonPayload(ctx context.Context, diags *di
 
     {{- if .HasChild }}
     childPayloads := []map[string]interface{}{}
-    {{- range .Children }}
+    	{{- range .Children }}
     {{ .ResourceClassName }}childPayloads := get{{ $.ResourceClassName }}{{ .ResourceClassName }}ChildPayloads(ctx, diags, data, {{ .PkgName }}Plan, {{ .PkgName }}State)
     if {{ .ResourceClassName }}childPayloads == nil {
         return nil
     }
     childPayloads = append(childPayloads, {{ .ResourceClassName }}childPayloads...)
-    {{- end }}
+    	{{- end }}
     payloadMap["children"] = childPayloads
     {{- end }}
 

diff --git a/gen/templates/resource.md.tmpl b/gen/templates/resource.md.tmpl
@@ -95,7 +95,7 @@ All examples for the {{.ResourceNameAsDescription}} resource can be found in the
 * `parent_dn` - (string) The distinguished name (DN) of the parent object, possible resources:
   {{- if .MultiParentFormats }}
     {{- range .DocumentationParentDns}}
-      - {{ . }}
+  - {{ . }}
     {{- end}}
   {{- end}}
 {{- end -}}