[STK-227] Pingdom support #55
Conversation
|
@waseem-h Image is available for testing. |
| @@ -0,0 +1,78 @@ | |||
| package main | |||
|
|
|||
| // import "testing" | |||
There was a problem hiding this comment.
@waseem-h Why is this commented?
There was a problem hiding this comment.
We cant run these tests in our pipeline. They require pingdom api key
There was a problem hiding this comment.
@rasheedamir are we good to go with this?
|
@waseem-h Image is available for testing. |
|
|
||
| | Annotation | Description | | ||
| |:--------------------------------------------------------:|:------------------------------------------------:| | ||
| | monitor.stakater.com/pingdom/resolution | The pingdom check interval in minutes | |
There was a problem hiding this comment.
Why do we have this as a pingdom the only configuration? Surely this could be abstracted away and used between uptimerobot as well?
There was a problem hiding this comment.
Yes you are correct, this should be done. Feel free to leave an issue or create a PR and we'll be welcome to add this change as soon as possible
No description provided.