-
Notifications
You must be signed in to change notification settings - Fork 4
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
Error in mist_org_alarmtemplate resource nested schema for rules.delivery #63
Comments
Hi Thomas @tmunzer , when this can be added to a new release? |
Hi @nunopp , Best regards |
Hi @nunopp, Version v0.2.12 has been published with the fix. Best regards |
Hi @tmunzer , after the upgrade when doing the apply I've got errors (mainly for attributes = null) on the resource mist_org_wlan, some examples: |
Yes, this should occur only during the first config sync because some default values have been updated. |
For information, I added a release note page in the Provider documentation. |
Sorry for the late reply @nunopp, I'll work on it |
Hi @nunopp, I just released v0.2.13. Can you test it? Thanks |
Hi @tmunzer , plugin crashed Thanks |
Sorry for that, I just released v0.2.14 which should fix this issue |
Hi @tmunzer, sorry to not come back earlier, I'm back now. So in applying the last version (0.2.15) I'm getting Errors in the following resources:
can you please help? |
Hi @nunopp , no problem. Can you check? |
@tmunzer however I've noticed that it works with vs 0.2.14, only when moving to 0.2.15 the above errors are triggered |
yes, I updated the Go SDK in-between (mostly because of fixes in the OpenAPI Spec used to generate the SDK, but it also bring some other changes in the SDK), which introduced this change. |
@tmunzer with version 0.2.16 I'm still getting the errors for mist_org_psk for vlan_id, however seems fixed for mist_org_wlan> dynamic_vlan.default_vlan_ids |
sorry for that. I just released a new version fixing this issue. |
@tmunzer the new version 0.2.17 apparently fixed all the errors, thank you! |
Thanks for the confirmation. I'm closing this issue. |
Describe the bug
rules.delivery attribute seems not working
To Reproduce
Steps to reproduce the behavior:
resource "mist_org_alarmtemplate" "alarmtemplate_one" {
org_id = var.xxx_org
name = "Alarm_Template_1"
delivery = {
enabled = true
to_org_admins = false
additional_emails = [ omitted emails ]
}
rules = {
dhcp_failure : {
enabled : true
delivery : {
enabled : true
}
}
}
Expected behavior
Should enable the alarm delivery via email (alarm notification via email)
Error Message
When applying changes to mist_org_alarmtemplate.alarmtemplate_one, provider "provider["registry.terraform.io/juniper/mist"]" produced an unexpected new value:
.rules["dhcp_failure"].delivery.additional_emails: was null, but now cty.ListValEmpty(cty.String).
Software
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: