)]}'
{"nova/notifier/monitor_notifier.py":[{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"02f8e4563d01ee23244d5c04ccbe9efa7c1d2596","unresolved":false,"context_lines":[{"line_number":25,"context_line":"monitor_topic_opt \u003d \\"},{"line_number":26,"context_line":"    cfg.StrOpt(\u0027monitor_topic\u0027,"},{"line_number":27,"context_line":"               default\u003d\u0027monitor\u0027,"},{"line_number":28,"context_line":"               help\u003d\u0027RabbitMQ topic used for Nova monitor notifications\u0027)"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"FLAGS \u003d flags.FLAGS"},{"line_number":31,"context_line":"FLAGS.add_option(monitor_topic_opt)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAEn%2F%2F6bs%3D","line":28,"updated":"2012-02-15 14:44:03.000000000","message":"Since this code uses the rpc API, it\u0027s not RabbitMQ specific.  It could also be Qpid, or soon ZeroMQ.","commit_id":"ad78fc1f946800f66a3f2b315a4e79b9e18ea901"}],"nova/notifier/rabbit_notifier.py":[{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"8a4c7835836a7b883899c14334e38925d79a936d","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from nova import rpc"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"notification_topic_opt \u003d cfg.ListOpt(\u0027notification_topics\u0027,"},{"line_number":25,"context_line":"        default\u003d[\u0027notifications\u0027,],"},{"line_number":26,"context_line":"        help\u003d\u0027AMQP topic used for Nova notifications\u0027)"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAEn%2F%2F6VA%3D","line":24,"updated":"2012-02-15 18:33:42.000000000","message":"What do you think about configuration backwards compatibility here?  Would it be worth still supporting the old option, as well?  If not, is there a standard place where changes that affect backwards compatibility are documented between releases?","commit_id":"6660e53b6d791d110f515be88293aa2c6794d221"},{"author":{"_account_id":688,"name":"Sandy Walsh","email":"sandy.walsh@rackspace.com","username":"sandy-walsh"},"change_message_id":"cb77baae8dce029b25b41b791776341023f7099a","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from nova import rpc"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"notification_topic_opt \u003d cfg.ListOpt(\u0027notification_topics\u0027,"},{"line_number":25,"context_line":"        default\u003d[\u0027notifications\u0027,],"},{"line_number":26,"context_line":"        help\u003d\u0027AMQP topic used for Nova notifications\u0027)"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAEn%2F%2F6TA%3D","line":24,"in_reply_to":"AAAAEn%2F%2F6VA%3D","updated":"2012-02-15 19:24:23.000000000","message":"Yeah I debated leaving it the singular for that reason. When the branch looks close to landing I\u0027ll spam the list","commit_id":"6660e53b6d791d110f515be88293aa2c6794d221"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"349daf812f0e2feda345a84bf97146f2e208fc57","unresolved":false,"context_lines":[{"line_number":41,"context_line":"            rpc.notify(context, topic, message)"},{"line_number":42,"context_line":"        except Exception, e:"},{"line_number":43,"context_line":"            LOG.exception(_(\"Could not send notification to %(topic)s. \""},{"line_number":44,"context_line":"                            \"Payload\u003d%(payload)s\" % locals()))"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAEn%2F%2F4pE%3D","line":44,"updated":"2012-02-20 21:19:45.000000000","message":"I don\u0027t see a \u0027payload\u0027 for the LOG.exception.  s/payload/message ?","commit_id":"94a01246dded022d4df6e3ea719194d3c7fabf7b"}],"nova/scheduler/distributed_scheduler.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"d059131c6acf410b5dbb6c3ffb2494d0ae27d005","unresolved":false,"context_lines":[{"line_number":122,"context_line":"        instance \u003d self.create_instance_db_entry(context, request_spec)"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"        payload \u003d dict(request_spec\u003drequest_spec,"},{"line_number":125,"context_line":"                       weighted_host\u003dweighted_host.to_dict(),"},{"line_number":126,"context_line":"                       instance_id\u003dinstance[\u0027uuid\u0027])"},{"line_number":127,"context_line":"        notifier.notify(notifier.publisher_id(\"scheduler\"),"},{"line_number":128,"context_line":"                        \u0027scheduler.run_instance.scheduled\u0027, notifier.INFO,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAEn%2F%2F4l8%3D","line":125,"updated":"2012-02-21 01:01:33.000000000","message":"Hm... Sorry, I missed this the first time.  I think WeightedHost.to_dict() was removed when zones was ripped out, as it wasn\u0027t used anymore.  I\u0027m not seeing it in trunk","commit_id":"f1a0d5014f6ea19e3e2b918e713ab501c62a0fb8"}]}
