Skip to content
This repository has been archived by the owner on Jul 3, 2023. It is now read-only.

Removing the deprecated path_patterns in pingaccess_application_resource results in a lack of idempotency #134

Closed
ParadigmZero opened this issue Jun 24, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@ParadigmZero
Copy link

ParadigmZero commented Jun 24, 2022

Terraform Version

(tested both)
0.14.5
1.1.6

Provider version

0.9.0 ( latest )

Affected Resource(s)

pingaccess_application_resource

https://registry.terraform.io/providers/iwarapter/pingaccess/latest/docs/resources/application_resource

specifically,
path_patterns

Other information

Docker containers:
PingAccess 6.1.4-edge
PingFederate 9.3.3-edge

Background

path_prefixes in pingaccess_application_resource are deprecated. We wish to remove them, since the same information is contained in path_patterns, as an example output states below:

│ Warning: Argument is deprecated
│ 
│   with pingaccess_application_resource.placeholder,
│   on application_resources.tf line 11, in resource "pingaccess_application_resource" "placeholder":
│   11:   path_prefixes = ["/*"]
│ 
│ To be removed in a future release; please use 'path_patterns' instead
│ 
│ (and 130 more similar warnings elsewhere)

Expected Behavior

You remove the path_prefixes and there are no changes to the Terraform when doing a plan/apply. Have idempotency.

Actual Behavior

The Terraform will contuinly say the resource is being changed ( the path_prefixes are being removed). Idempotency is lost.

Steps to Reproduce

This is a specific case, however perhaps it could be recreated by:

  1. Create a pingaccess_application_resource but do not fill out the path_prefixes. Apply, then apply again ( without any further change to code). Your issue will then be loss of idempotency since it will say you are changing resources, but you have not changed anything in the code.

  2. Put in the path_prefixes into your code. It will give a warning and say the argument is deprecated.

@ParadigmZero ParadigmZero added the bug Something isn't working label Jun 24, 2022
iwarapter added a commit that referenced this issue Jul 3, 2022
iwarapter added a commit that referenced this issue Jul 4, 2022
iwarapter added a commit that referenced this issue Jul 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant