)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"0cdf8291dc872704f4b6ce535f2fce0ca18f501a","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Disable ceilometer-aipmi by default for devstack"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Due to change https://review.openstack.org/#/c/295011/ the"},{"line_number":10,"context_line":"ceiloemter-aipmi polling agent no longer works by default.  This change"},{"line_number":11,"context_line":"set updates Ceilometer\u0027s devstack plugin to disable that for standard"},{"line_number":12,"context_line":"stacks."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba0121b8_6ab46734","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":10},"updated":"2016-03-31 07:16:12.000000000","message":"Hi Drew\n\nIt should be \u0027\nceilometer-polling --polling-namespaces ipmi\u0027.","commit_id":"d5ef11ae7d85a3556d0a26d64b042a34b569e3c4"},{"author":{"_account_id":8190,"name":"Drew Thorstensen (thorst)","email":"thorst@us.ibm.com","username":"thorst"},"change_message_id":"856125d7da9fc51cf1158a2ad89d834cb7cd5f0b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Disable ceilometer-aipmi by default for devstack"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Due to change https://review.openstack.org/#/c/295011/ the"},{"line_number":10,"context_line":"ceiloemter-aipmi polling agent no longer works by default.  This change"},{"line_number":11,"context_line":"set updates Ceilometer\u0027s devstack plugin to disable that for standard"},{"line_number":12,"context_line":"stacks."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba0121b8_30a5b1a1","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":10},"in_reply_to":"ba0121b8_6ab46734","updated":"2016-03-31 12:14:03.000000000","message":"Done","commit_id":"d5ef11ae7d85a3556d0a26d64b042a34b569e3c4"}],"devstack/plugin.sh":[{"author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"change_message_id":"d69407796414a187e2c30420bd6209998d51f578","unresolved":false,"context_lines":[{"line_number":25,"context_line":"# ceilometer-anotification service. If you do this disable the IPMI"},{"line_number":26,"context_line":"# polling agent (disabled by default):"},{"line_number":27,"context_line":"#"},{"line_number":28,"context_line":"# disable_service ceilometer-aipmi"},{"line_number":29,"context_line":"#"},{"line_number":30,"context_line":"# The other way is to use Ceilometer ipmi agent only to get the IPMI based"},{"line_number":31,"context_line":"# meters. To avoid duplicated meters, users need to make sure to set the"}],"source_content_type":"text/x-sh","patch_set":1,"id":"ba0121b8_75960d40","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":34},"updated":"2016-03-31 02:47:19.000000000","message":"I am not sure, this \"disable_service ceilometer-aipmi\" only can used after \"enable_plugin ceilometer git://git.openstack.org/openstack/ceilometer\" that all the ceilometer services have been enabled, if the \"enable_plugin ceilometer\" exclude \"ceilometer-aipmi\" service by default do we still need this ? so I think cases may are: \"enable_plugin ceilometer\" or \"enable_plugin ceilometer\" + \"enable_service ceilometer-aipmi\".","commit_id":"d5ef11ae7d85a3556d0a26d64b042a34b569e3c4"},{"author":{"_account_id":8190,"name":"Drew Thorstensen (thorst)","email":"thorst@us.ibm.com","username":"thorst"},"change_message_id":"856125d7da9fc51cf1158a2ad89d834cb7cd5f0b","unresolved":false,"context_lines":[{"line_number":25,"context_line":"# ceilometer-anotification service. If you do this disable the IPMI"},{"line_number":26,"context_line":"# polling agent (disabled by default):"},{"line_number":27,"context_line":"#"},{"line_number":28,"context_line":"# disable_service ceilometer-aipmi"},{"line_number":29,"context_line":"#"},{"line_number":30,"context_line":"# The other way is to use Ceilometer ipmi agent only to get the IPMI based"},{"line_number":31,"context_line":"# meters. To avoid duplicated meters, users need to make sure to set the"}],"source_content_type":"text/x-sh","patch_set":1,"id":"ba0121b8_50bf85a9","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":34},"in_reply_to":"ba0121b8_047b1729","updated":"2016-03-31 12:14:03.000000000","message":"I changed the wording a bit, as it is still valuable information.  But I agree, what I have in this iteration is kind of...odd.\n\nThanks!","commit_id":"d5ef11ae7d85a3556d0a26d64b042a34b569e3c4"},{"author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"change_message_id":"af9b00d6a21ab741e9430e3daf67bcc6cf11da0a","unresolved":false,"context_lines":[{"line_number":25,"context_line":"# ceilometer-anotification service. If you do this disable the IPMI"},{"line_number":26,"context_line":"# polling agent (disabled by default):"},{"line_number":27,"context_line":"#"},{"line_number":28,"context_line":"# disable_service ceilometer-aipmi"},{"line_number":29,"context_line":"#"},{"line_number":30,"context_line":"# The other way is to use Ceilometer ipmi agent only to get the IPMI based"},{"line_number":31,"context_line":"# meters. To avoid duplicated meters, users need to make sure to set the"}],"source_content_type":"text/x-sh","patch_set":1,"id":"ba0121b8_047b1729","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":34},"in_reply_to":"ba0121b8_75960d40","updated":"2016-03-31 10:56:29.000000000","message":"agree, we do not need this if aipmi won\u0027t be in default ceilometer plugin services","commit_id":"d5ef11ae7d85a3556d0a26d64b042a34b569e3c4"}]}
