)]}'
{"nova/tests/functional/notification_sample_tests/test_aggregate.py":[{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"4b3a593004603f4b0df3b5801372180053c0360f","unresolved":false,"context_lines":[{"line_number":92,"context_line":"        }"},{"line_number":93,"context_line":"        self.admin_api.post_aggregate_action(aggregate[\u0027id\u0027], remove_host_req)"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        self.assertEqual(4, len(fake_notifier.VERSIONED_NOTIFICATIONS))"},{"line_number":96,"context_line":"        self._verify_notification("},{"line_number":97,"context_line":"            \u0027aggregate-remove_host-start\u0027,"},{"line_number":98,"context_line":"            replacements\u003d{"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a30ddce_5be583a6","line":95,"range":{"start_line":95,"start_character":8,"end_line":95,"end_character":71},"updated":"2017-03-06 23:59:46.000000000","message":"nit: Can we use fake_notifier.reset() before this and assert for 2 notifications which would be more obvious as remove_host emits two notifications and we already asserted for add_host notifications above.","commit_id":"a0aa62c9cc03182c25a1c3e91969d4847ce60713"}],"nova/tests/unit/compute/test_compute.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"6ceaea67828303aa352ca72c96da5d66cb2b3dc0","unresolved":false,"context_lines":[{"line_number":11425,"context_line":"    @mock.patch(\u0027nova.compute.utils.notify_about_aggregate_action\u0027)"},{"line_number":11426,"context_line":"    @mock.patch(\u0027nova.compute.rpcapi.ComputeAPI.remove_aggregate_host\u0027)"},{"line_number":11427,"context_line":"    @mock.patch.object(scheduler_client.SchedulerClient, \u0027update_aggregates\u0027)"},{"line_number":11428,"context_line":"    def test_remove_host_from_aggregate(self, update_aggregates,"},{"line_number":11429,"context_line":"                                        mock_remove_agg, mock_notify):"},{"line_number":11430,"context_line":"        self.api._update_az_cache_for_host \u003d mock.Mock()"},{"line_number":11431,"context_line":"        agg \u003d objects.Aggregate(name\u003d\u0027fake\u0027, metadata\u003d{})"}],"source_content_type":"text/x-python","patch_set":9,"id":"3f044301_1eda1b4b","line":11428,"updated":"2017-05-10 15:53:48.000000000","message":"The actual notification could be asserted as well.","commit_id":"c6c4035b8c546bc0da6b87a1815c2c151ea4a313"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e17b9a483461e7179d9213fb50723bca949290eb","unresolved":false,"context_lines":[{"line_number":11425,"context_line":"    @mock.patch(\u0027nova.compute.utils.notify_about_aggregate_action\u0027)"},{"line_number":11426,"context_line":"    @mock.patch(\u0027nova.compute.rpcapi.ComputeAPI.remove_aggregate_host\u0027)"},{"line_number":11427,"context_line":"    @mock.patch.object(scheduler_client.SchedulerClient, \u0027update_aggregates\u0027)"},{"line_number":11428,"context_line":"    def test_remove_host_from_aggregate(self, update_aggregates,"},{"line_number":11429,"context_line":"                                        mock_remove_agg, mock_notify):"},{"line_number":11430,"context_line":"        self.api._update_az_cache_for_host \u003d mock.Mock()"},{"line_number":11431,"context_line":"        agg \u003d objects.Aggregate(name\u003d\u0027fake\u0027, metadata\u003d{})"}],"source_content_type":"text/x-python","patch_set":9,"id":"ff0f0b1f_e473bc7f","line":11428,"in_reply_to":"3f044301_1eda1b4b","updated":"2017-05-18 15:19:34.000000000","message":"Done","commit_id":"c6c4035b8c546bc0da6b87a1815c2c151ea4a313"}]}