)]}'
{"ceilometer/alarm/service.py":[{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"fe5f770cfcbfd01c7221f8f2c94319b8d8d9718e","unresolved":false,"context_lines":[{"line_number":49,"context_line":"                    group\u003d\u0027alarm\u0027)"},{"line_number":50,"context_line":"cfg.CONF.import_opt(\u0027partition_rpc_topic\u0027, \u0027ceilometer.alarm.rpc\u0027,"},{"line_number":51,"context_line":"                    group\u003d\u0027alarm\u0027)"},{"line_number":52,"context_line":"cfg.CONF.import_opt(\u0027heartbeat\u0027, \u0027ceilometer.coordination\u0027,"},{"line_number":53,"context_line":"                    group\u003d\u0027coordination\u0027)"},{"line_number":54,"context_line":"cfg.CONF.import_opt(\u0027http_timeout\u0027, \u0027ceilometer.service\u0027)"},{"line_number":55,"context_line":"cfg.CONF.import_group(\u0027service_credentials\u0027, \u0027ceilometer.service\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a961159_59f5fb81","side":"PARENT","line":52,"updated":"2014-12-08 16:52:20.000000000","message":"Are you sure that this is for the alarms? I think this is used but the central agent for the HA stuff","commit_id":"85acec02b2b9443f376bde1b6d96945430f70fbc"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"ae2d331e8da241c21dd4ff40afb289ee5718d004","unresolved":false,"context_lines":[{"line_number":49,"context_line":"                    group\u003d\u0027alarm\u0027)"},{"line_number":50,"context_line":"cfg.CONF.import_opt(\u0027partition_rpc_topic\u0027, \u0027ceilometer.alarm.rpc\u0027,"},{"line_number":51,"context_line":"                    group\u003d\u0027alarm\u0027)"},{"line_number":52,"context_line":"cfg.CONF.import_opt(\u0027heartbeat\u0027, \u0027ceilometer.coordination\u0027,"},{"line_number":53,"context_line":"                    group\u003d\u0027coordination\u0027)"},{"line_number":54,"context_line":"cfg.CONF.import_opt(\u0027http_timeout\u0027, \u0027ceilometer.service\u0027)"},{"line_number":55,"context_line":"cfg.CONF.import_group(\u0027service_credentials\u0027, \u0027ceilometer.service\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a961159_e4bb5ec2","side":"PARENT","line":52,"in_reply_to":"3a961159_59f5fb81","updated":"2014-12-08 17:16:17.000000000","message":"See L30, we already import that module ceilometer.coordination","commit_id":"85acec02b2b9443f376bde1b6d96945430f70fbc"},{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"361df87f0191d8ea09acbdee3b6769e274677842","unresolved":false,"context_lines":[{"line_number":49,"context_line":"                    group\u003d\u0027alarm\u0027)"},{"line_number":50,"context_line":"cfg.CONF.import_opt(\u0027partition_rpc_topic\u0027, \u0027ceilometer.alarm.rpc\u0027,"},{"line_number":51,"context_line":"                    group\u003d\u0027alarm\u0027)"},{"line_number":52,"context_line":"cfg.CONF.import_opt(\u0027heartbeat\u0027, \u0027ceilometer.coordination\u0027,"},{"line_number":53,"context_line":"                    group\u003d\u0027coordination\u0027)"},{"line_number":54,"context_line":"cfg.CONF.import_opt(\u0027http_timeout\u0027, \u0027ceilometer.service\u0027)"},{"line_number":55,"context_line":"cfg.CONF.import_group(\u0027service_credentials\u0027, \u0027ceilometer.service\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a961159_cebe31d4","side":"PARENT","line":52,"in_reply_to":"3a961159_b0e885b5","updated":"2014-12-10 15:43:02.000000000","message":"Ok, thanks got it","commit_id":"85acec02b2b9443f376bde1b6d96945430f70fbc"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"0f147d4d680aa784d56db0993b8bde1abfd1748e","unresolved":false,"context_lines":[{"line_number":49,"context_line":"                    group\u003d\u0027alarm\u0027)"},{"line_number":50,"context_line":"cfg.CONF.import_opt(\u0027partition_rpc_topic\u0027, \u0027ceilometer.alarm.rpc\u0027,"},{"line_number":51,"context_line":"                    group\u003d\u0027alarm\u0027)"},{"line_number":52,"context_line":"cfg.CONF.import_opt(\u0027heartbeat\u0027, \u0027ceilometer.coordination\u0027,"},{"line_number":53,"context_line":"                    group\u003d\u0027coordination\u0027)"},{"line_number":54,"context_line":"cfg.CONF.import_opt(\u0027http_timeout\u0027, \u0027ceilometer.service\u0027)"},{"line_number":55,"context_line":"cfg.CONF.import_group(\u0027service_credentials\u0027, \u0027ceilometer.service\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a961159_b0e885b5","side":"PARENT","line":52,"in_reply_to":"3a961159_e4bb5ec2","updated":"2014-12-09 17:39:15.000000000","message":"Hi, Fabio Giannetti\n\nThis value is used in L146, but no need to import it, since we already imported whole module (which definitely registered options defined in that module) in L30, so no need to explicitly import_opt again","commit_id":"85acec02b2b9443f376bde1b6d96945430f70fbc"}]}