)]}'
{"nova/tests/unit/compute/test_compute_mgr.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"3cf1ef7de94b0ccbbcb0e2990d2f3a7df1d8ab01","unresolved":false,"context_lines":[{"line_number":6051,"context_line":"                    return_value\u003d22)"},{"line_number":6052,"context_line":"        @mock.patch(\u0027nova.objects.ComputeNode.get_by_host_and_nodename\u0027)"},{"line_number":6053,"context_line":"        @mock.patch.object(self.compute, \"_notify_about_instance_usage\")"},{"line_number":6054,"context_line":"        @mock.patch.object(compute_utils, \u0027notify_about_instance_action\u0027)"},{"line_number":6055,"context_line":"        @mock.patch.object(self.compute, \"_set_instance_info\")"},{"line_number":6056,"context_line":"        @mock.patch.object(self.instance, \u0027save\u0027)"},{"line_number":6057,"context_line":"        @mock.patch.object(self.migration, \u0027save\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_10cf1b29","line":6054,"updated":"2017-12-12 16:54:20.000000000","message":"Just a note to future myself:\nOK so this mock was not needed before as the notify_about_instance_action call did not try to load BDMs as the default of the bdms_in_notifications config option is False.\n\nAfter this patch the manager pass in a not empty list of BDMs (it loaded before) and our notification call tries to access the \u0027volume_id\u0027 fields on those BDMs which fails as the fake BDMs in this test are partially defined.","commit_id":"6507dfa01bab88c56bc0e8c8f92530ff9599b0b9"}]}