CiscoISE / terraform-provider-ciscoise

Terraform Provider for Cisco ISE
https://registry.terraform.io/providers/CiscoISE/ciscoise/latest/docs
MIT License
9 stars 4 forks source link

Resource 'ciscoise_downloadable_acl' returns errors for 'dacl' and 'dacl_type' parameters #65

Closed aussietexan closed 1 year ago

aussietexan commented 1 year ago

Environment: ISE version and patch: ISE 3.1 patch 4 Terraform version: 1.3.1 ISE provider version: 0.6.9-beta OS version: MacOS 12.6

Describe the bug The 'ciscoise_downloadable_acl' resource returns the following errors for valid parameters 'dacl' and 'dacl_type' as per documentation.

Error: Unsupported argument
│ 
│   on policy.tf line 125, in resource "ciscoise_downloadable_acl" "mm-dacl-ad-computer":
│  125:     dacl        = "permit ip any any"
│ 
│ An argument named "dacl" is not expected here.
╵
╷
│ Error: Unsupported argument
│ 
│   on policy.tf line 126, in resource "ciscoise_downloadable_acl" "mm-dacl-ad-computer":
│  126:     dacl_type   = "IPV4"
│ 
│ An argument named "dacl_type" is not expected here.

Terraform code

resource "ciscoise_downloadable_acl" "mm-dacl-ad-computer" {
  provider = ciscoise.ise31-2
  parameters {
    dacl        = "permit ip any any"
    dacl_type   = "IPV4"
    description = ""
    name        = "MM-DACL-AD-Computer"
  }
}

API PUT call to '/ers/config/downloadableacl' with similar config returns a 201 Created response

{
    "DownloadableAcl": {
        "name": "MM-DACL-AD-Computer",
        "description": "",
        "dacl": "permit ip any any",
        "daclType": "IPV4"
    }
}
fmunozmiranda commented 1 year ago

@aussietexan , could you please try it again with 0.6.10-beta a tell us if it now works for you?

aussietexan commented 1 year ago

Hi @fmunozmiranda. I tested with the 0.6.10-beta and the DACL creation works as expected. Thanks for the fast turnaround!

fmunozmiranda commented 1 year ago

Happy to help!