site24x7 / terraform-provider-site24x7

Terraform provider for Site24x7
MIT License
22 stars 33 forks source link

Web Page Speed Monitor - check_frequency issue #229

Closed markkelly-payroc closed 8 months ago

markkelly-payroc commented 8 months ago

Hi team, I am getting an exception in terraform when creating a web page speed monitor, has anyone seen the same. I do not have the same issue with a web site monitor.

core doc i used https://registry.terraform.io/providers/site24x7/site24x7/latest/docs/resources/web_page_speed_monitor

output from scalr │ Error: Check frequency not supported. │ │ with module.crp_website_mail2_caledoncard_com.site24x7_web_page_speed_monitor.web_page_speed_monitor, │ on payroc/webpage_speed/main.tf line 17, in resource "site24x7_web_page_speed_monitor" "web_page_speed_monitor": │ 17: resource "site24x7_web_page_speed_monitor" "web_page_speed_monitor" { │ ╵ Error: Failed to perform terraform apply. Unexpected exit code: 1

variables.tf variable "display_name" { type = string description = "// (Required) Display name for the monitor"

}

variable "url" { type = string description = "(Required) Website address to monitor."

}

variable "check_frequency" { type = string description = "(Optional) Check interval for monitoring. Default: 1 https://www.site24x7.com/help/api/#check-interval for all supported" default = "5" }

main.tf // Data source to fetch a monitor group data "site24x7_monitor_group" "monitor_group" { // (Required) Regular expression denoting the name of the monitor group. name_regex = var.monitor_group_name }

data "site24x7_user_group" "user_group" { name_regex = "TOps Notifications" }

// Website Monitor API doc: https://www.site24x7.com/help/api/#web-page-speed-(browser) resource "site24x7_web_page_speed_monitor" "web_page_speed_monitor" { // (Required) Display name for the monitor // display_name = "mymonitor" display_name = var.display_name

// (Required) Website address to monitor. // website = "https://foo.bar" website = var.url

// (Optional) Check interval for monitoring. Default: 1. See // https://www.site24x7.com/help/api/#check-interval for all supported // values. // check_frequency = "15" check_frequency = var.check_frequency }

cal.tf module "crp_website_mail2_caledoncard_com" { source = "./payroc/webpage_speed" display_name = "testmk.caledoncard.com" url = "https://mail2.caledoncard.com" //alert=false monitor_group_name = "cal" //monitor_ssl = false

}

markkelly-payroc commented 8 months ago

I note in the file resource_vs_attribute_types.json there does not seem to be an entry for web page speed monitors.

also noticing check frequency is an int for web transaction browser and a string for everything else

not sure if this is related

sujithramagesh commented 7 months ago

Hi markkellypayroc, Thank you for using our service and we are sorry for the issue.Generally "5" minutes check_frequency is not supported in web page speed browser unless you are an "Enterprise" customer.So kindly use "10 min" or above...

markkelly-payroc commented 7 months ago

Thanks team

Mark Kelly

Mark Kelly Senior Site Reliability Engineer II e: @.*** w: www.payroc.com


From: sujithramagesh @.> Sent: Wednesday, January 17, 2024 7:09:31 AM To: site24x7/terraform-provider-site24x7 @.> Cc: Mark Kelly @.>; State change @.> Subject: Re: [site24x7/terraform-provider-site24x7] Web Page Speed Monitor - check_frequency issue (Issue #229)

Hi markkellypayroc, Thank you for using our service and we are sorry for the issue.Generally "5" minutes check_frequency is not supported in web page speed browser unless you are an "Enterprise" customer.So kindly use "10 min" or above...

— Reply to this email directly, view it on GitHubhttps://github.com/site24x7/terraform-provider-site24x7/issues/229#issuecomment-1895190230, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVHFAANTBDB6AXR6TSSTSRDYO52KXAVCNFSM6AAAAABBWFAH4OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJVGE4TAMRTGA. You are receiving this because you modified the open/close state.Message ID: @.***>