search

CiscoDevNet / terraform-provider-aci

Terraform Cisco ACI provider
https://registry.terraform.io/providers/CiscoDevNet/aci/latest/docs
Mozilla Public License 2.0
91 stars 101 forks source link

[bugfix] Prevent error by setting flood_on_encap and prio for aci_end… #1209

Closed akinross closed 6 months ago

akinross commented 6 months ago

…point_security_group

In https://github.com/CiscoDevNet/terraform-provider-aci/commit/f305774eded80334087acd613c634dda068de9a2 flood_on_encap and prio were removed from schema however in the import/create/update these values were still set.

Which led to the following error:

│ Error: Plugin did not respond │ │ The plugin encountered an error, and failed to respond to the plugin6.(*GRPCProvider).ImportResourceState call. The plugin │ logs may contain more details. ╵

Stack trace from the terraform-provider-aci plugin:

panic: Invalid address to set: []string{"prio"}

Cleaning up the attributes setting from the functions cleaned up the error.

akinross commented 6 months ago

Should the data source also still have these attributes in the schema?

Officially no, but did not want to create changes to the schema since we will be making those changes anyway soon when we are working on the migrate of the resource. But since @sajagana was also asking the same I will add the change.