)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10796,"name":"Deepak C Shetty (deepakcs)","email":"dpkshetty@gmail.com","username":"deepakcs"},"change_message_id":"4e66addbedf0d0df513a894e954ee3b12ec35804","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Change this into an assert based on that url being correct. We still"},{"line_number":15,"context_line":"need to mock out the http connection because that\u0027s done magically in"},{"line_number":16,"context_line":"__init__."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Unblocks nova patches, which are failing on unit test since the latest"},{"line_number":19,"context_line":"requests release and one of the function signatures changing."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3acd31a7_75279464","line":17,"updated":"2015-05-04 13:00:34.000000000","message":"nit: \"still need to\" sounds like a todo but its already addressed in 2nd patchset.","commit_id":"117351d6c46bd6d00cc92fcd234d67a913833cbc"}],"nova/tests/unit/virt/vmwareapi/test_read_write_util.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"33cc797d341bfd427101e0c1a685402d2fa12c2b","unresolved":false,"context_lines":[{"line_number":27,"context_line":"class ReadWriteUtilTestCase(test.NoDBTestCase):"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    # patch out requests so we don\u0027t actually touch the network"},{"line_number":30,"context_line":"    @mock.patch.object(requests.api, \u0027request\u0027)"},{"line_number":31,"context_line":"    def test_ipv6_host_read(self, mock_request):"},{"line_number":32,"context_line":"        ipv6_host \u003d \u0027fd8c:215d:178e:c51e:200:c9ff:fed1:584c\u0027"},{"line_number":33,"context_line":"        port \u003d 7443"}],"source_content_type":"text/x-python","patch_set":1,"id":"3acd31a7_ace16cd2","line":30,"updated":"2015-05-04 11:33:10.000000000","message":"you should mock out _create_read_connection\nwhat if the underlying code no longer used requests","commit_id":"83882123ecc9dec169863ee7c63b5e7ed64fccfc"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"33cc797d341bfd427101e0c1a685402d2fa12c2b","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        param_list \u003d {\"dcPath\": \u0027fake_dc\u0027, \"dsName\": \u0027fake_ds\u0027}"},{"line_number":42,"context_line":"        base_url \u003d \u0027https://[%s]:%s/folder/%s\u0027 % (ipv6_host, port, folder)"},{"line_number":43,"context_line":"        base_url +\u003d \u0027?\u0027 + urllib.urlencode(param_list)"},{"line_number":44,"context_line":"        self.assertEqual(reader._base_url,"},{"line_number":45,"context_line":"                         base_url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3acd31a7_8cb110e4","line":44,"updated":"2015-05-04 11:33:10.000000000","message":"base_url is the expected parameter so this one should come first.","commit_id":"83882123ecc9dec169863ee7c63b5e7ed64fccfc"}]}