mihirsoni / odfe-monitor-cli

Manage your Alerting monitors
Apache License 2.0
34 stars 22 forks source link

Updated ES query to return only monitors, instead of monitors AND destinations #17

Open jayschell opened 3 years ago

jayschell commented 3 years ago

With AWS hosted ElasticSearch 7.9, the query in the GetAllRemote function in monitor/remoteOperations.go returned both monitors and destinations in the hits. This led to issues with writing the monitors because the destinations do not contain fields needed, which results in a partially written monitor file that contains a monitor with blank values for the required fields.