Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Timeouts (requires DiffSuppress strings) #75

Open
Tracked by #66
migara opened this issue Apr 19, 2024 · 1 comment
Open
Tracked by #66

Timeouts (requires DiffSuppress strings) #75

migara opened this issue Apr 19, 2024 · 1 comment

Comments

@migara
Copy link
Member

migara commented Apr 19, 2024

We can control the timeouts at a resource level, changing these timeout values shouldn't detect as a state diff

@migara migara mentioned this issue Apr 19, 2024
21 tasks
@shinmog
Copy link
Collaborator

shinmog commented Apr 25, 2024

Assuming we are not pursuing diff suppress anymore as it is mentioned in #76 then we should probably use the standard library HashiCorp has for timeouts:

https://github.com/hashicorp/terraform-plugin-framework-timeouts

That being said, I don't think standard resources need timeouts. Only uuid-style resources, panos_address_objects, panos_service_objects (if that was going to exist, I don't know), and any other specialty resources or data sources. One such data source that comes to mind is the one to export the tech support file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants