site24x7 / terraform-provider-site24x7

Terraform provider for Site24x7
MIT License
23 stars 36 forks source link

inconsistency in default value #291

Open valugi opened 4 months ago

valugi commented 4 months ago

For web_page_speed_monitor, for check_frequency parameter the default value is 1 (String) and is measured in minutes. According to the example in the docs. If I set 1 or 5 as a value I get an exception that is not even very clear - implying that the parameter is not supported, not the value. "Check frequency not supported."

https://registry.terraform.io/providers/site24x7/site24x7/latest/docs/resources/web_page_speed_monitor

But for this type of monitor the minimal value is 10. https://www.site24x7.com/help/api/#check-interval

I would make it an integer and set the default to 10 instead and update the docs.