Skip to content

Commit f4183a7

Browse files
authored
Merge pull request #37 from dkorn/delete_trigger
Add option to delete an existing standard or group member trigger
2 parents f6013e2 + 7412758 commit f4183a7

2 files changed

Lines changed: 10 additions & 0 deletions

File tree

hawkular/alerts.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,11 @@ def get_trigger(self, trigger_id, full=False):
152152
returned_dict = self._get(self._service_url(['triggers', trigger_id]))
153153
return Trigger(returned_dict)
154154

155+
def delete_trigger(self, trigger_id):
156+
""" Delete an existing standard or group member trigger
157+
"""
158+
self._delete(self._service_url(['triggers', trigger_id]))
159+
155160
def create_group_trigger(self, trigger):
156161
data = self._serialize_object(trigger)
157162
return Trigger(self._post(self._service_url(['triggers', 'groups']), data))

tests/test_alerts.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -231,6 +231,11 @@ def test_create_groups(self):
231231
self.assertEqual(len(gm), 1)
232232
self.assertEqual(gm[0].id, 'member1')
233233

234+
# Delete group member trigger
235+
self.client.delete_trigger('member1')
236+
gm = self.client.get_group_members('a-group-trigger')
237+
self.assertFalse(gm)
238+
234239
# Create group trigger dampening
235240
self.client.create_group_dampening('a-group-trigger', dampening)
236241
gt = self.client.get_trigger('a-group-trigger', full=True)

0 commit comments

Comments
 (0)