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_network_access_authentication_rules fails #54

Closed wiwa1978 closed 1 year ago

wiwa1978 commented 1 year ago

Prerequisites

Describe the bug When using ciscoise_network_access_authentication_rules resource, Terraform outputs that id and name are not expected.

resource "ciscoise_network_access_authentication_rules" "example" {
  provider = ciscoise
  parameters {
    identity_source_name = "Internal Endpoints"
    if_auth_fail         = "REJECT"
    if_process_fail      = "DROP"
    if_user_not_found    = "REJECT"

    policy_id = "d2da0d1d-5d32-41e2-a88d-7dd2107bf0ca"
    rule {
      condition {
          condition_type= "ConditionReference"
          is_negate = false
          name = "Wired_MAB"
          id = "9aab0da7-e3e3-4cd7-81c2-18c3ebbe6a96"
      }
      default    = "false"
      id         = "string"
      name       = "auth_wim"
      rank       = 1
      state      = "enabled"
    }
  }
}

Results in:

│ Error: Unsupported argument
│ 
│   on network_access_authentication_rules.tf line 15, in resource "ciscoise_network_access_authentication_rules" "example":
│   15:           name = "Wired_MAB"
│ 
│ An argument named "name" is not expected here.
╵
╷
│ Error: Unsupported argument
│ 
│   on network_access_authentication_rules.tf line 16, in resource "ciscoise_network_access_authentication_rules" "example":
│   16:           id = "9aab0da7-e3e3-4cd7-81c2-18c3ebbe6a96"
│ 
│ An argument named "id" is not expected here.

Expected behavior As per documentation, the id and name are expected attributes, even though they are optional. They should be accepted if passed along.

Screenshots Request: image

Response: image

Environment (please complete the following information):

Additional context

Issue looks similar to #51.

fmunozmiranda commented 1 year ago

Hey @wiwa1978 there's a new version of provider (0.6.8-beta), test it again let us know if it works for you.

wiwa1978 commented 1 year ago

Fixed!