)]}'
{"nova/tests/unit/objects/test_objects.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"77122c23bbd99b87d732d7a4a5dfb84c6170ce1f","unresolved":false,"context_lines":[{"line_number":358,"context_line":"        object_versions \u003d {six.text_type(o): six.text_type(v)"},{"line_number":359,"context_line":"                           for o, v in object_versions.items()}"},{"line_number":360,"context_line":"        args, kwargs \u003d self._canonicalize_args(context, args, kwargs)"},{"line_number":361,"context_line":"        objver \u003d object_versions[objname]"},{"line_number":362,"context_line":"        cls \u003d base.NovaObject.obj_class_from_name(objname, objver)"},{"line_number":363,"context_line":"        with mock.patch(\u0027nova.objects.base.NovaObject.\u0027"},{"line_number":364,"context_line":"                        \u0027indirection_api\u0027, new\u003dNone):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_f8517a63","line":361,"updated":"2015-11-17 01:16:59.000000000","message":"Trying to help debug the cells test failures, I found that objver here is 2.0 even though objects.Instance.get_by_uuid() called in the test is with objects.Instance mocked as instance.InstanceV1. The Instance object that comes back over RPC is version 2.0 as a result and then fails the mox.IsA(objects.Instance) assertion because the instance is a instance.InstanceV2 when instance.InstanceV1 is expected. All of the failures are TestInstanceV1RemoteObject.\n\nI don\u0027t yet understand how the wrong object_versions are getting discovered. It looks like it comes from obj_tree_get_versions","commit_id":"a9340e238d7bb9b17e0ac7de4559c818016bc317"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"e9678aec0fe1285899bc6c8b156ccc682eda0573","unresolved":false,"context_lines":[{"line_number":358,"context_line":"        object_versions \u003d {six.text_type(o): six.text_type(v)"},{"line_number":359,"context_line":"                           for o, v in object_versions.items()}"},{"line_number":360,"context_line":"        args, kwargs \u003d self._canonicalize_args(context, args, kwargs)"},{"line_number":361,"context_line":"        objver \u003d object_versions[objname]"},{"line_number":362,"context_line":"        cls \u003d base.NovaObject.obj_class_from_name(objname, objver)"},{"line_number":363,"context_line":"        with mock.patch(\u0027nova.objects.base.NovaObject.\u0027"},{"line_number":364,"context_line":"                        \u0027indirection_api\u0027, new\u003dNone):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_c486e264","line":361,"in_reply_to":"ba8a016a_f738923f","updated":"2015-11-17 19:03:18.000000000","message":"What I mean is this patch is currently blocked for merge by Jenkins failing on some of the cells-related unit tests in test_instance.py. I did some debugging locally to try to figure out how we can get past that","commit_id":"a9340e238d7bb9b17e0ac7de4559c818016bc317"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"123a3c0e7237e197285a5e7504c0c71d8c7ca6e5","unresolved":false,"context_lines":[{"line_number":358,"context_line":"        object_versions \u003d {six.text_type(o): six.text_type(v)"},{"line_number":359,"context_line":"                           for o, v in object_versions.items()}"},{"line_number":360,"context_line":"        args, kwargs \u003d self._canonicalize_args(context, args, kwargs)"},{"line_number":361,"context_line":"        objver \u003d object_versions[objname]"},{"line_number":362,"context_line":"        cls \u003d base.NovaObject.obj_class_from_name(objname, objver)"},{"line_number":363,"context_line":"        with mock.patch(\u0027nova.objects.base.NovaObject.\u0027"},{"line_number":364,"context_line":"                        \u0027indirection_api\u0027, new\u003dNone):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_f738923f","line":361,"in_reply_to":"ba8a016a_f8517a63","updated":"2015-11-17 11:28:33.000000000","message":"Melanie, I\u0027m not sure I see that as a related problem with that change, right?\n\nI mean, that sounds indeed a bug, but since the liberty gate is wedged because of that, could we just leave that patch be merged and open a follow-up bug for the object_versions discovery you mention ? (IIUC, that could be an o.vo bug, right?)","commit_id":"a9340e238d7bb9b17e0ac7de4559c818016bc317"}]}