artifactory_permission_target should not fail when permission with same name already exists

[bg2132332]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version

terraform 0.11.14

Affected Resource(s)

• artifactory_permission_target

Terraform Configuration Files

resource "artifactory_permission_target" "test" {
  name = "test_permission"

  repo {
    includes_pattern = ["$test/**"]
    repositories     = "myrepo1"

    actions = {
      groups = [
        {
          name        = "testgroup"
          permissions = ["read", "annotate", "write", "delete"]
        },
      ]
    }
  }
}

Expected Behavior

Should not fail. Needs to import the permission definition into the state file, for ex. see the same behavior with artifactory_group resource.

Actual Behavior

1 errors occurred:
* artifactory_permission_target.test: 1 error occurred:
* artifactory_permission_target.test: POST https://xxxxxxxxxxx/artifactory/api/v2/security/permissions/test_permission: 409 [{Status:409 Message:Can't create permission target 'test_permission' for type repo. It already exists.}]

Steps to Reproduce

1. terraform apply

[jamestoyer]

I'm not sure this is really the incorrect behaviour. You should be able to import the resource using the terraform import command