)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"ea8718ec3b72ee68276b06dd1c08e35716e8ad8f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Balazs Gibizer \u003cbalazs.gibizer@ericsson.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-27 13:06:46 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Short circuit notifiations when not enabled"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Wasting time on generating the payload of the notifications is"},{"line_number":10,"context_line":"unnecessary if the notification will not be emitted due to"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"dfeb2761_20a2df9b","line":7,"range":{"start_line":7,"start_character":14,"end_line":7,"end_character":26},"updated":"2017-03-30 22:30:54.000000000","message":"Typo \"notifications\"","commit_id":"8471aafd888abbee25839688febc3759e2a030d0"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"ea8718ec3b72ee68276b06dd1c08e35716e8ad8f","unresolved":false,"context_lines":[{"line_number":14,"context_line":"a way to check if a Notifier will emit the notification or not."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Depends-On: Ib992f5c20fef85224fb00823e1d8d9c6cff19bec"},{"line_number":17,"context_line":"Depends-On: I094b3bf09b6c19a48e31ba868556867705b47abc"},{"line_number":18,"context_line":"Change-Id: I3e6741d59df49e1e58409314008c2ed609fdedc1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"dfeb2761_409de357","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":53},"updated":"2017-03-30 22:30:54.000000000","message":"Looks like the patch linked here is Abandoned.","commit_id":"8471aafd888abbee25839688febc3759e2a030d0"}],"nova/rpc.py":[{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"408871c205d40bfe2f8604150ccc6345b5a17c97","unresolved":false,"context_lines":[{"line_number":226,"context_line":"    return NOTIFIER.prepare(publisher_id\u003dpublisher_id)"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"def notifications_enabled_in_oslo(conf):"},{"line_number":230,"context_line":"    \"\"\"Check if oslo notifications are enabled.\"\"\""},{"line_number":231,"context_line":"    notifications_driver \u003d set(conf.oslo_messaging_notifications.driver)"},{"line_number":232,"context_line":"    return notifications_driver and notifications_driver !\u003d {\u0027noop\u0027}"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"def versioned_notifications_enabled(conf):"},{"line_number":236,"context_line":"    \"\"\"Check if versioned notifications are enabled.\"\"\""},{"line_number":237,"context_line":"    return conf.notifications.notification_format in (\u0027both\u0027, \u0027versioned\u0027)"},{"line_number":238,"context_line":""},{"line_number":239,"context_line":""},{"line_number":240,"context_line":"def if_notifications_enabled(f):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a461143_c2cebdb3","line":237,"range":{"start_line":229,"start_character":0,"end_line":237,"end_character":52},"updated":"2017-02-02 17:48:34.000000000","message":"I\u0027m not sure it makes sense to have these be separate functions.  Do you expect them to be used by anything else?","commit_id":"e2e6cb5f3bfccd98460a7a9c1ff9b10b7d46fbb4"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"0afd344e9b52f34bb0171195ed4cb03eb410065c","unresolved":false,"context_lines":[{"line_number":226,"context_line":"    return NOTIFIER.prepare(publisher_id\u003dpublisher_id)"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"def notifications_enabled_in_oslo(conf):"},{"line_number":230,"context_line":"    \"\"\"Check if oslo notifications are enabled.\"\"\""},{"line_number":231,"context_line":"    notifications_driver \u003d set(conf.oslo_messaging_notifications.driver)"},{"line_number":232,"context_line":"    return notifications_driver and notifications_driver !\u003d {\u0027noop\u0027}"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"def versioned_notifications_enabled(conf):"},{"line_number":236,"context_line":"    \"\"\"Check if versioned notifications are enabled.\"\"\""},{"line_number":237,"context_line":"    return conf.notifications.notification_format in (\u0027both\u0027, \u0027versioned\u0027)"},{"line_number":238,"context_line":""},{"line_number":239,"context_line":""},{"line_number":240,"context_line":"def if_notifications_enabled(f):"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_bb4968b9","line":237,"range":{"start_line":229,"start_character":0,"end_line":237,"end_character":52},"in_reply_to":"3a461143_c2cebdb3","updated":"2017-02-24 02:48:05.000000000","message":"OK, Let\u0027s make it more compact. Done.","commit_id":"e2e6cb5f3bfccd98460a7a9c1ff9b10b7d46fbb4"}],"nova/test.py":[{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"97f45250b1ebe440adc8f7d2d51cbc47400984b3","unresolved":false,"context_lines":[{"line_number":223,"context_line":"        self.useFixture(conf_fixture.ConfFixture(CONF))"},{"line_number":224,"context_line":"        self.useFixture(nova_fixtures.RPCFixture(\u0027nova.test\u0027))"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"        # we cannot set this in the ConfFixture as oslo only registers the"},{"line_number":227,"context_line":"        # notification opts at the first instantiation of a Notifier that"},{"line_number":228,"context_line":"        # happens only in the RPCFixture"},{"line_number":229,"context_line":"        CONF.set_default(\u0027driver\u0027, [\u0027test\u0027],"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba2be162_1ffcdeb6","line":226,"updated":"2017-03-06 16:08:23.000000000","message":"Nit: you could add NOTE tag with yours irc handle for future references :)","commit_id":"ce24699a44346a568b46fc3b81429763b6efc5aa"}]}
