-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Ensure terraform import documented for all supported resources #203
Comments
🤔 reading #159 and noting that sigsci_site_templated_rule does actually have a section titled import: terraform-provider-sigsci/docs/resources/site_templated_rule.md Lines 125 to 131 in 2feaf97
I don't know if I can properly evaluate this list, given that I've not tried to import these. The ids of a lot of these seem somewhat obfuscated on the admin dashboard. |
Thanks for the comment @DanInProgress . I've been doing some tinkering and this format works for some of the templated rules.
Some of the templated rules are managed the sigsci_site_templated_rule resource and others are not. I did make a tool to automagically generate HCL for an NGWAF config. You are welcome to give it a try and see if it at least can build out the HCL as you would expect it to. https://github.com/BrooksCunningham/ngwaf-terraformify As a disclaimer, this tool is not an official Fastly supported tool. |
The following resources have import documentation, but the markdown is labelled incorrectly as
See: terraform-provider-sigsci/docs/resources/site_templated_rule.md Lines 129 to 131 in 0b85b22
Should be like this instead (notice the terraform-provider-sigsci/docs/resources/site_list.md Lines 51 to 53 in 0b85b22
|
Most of the terraform providers support
terraform import
but mutliple are lacking documentation,creating an issue to inventory all resrouces and ensure that they have proper documentation.
This is especially important given the frequency of failures due to parallelism issues.
terraform import
is a vital tool for users when trying to fix bad terraform state.I'll review some of the links above and update with what I find. I've already seen a couple of related issues #159 and #202
The text was updated successfully, but these errors were encountered: