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

Errors when try to get an output or add a new meraki_networks_wireless_ssids_firewall_l7_firewall_rules resource #107

Closed
tienphan181 opened this issue Jul 11, 2024 · 5 comments

Comments

@tienphan181
Copy link

Describe the bug
I tried to get an output of one of our SSIDs Layer 7 firewalls rules but got the following error:
╷ │ Error: Resource NetworksWirelessSSIDsFirewallL7FirewallRules only have update context, not create. │ │ with meraki_networks_wireless_ssids_firewall_l7_firewall_rules.test-Terraform, │ on firewall-rules.tf line 16, in resource "meraki_networks_wireless_ssids_firewall_l7_firewall_rules" "test-Terraform": │ 16: resource "meraki_networks_wireless_ssids_firewall_l7_firewall_rules" "test-Terraform" { │ │ json: cannot unmarshal object into Go struct field │ ResponseWirelessGetNetworkWirelessSSIDFirewallL7FirewallRulesRules.rules.value │ of type stri
I got the same error above when I tried to create a new resource as well.

I ran the same code with an SSID that has no Layer 7 rule and the code ran fine with no error:
`Changes to Outputs:

  • meraki_networks_wireless_ssids_firewall_l7_firewall_rules_example = {
    • rules = []
      }`

Expected behavior
To be able to output and create the SSID layer 7 rules

Environment (please complete the following information):
Meraki Dashboard version:
Terraform version: OpenTofu v1.7.2
Meraki provider version: 0.2.5-alpha
OS Version: macOS 13.6.7

@fmunozmiranda
Copy link
Collaborator

Could you activate the provider debug and share it?

provider "meraki" {
  meraki_debug = "true"
}
export TF_LOG="DEBUG"

I test it and it works for me.

Please make sure Debug is enable, you must be able to see all API Requests.

fmunozmiranda added a commit that referenced this issue Jul 11, 2024
Fixing sanity problems.
@tienphan181
Copy link
Author

Hello,

Below is the logs with all API requests for layer 7.

Thank you!
layer-7-error-logs.txt

fmunozmiranda added a commit that referenced this issue Jul 15, 2024
BUGFIXES:
* Issue #88 fixed.
* Issue #89 fixed.
* Issue #92 fixed.
* Issue #93 fixed.
* Issue #94 fixed.
* Issue #96 fixed.
* Issue #97 fixed.
* Issue #98 fixed.
* Issue #99 fixed.
* Issue #100 fixed.
* Issue #101 fixed.
* Issue #107 fixed.
* Issue #108 fixed.
* Issue #109 fixed.
@fmunozmiranda fmunozmiranda mentioned this issue Jul 15, 2024
1 task
@fmunozmiranda
Copy link
Collaborator

@fmunozmiranda
Copy link
Collaborator

If we do not receive a response it will be closed in the next 24 hours.

@tienphan181
Copy link
Author

The new version fixed the issue. Thank you!

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

2 participants