openstack
/
deb-python-monascaclient
Code Issues Proposed changes

Fixed missing Pagerduty refs in docs and cli messages 

Change-Id: I9ecf3da5fa4b6eb990cf180cba3e45b9225dd3cf
This commit is contained in:
Victor Ion Munteanu 2015-02-04 16:54:46 +01:00
parent 5acfbb0f6e
commit d6bd434f21
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
README.rst
View File

@ -222,6 +222,7 @@ notification-create::
monasca notification-create cindyemail1 EMAIL cindy.employee@hp.com
monasca notification-create myapplication WEBHOOK http://localhost:5000
monasca notification-create mypagerduty PAGERDUTY nzH2LVRdMzun11HNC2oD
notification-list::
@ -231,6 +232,7 @@ notification-list::
+---------------+--------------------------------------+-------+----------------------+
| cindyemail1 | 5651406c-447d-40bd-b868-b2b3e6b59e32 | EMAIL |cindy.employee@hp.com |
| myapplication | 55905ce2-91e3-41ce-b45a-de7032f8d718 | WEBHOOK |http://localhost:5000
| mypagerduty | 5720ccb5-6a3d-22ba-545g-ce467a5b41a2 | PAGERDUTY |nzH2LVRdMzun11HNC2oD
+---------------+--------------------------------------+-------+----------------------+

8
monascaclient/v2_0/shell.py
View File

@ -357,9 +357,9 @@ def do_metric_statistics(mc, args):
@utils.arg('name', metavar='<NOTIFICATION_NAME>',
help='Name of the notification to create.')
@utils.arg('type', metavar='<TYPE>',
help='The notification type. Type must be either EMAIL or WEBHOOK.')
help='The notification type. Type must be EMAIL, WEBHOOK, or PAGERDUTY.')
@utils.arg('address', metavar='<ADDRESS>',
help='A valid EMAIL Address or URL')
help='A valid EMAIL Address, URL, or SERVICE KEY')
def do_notification_create(mc, args):
'''Create notification.'''
if args.type.upper() not in notification_types:
@ -460,9 +460,9 @@ def do_notification_delete(mc, args):
@utils.arg('name', metavar='<NOTIFICATION_NAME>',
help='Name of the notification.')
@utils.arg('type', metavar='<TYPE>',
help='The notification type. Type must be either EMAIL or WEBHOOK.')
help='The notification type. Type must be either EMAIL, WEBHOOK, or PAGERDUTY.')
@utils.arg('address', metavar='<ADDRESS>',
help='A valid EMAIL Address or URL')
help='A valid EMAIL Address, URL, or SERVICE KEY')
def do_notification_update(mc, args):
'''Update notification.'''
fields = {}