)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"1a3883019fcd7d9dec2d944f392792f91d573fa8","unresolved":false,"context_lines":[{"line_number":10,"context_line":"RPC."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes bug: #1005933"},{"line_number":13,"context_line":"Co-Authored-By: Ala Rezmerita \u003cala.rezmerita@cloudwatt.com\u003e"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Idc40c148ef60d5e1349d30c66ba85691d93c5675"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAW3%2F%2Fwms%3D","line":13,"updated":"2014-04-10 15:24:43.000000000","message":"can you also add \nblueprint replace-rpc-cast-with-notifications","commit_id":"c61b7bc2963ad742ace0edc9ce7af59197b4128a"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"fc4c2d6dd235c6e428762778c9c48a695c51c010","unresolved":false,"context_lines":[{"line_number":10,"context_line":"RPC."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes bug: #1005933"},{"line_number":13,"context_line":"Co-Authored-By: Ala Rezmerita \u003cala.rezmerita@cloudwatt.com\u003e"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Idc40c148ef60d5e1349d30c66ba85691d93c5675"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAW3%2F%2FrEE%3D","line":13,"in_reply_to":"AAAAW3%2F%2Fwms%3D","updated":"2014-04-11 08:47:31.000000000","message":"Done","commit_id":"c61b7bc2963ad742ace0edc9ce7af59197b4128a"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"3ddad43e9fb1708e6e5307fe33cfab95f843fa6d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Permit usage of notifications for metering"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch introduce a new publisher that use notification intead of"},{"line_number":10,"context_line":"RPC."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes bug: #1005933"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":19,"id":"baada198_109de3bb","line":9,"range":{"start_line":9,"start_character":59,"end_line":9,"end_character":65},"updated":"2014-07-30 08:21:50.000000000","message":"introduce -\u003e introduces\nuse -\u003e uses\nintead -\u003e instead","commit_id":"9a443b1f33ebdf1645b20eca2be4472259607385"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"bc4876bdb706498b30f78afd8419bb82d0f545d6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Permit usage of notifications for metering"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch introduce a new publisher that use notification intead of"},{"line_number":10,"context_line":"RPC."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes bug: #1005933"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":19,"id":"baada198_21da9b4b","line":9,"in_reply_to":"baada198_109de3bb","updated":"2014-07-30 09:46:33.000000000","message":"Done","commit_id":"9a443b1f33ebdf1645b20eca2be4472259607385"},{"author":{"_account_id":8052,"name":"Nejc Saje","email":"nejc@saje.info","username":"nsaje"},"change_message_id":"979158bffa3595f82c86324415d1d5e0f55f5a2c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Permit usage of notifications for metering"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch introduces a new publisher that uses notification inteads"},{"line_number":10,"context_line":"of RPC."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes bug: #1005933"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"baada198_f7dea10f","line":9,"updated":"2014-07-30 11:04:18.000000000","message":"\u0027instead\u0027, only if you need another patch","commit_id":"81c607c1f5efaa85a983d92bc9312d6f2958005d"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"e6c2ce37457b5b6033bc8272d0cd2f0b81e13522","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Permit usage of notifications for metering"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch introduces a new publisher that uses notification inteads"},{"line_number":10,"context_line":"of RPC."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes bug: #1005933"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"baada198_7e8b62c0","line":9,"in_reply_to":"baada198_f7dea10f","updated":"2014-08-06 07:16:25.000000000","message":"Done","commit_id":"81c607c1f5efaa85a983d92bc9312d6f2958005d"}],"ceilometer/collector.py":[{"author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"change_message_id":"602e009f9467fd88581510e504d45fb2c17cecc6","unresolved":false,"context_lines":[{"line_number":41,"context_line":"cfg.CONF.register_opts(OPTS, group\u003d\"collector\")"},{"line_number":42,"context_line":"cfg.CONF.import_opt(\u0027metering_topic\u0027, \u0027ceilometer.publisher.messaging\u0027,"},{"line_number":43,"context_line":"                    group\u003d\"publisher_rpc\")"},{"line_number":44,"context_line":"cfg.CONF.import_opt(\u0027metering_topic\u0027, \u0027ceilometer.publisher.messaging\u0027,"},{"line_number":45,"context_line":"                    group\u003d\"publisher_notifier\")"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":23,"id":"1abeadc6_25bfded1","line":44,"updated":"2014-08-19 19:32:49.000000000","message":"Could you clarify why do we need two similar cfg.CONF.import_opt ?  It seems that that duplicates is useless.","commit_id":"159e14fc2b3b27e850eb607e2e1afc395144632b"},{"author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"change_message_id":"812a463e1f76f965df52f066696891e37df3d13e","unresolved":false,"context_lines":[{"line_number":41,"context_line":"cfg.CONF.register_opts(OPTS, group\u003d\"collector\")"},{"line_number":42,"context_line":"cfg.CONF.import_opt(\u0027metering_topic\u0027, \u0027ceilometer.publisher.messaging\u0027,"},{"line_number":43,"context_line":"                    group\u003d\"publisher_rpc\")"},{"line_number":44,"context_line":"cfg.CONF.import_opt(\u0027metering_topic\u0027, \u0027ceilometer.publisher.messaging\u0027,"},{"line_number":45,"context_line":"                    group\u003d\"publisher_notifier\")"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":23,"id":"1abeadc6_a573eee5","line":44,"in_reply_to":"1abeadc6_25bfded1","updated":"2014-08-19 19:36:06.000000000","message":"Sorry was not attentive enough","commit_id":"159e14fc2b3b27e850eb607e2e1afc395144632b"},{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"9d9784bfc812ba88dedc0c3b3f2faaeac51fe117","unresolved":false,"context_lines":[{"line_number":38,"context_line":"               help\u003d\u0027Port to which the UDP socket is bound.\u0027),"},{"line_number":39,"context_line":"]"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"cfg.CONF.register_opts(OPTS, group\u003d\"collector\")"},{"line_number":42,"context_line":"cfg.CONF.import_opt(\u0027metering_topic\u0027, \u0027ceilometer.publisher.messaging\u0027,"},{"line_number":43,"context_line":"                    group\u003d\"publisher_rpc\")"},{"line_number":44,"context_line":"cfg.CONF.import_opt(\u0027metering_topic\u0027, \u0027ceilometer.publisher.messaging\u0027,"}],"source_content_type":"text/x-python","patch_set":27,"id":"1abeadc6_ce2f103d","line":41,"updated":"2014-08-27 15:35:14.000000000","message":"the following is for backward compatibility, right?","commit_id":"fec77dbc85fed701fdbc3a96bae57e5ae3a705cb"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"be80673982261e5e7a13e82cf51586c5a5188060","unresolved":false,"context_lines":[{"line_number":38,"context_line":"               help\u003d\u0027Port to which the UDP socket is bound.\u0027),"},{"line_number":39,"context_line":"]"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"cfg.CONF.register_opts(OPTS, group\u003d\"collector\")"},{"line_number":42,"context_line":"cfg.CONF.import_opt(\u0027metering_topic\u0027, \u0027ceilometer.publisher.messaging\u0027,"},{"line_number":43,"context_line":"                    group\u003d\"publisher_rpc\")"},{"line_number":44,"context_line":"cfg.CONF.import_opt(\u0027metering_topic\u0027, \u0027ceilometer.publisher.messaging\u0027,"}],"source_content_type":"text/x-python","patch_set":27,"id":"1abeadc6_4e686021","line":41,"in_reply_to":"1abeadc6_ce2f103d","updated":"2014-08-27 15:37:49.000000000","message":"right... or rpc is still a valid publisher option... it\u0027s just that using notifier is recommended","commit_id":"fec77dbc85fed701fdbc3a96bae57e5ae3a705cb"}],"ceilometer/publisher/messaging.py":[{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"3ddad43e9fb1708e6e5307fe33cfab95f843fa6d","unresolved":false,"context_lines":[{"line_number":52,"context_line":"               ),"},{"line_number":53,"context_line":"    cfg.StrOpt(\u0027metering_driver\u0027,"},{"line_number":54,"context_line":"               default\u003d\u0027messagingv2\u0027,"},{"line_number":55,"context_line":"               help\u003d\u0027The topic that ceilometer uses for metering \u0027"},{"line_number":56,"context_line":"               \u0027notifications.\u0027,"},{"line_number":57,"context_line":"               )"},{"line_number":58,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":19,"id":"baada198_70d9bf83","line":55,"range":{"start_line":55,"start_character":25,"end_line":55,"end_character":30},"updated":"2014-07-30 08:21:50.000000000","message":"topic -\u003e driver","commit_id":"9a443b1f33ebdf1645b20eca2be4472259607385"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"bc4876bdb706498b30f78afd8419bb82d0f545d6","unresolved":false,"context_lines":[{"line_number":52,"context_line":"               ),"},{"line_number":53,"context_line":"    cfg.StrOpt(\u0027metering_driver\u0027,"},{"line_number":54,"context_line":"               default\u003d\u0027messagingv2\u0027,"},{"line_number":55,"context_line":"               help\u003d\u0027The topic that ceilometer uses for metering \u0027"},{"line_number":56,"context_line":"               \u0027notifications.\u0027,"},{"line_number":57,"context_line":"               )"},{"line_number":58,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":19,"id":"baada198_e19d036e","line":55,"in_reply_to":"baada198_70d9bf83","updated":"2014-07-30 09:46:33.000000000","message":"Done","commit_id":"9a443b1f33ebdf1645b20eca2be4472259607385"},{"author":{"_account_id":8052,"name":"Nejc Saje","email":"nejc@saje.info","username":"nsaje"},"change_message_id":"979158bffa3595f82c86324415d1d5e0f55f5a2c","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"METER_PUBLISH_NOTIFIER_OPTS \u003d ["},{"line_number":48,"context_line":"    cfg.StrOpt(\u0027metering_topic\u0027,"},{"line_number":49,"context_line":"               default\u003d\u0027metering\u0027,"},{"line_number":50,"context_line":"               help\u003d\u0027The topic that ceilometer uses for metering \u0027"},{"line_number":51,"context_line":"               \u0027notifications.\u0027,"},{"line_number":52,"context_line":"               ),"}],"source_content_type":"text/x-python","patch_set":20,"id":"baada198_77ca9147","line":49,"updated":"2014-07-30 11:04:18.000000000","message":"is it ok to have the same default topic for both rpc and notifications?","commit_id":"81c607c1f5efaa85a983d92bc9312d6f2958005d"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"e6c2ce37457b5b6033bc8272d0cd2f0b81e13522","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"METER_PUBLISH_NOTIFIER_OPTS \u003d ["},{"line_number":48,"context_line":"    cfg.StrOpt(\u0027metering_topic\u0027,"},{"line_number":49,"context_line":"               default\u003d\u0027metering\u0027,"},{"line_number":50,"context_line":"               help\u003d\u0027The topic that ceilometer uses for metering \u0027"},{"line_number":51,"context_line":"               \u0027notifications.\u0027,"},{"line_number":52,"context_line":"               ),"}],"source_content_type":"text/x-python","patch_set":20,"id":"baada198_1e8426b1","line":49,"in_reply_to":"baada198_77ca9147","updated":"2014-08-06 07:16:25.000000000","message":"They are no problem here, the queue created for notification are suffixed by the priority (.sample for ceilometer)","commit_id":"81c607c1f5efaa85a983d92bc9312d6f2958005d"}],"ceilometer/tests/api/v2/test_post_samples_scenarios.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"e3b5b9c58f4994a5a100ad8d2f9333f95ea37e44","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        self.useFixture(mockpatch.Patch(\u0027oslo.messaging.Notifier\u0027,"},{"line_number":43,"context_line":"                                        return_value\u003dnotifier))"},{"line_number":44,"context_line":"        super(TestPostSamples, self).setUp()"},{"line_number":45,"context_line":"        self.n \u003d notifier"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def test_one(self):"},{"line_number":48,"context_line":"        s1 \u003d [{\u0027counter_name\u0027: \u0027apples\u0027,"}],"source_content_type":"text/x-python","patch_set":22,"id":"1abeadc6_e47371af","line":45,"updated":"2014-08-18 12:11:07.000000000","message":"what\u0027s this for?","commit_id":"90b5edfb88eaadd7d52164a821303a8ae24506e7"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"bc802ea77e28179c1635e5a0b2df72de1260758f","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        self.useFixture(mockpatch.Patch(\u0027oslo.messaging.Notifier\u0027,"},{"line_number":43,"context_line":"                                        return_value\u003dnotifier))"},{"line_number":44,"context_line":"        super(TestPostSamples, self).setUp()"},{"line_number":45,"context_line":"        self.n \u003d notifier"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def test_one(self):"},{"line_number":48,"context_line":"        s1 \u003d [{\u0027counter_name\u0027: \u0027apples\u0027,"}],"source_content_type":"text/x-python","patch_set":22,"id":"1abeadc6_2409e931","line":45,"in_reply_to":"1abeadc6_04206d48","updated":"2014-08-18 12:32:31.000000000","message":"Just a nit, removed.","commit_id":"90b5edfb88eaadd7d52164a821303a8ae24506e7"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"3e75536f81c58d5f63e9062c6922b4aae08260b5","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        self.useFixture(mockpatch.Patch(\u0027oslo.messaging.Notifier\u0027,"},{"line_number":43,"context_line":"                                        return_value\u003dnotifier))"},{"line_number":44,"context_line":"        super(TestPostSamples, self).setUp()"},{"line_number":45,"context_line":"        self.n \u003d notifier"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def test_one(self):"},{"line_number":48,"context_line":"        s1 \u003d [{\u0027counter_name\u0027: \u0027apples\u0027,"}],"source_content_type":"text/x-python","patch_set":22,"id":"1abeadc6_04206d48","line":45,"in_reply_to":"1abeadc6_e47371af","updated":"2014-08-18 12:16:02.000000000","message":"yeah, I could not find where self.n is actually used...","commit_id":"90b5edfb88eaadd7d52164a821303a8ae24506e7"}],"ceilometer/tests/publisher/test_messaging_publisher.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"7d8191b3a24be38e25596dfffe03dbf136eb573c","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    ]"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    def setUp(self):"},{"line_number":97,"context_line":"        super(TestPublish, self).setUp()"},{"line_number":98,"context_line":"        self.CONF \u003d self.useFixture(fixture_config.Config()).conf"},{"line_number":99,"context_line":"        self.setup_messaging(self.CONF)"},{"line_number":100,"context_line":"        self.published \u003d []"}],"source_content_type":"text/x-python","patch_set":24,"id":"1abeadc6_74d71424","line":97,"updated":"2014-08-25 13:39:45.000000000","message":"i think you missed this in rebase","commit_id":"6dee77b1f0da6c7a0c3a6f4bcd71ebd13cf4eeec"}]}