)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"2379e7786e33a0a515c28cde4eee75e39bb9856c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sergey Nikitin \u003csnikitin@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-07-24 13:16:02 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added removing of tags from instance after its destroying"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"After destroying of instance, its tags must be deleted."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a50d1a3_0fd16ecd","line":7,"updated":"2015-07-26 04:29:44.000000000","message":"its destroying \u003d\u003e its deletion","commit_id":"412d95d75c1cb7f4fa48d76fe9459aad624de4bf"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"2379e7786e33a0a515c28cde4eee75e39bb9856c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added removing of tags from instance after its destroying"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"After destroying of instance, its tags must be deleted."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ic773d32b510cab31eedd2a7b62816638a93a3a93"},{"line_number":12,"context_line":"Implements: blueprint tag-instances"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a50d1a3_2fd46abb","line":9,"updated":"2015-07-26 04:29:44.000000000","message":"of \u003d\u003e the","commit_id":"412d95d75c1cb7f4fa48d76fe9459aad624de4bf"}],"nova/tests/unit/db/test_db_api.py":[{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"7957bcb7e2fb34f37fe2fd612301d8a9987c1ff3","unresolved":false,"context_lines":[{"line_number":2682,"context_line":"        self.assertIsInstance(instance[\u0027access_ip_v4\u0027], six.string_types)"},{"line_number":2683,"context_line":"        self.assertIsInstance(instance[\u0027access_ip_v6\u0027], six.string_types)"},{"line_number":2684,"context_line":""},{"line_number":2685,"context_line":"    @mock.patch(\u0027nova.db.sqlalchemy.api._check_instance_exists\u0027)"},{"line_number":2686,"context_line":"    def test_instance_destroy(self, mock_check_inst_exists):"},{"line_number":2687,"context_line":"        ctxt \u003d context.get_admin_context()"},{"line_number":2688,"context_line":"        values \u003d {"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_a930a8e9","line":2685,"updated":"2015-07-24 09:48:15.000000000","message":"I wonder why we need this mock, isn\u0027t instance DB entry created normally here?","commit_id":"9615555782c010decb563ad9f8542a2519a956f6"},{"author":{"_account_id":9569,"name":"Sergey Nikitin","email":"nikitinserv@gmail.com","username":"snikitin"},"change_message_id":"67c447a833605d553b675d824e0cbf835e5a324d","unresolved":false,"context_lines":[{"line_number":2697,"context_line":"                          db.instance_get, ctxt, inst_uuid)"},{"line_number":2698,"context_line":"        self.assertIsNone(db.instance_info_cache_get(ctxt, inst_uuid))"},{"line_number":2699,"context_line":"        self.assertEqual({}, db.instance_metadata_get(ctxt, inst_uuid))"},{"line_number":2700,"context_line":"        self.assertEqual([], db.instance_tag_get_by_instance_uuid("},{"line_number":2701,"context_line":"            ctxt, inst_uuid))"},{"line_number":2702,"context_line":"        ctxt.read_deleted \u003d \u0027yes\u0027"},{"line_number":2703,"context_line":"        self.assertEqual(values[\u0027system_metadata\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_64be45d6","line":2700,"updated":"2015-07-24 10:18:04.000000000","message":"We need this mock here. At this moment instance is already destroyed, but we need to get tags from deleted instance. So to do not get InstanceNotFound exception we need this mock","commit_id":"9615555782c010decb563ad9f8542a2519a956f6"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"2caaaa44edc44084ca3c84a4bab8ddf05f29c27f","unresolved":false,"context_lines":[{"line_number":2702,"context_line":"            ctxt, inst_uuid))"},{"line_number":2703,"context_line":"        ctxt.read_deleted \u003d \u0027yes\u0027"},{"line_number":2704,"context_line":"        self.assertEqual(values[\u0027system_metadata\u0027],"},{"line_number":2705,"context_line":"                         db.instance_system_metadata_get(ctxt, inst_uuid))"},{"line_number":2706,"context_line":""},{"line_number":2707,"context_line":"    def test_instance_destroy_already_destroyed(self):"},{"line_number":2708,"context_line":"        ctxt \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a4dcd0f_a76545e2","line":2705,"updated":"2015-08-03 16:03:43.000000000","message":"its tempting to also check tags here, but its pointless due to the lack of soft_delete, so this is good I think.","commit_id":"ec19cf7dfcd2bf5a1712559599b90f9892c46595"}]}