-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59870/
-----------------------------------------------------------
Review request for Ambari, Alexandr Antonenko, Richard Zang, and Yusaku Sako.
Bugs: AMBARI-21123
https://issues.apache.org/jira/browse/AMBARI-21123
Repository: ambari
Description
-------
Web Codes Part
This patch amis to support creating alert target that inclueds property
ambari.dispatch-property.script.filename on web UI
More details, please see https://issues.apache.org/jira/browse/AMBARI-20739
Diffs (updated)
-----
ambari-web/app/controllers/main/alerts/manage_alert_notifications_controller.js
df15513
ambari-web/app/messages.js 02a54f7
ambari-web/app/templates/main/alerts/create_alert_notification.hbs 7ec5b1e
ambari-web/app/utils/validator.js c069724
ambari-web/test/controllers/main/alerts/manage_alert_notifications_controller_test.js
0d58afa
Diff: https://reviews.apache.org/r/59870/diff/1/
Testing
-------
1.cd ambari-web & mvn test
21211 passing (33s)
128 pending
2.Tested in a cluster
File Attachments
----------------
script_alert_notification_1.png
https://reviews.apache.org/media/uploaded/files/2017/06/07/c77c5e4c-4d64-400a-8df9-11df1369e7fd__script_alert_notification_1.png
script_alert_notification_2.png
https://reviews.apache.org/media/uploaded/files/2017/06/07/1c230f27-4158-4ef4-90a8-483b3cfd4d52__script_alert_notification_2.png
Thanks,
yao lei