)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"700bf9195c55ab0dd72aa425d75b9c76d163dd3d","unresolved":false,"context_lines":[{"line_number":11,"context_line":"a non-default port. This is because the URL to copy the image"},{"line_number":12,"context_line":"framed using vCenter port rather than ESXi\u0027s host port."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This patch address that issue by retriving host port explicitly to"},{"line_number":15,"context_line":"frame the URL."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I412eaa6b66ca876ed24ed0e95b0ba12c2930bdd5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"bacf61ea_3886d565","line":14,"range":{"start_line":14,"start_character":11,"end_line":14,"end_character":18},"updated":"2016-08-02 23:39:29.000000000","message":"addresses","commit_id":"f69c7afbf2958d614cd3a6887d8b7c0ebd420946"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"700bf9195c55ab0dd72aa425d75b9c76d163dd3d","unresolved":false,"context_lines":[{"line_number":11,"context_line":"a non-default port. This is because the URL to copy the image"},{"line_number":12,"context_line":"framed using vCenter port rather than ESXi\u0027s host port."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This patch address that issue by retriving host port explicitly to"},{"line_number":15,"context_line":"frame the URL."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I412eaa6b66ca876ed24ed0e95b0ba12c2930bdd5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"bacf61ea_987b817b","line":14,"range":{"start_line":14,"start_character":33,"end_line":14,"end_character":42},"updated":"2016-08-02 23:39:29.000000000","message":"retrieving the","commit_id":"f69c7afbf2958d614cd3a6887d8b7c0ebd420946"}],"nova/tests/unit/virt/vmwareapi/test_vmops.py":[{"author":{"_account_id":1865,"name":"Andrea Rosa","email":"andrea.rosa@gmail.com","username":"andrea-rosa-m"},"change_message_id":"885aefdd99f238135bf46570c922066c95af2912","unresolved":false,"context_lines":[{"line_number":2191,"context_line":"        cookies \u003d mock.Mock()"},{"line_number":2192,"context_line":"        mock_get_esx_host_port_and_cookies.return_value \u003d host, port, cookies"},{"line_number":2193,"context_line":"        self._vmops._fetch_image_as_file(self._context, vi, image_ds_loc)"},{"line_number":2194,"context_line":"        mock_get_esx_host_and_cookies.assert_called_once_with("},{"line_number":2195,"context_line":"                vi.datastore,"},{"line_number":2196,"context_line":"                dc_name,"},{"line_number":2197,"context_line":"                image_ds_loc.rel_path)"}],"source_content_type":"text/x-python","patch_set":4,"id":"dada55a8_06c4dd8e","line":2194,"range":{"start_line":2194,"start_character":8,"end_line":2194,"end_character":37},"updated":"2016-07-20 13:42:28.000000000","message":"That doesn\u0027t exist anymore","commit_id":"b18d888609d1bec580d9b5134fad687b8528e9d5"}],"nova/virt/vmwareapi/vmops.py":[{"author":{"_account_id":1865,"name":"Andrea Rosa","email":"andrea.rosa@gmail.com","username":"andrea-rosa-m"},"change_message_id":"a65b16b5e6eb3e70184d956ba30c6fbf609da013","unresolved":false,"context_lines":[{"line_number":331,"context_line":"        extra_specs.cores_per_socket \u003d topology.cores"},{"line_number":332,"context_line":"        return extra_specs"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    def _get_esx_host_and_cookies(self, datastore, dc_path, file_path):"},{"line_number":335,"context_line":"        hosts \u003d datastore.get_connected_hosts(self._session)"},{"line_number":336,"context_line":"        host \u003d ds_obj.Datastore.choose_host(hosts)"},{"line_number":337,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_0c688a86","line":334,"range":{"start_line":334,"start_character":8,"end_line":334,"end_character":33},"updated":"2016-07-20 10:27:48.000000000","message":"with this change thename of the method is wrong, calling this method we get back the port as well","commit_id":"e47bf04746be1031808e2f9d7467f94ce13e6740"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a740244c1c557f5eaf44f0de4636b56a2b404a6c","unresolved":false,"context_lines":[{"line_number":339,"context_line":"        props \u003d self._session._call_method(vutil, \"get_object_properties_dict\","},{"line_number":340,"context_line":"                                           host, lst_properties)"},{"line_number":341,"context_line":"        host_name \u003d props[\u0027name\u0027]"},{"line_number":342,"context_line":"        port \u003d props[\u0027summary.config.port\u0027]"},{"line_number":343,"context_line":"        url \u003d ds_obj.DatastoreURL(\u0027https\u0027, host_name, file_path, dc_path,"},{"line_number":344,"context_line":"                                  datastore.name)"},{"line_number":345,"context_line":"        cookie_header \u003d url.get_transfer_ticket(self._session, \u0027PUT\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"dada55a8_3ae86d7c","line":342,"updated":"2016-07-24 13:26:32.000000000","message":"i think that to be safe you should use a get here. what if the VC does not return this value","commit_id":"fa775a5fce878a18171fbc8d846b169796cddf53"},{"author":{"_account_id":9297,"name":"Johnson koil raj","email":"jraj@infoblox.com","username":"jkraj"},"change_message_id":"08c8e7ecf7849427d018f5b6aece266443fe55ff","unresolved":false,"context_lines":[{"line_number":339,"context_line":"        props \u003d self._session._call_method(vutil, \"get_object_properties_dict\","},{"line_number":340,"context_line":"                                           host, lst_properties)"},{"line_number":341,"context_line":"        host_name \u003d props[\u0027name\u0027]"},{"line_number":342,"context_line":"        port \u003d props[\u0027summary.config.port\u0027]"},{"line_number":343,"context_line":"        url \u003d ds_obj.DatastoreURL(\u0027https\u0027, host_name, file_path, dc_path,"},{"line_number":344,"context_line":"                                  datastore.name)"},{"line_number":345,"context_line":"        cookie_header \u003d url.get_transfer_ticket(self._session, \u0027PUT\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"bacf61ea_221cb812","line":342,"in_reply_to":"dada55a8_3ae86d7c","updated":"2016-07-28 18:45:54.000000000","message":"Done","commit_id":"fa775a5fce878a18171fbc8d846b169796cddf53"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"700bf9195c55ab0dd72aa425d75b9c76d163dd3d","unresolved":false,"context_lines":[{"line_number":335,"context_line":"        hosts \u003d datastore.get_connected_hosts(self._session)"},{"line_number":336,"context_line":"        host \u003d ds_obj.Datastore.choose_host(hosts)"},{"line_number":337,"context_line":""},{"line_number":338,"context_line":"        lst_properties \u003d [\"name\", \"summary.config.port\"]"},{"line_number":339,"context_line":"        props \u003d self._session._call_method(vutil, \"get_object_properties_dict\","},{"line_number":340,"context_line":"                                           host, lst_properties)"},{"line_number":341,"context_line":"        host_name \u003d props.get(\u0027name\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"bacf61ea_58e8e916","line":338,"range":{"start_line":338,"start_character":8,"end_line":338,"end_character":22},"updated":"2016-08-02 23:39:29.000000000","message":"rather something like property_list","commit_id":"f69c7afbf2958d614cd3a6887d8b7c0ebd420946"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"700bf9195c55ab0dd72aa425d75b9c76d163dd3d","unresolved":false,"context_lines":[{"line_number":335,"context_line":"        hosts \u003d datastore.get_connected_hosts(self._session)"},{"line_number":336,"context_line":"        host \u003d ds_obj.Datastore.choose_host(hosts)"},{"line_number":337,"context_line":""},{"line_number":338,"context_line":"        lst_properties \u003d [\"name\", \"summary.config.port\"]"},{"line_number":339,"context_line":"        props \u003d self._session._call_method(vutil, \"get_object_properties_dict\","},{"line_number":340,"context_line":"                                           host, lst_properties)"},{"line_number":341,"context_line":"        host_name \u003d props.get(\u0027name\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"bacf61ea_f8e39d31","line":338,"range":{"start_line":338,"start_character":26,"end_line":338,"end_character":55},"updated":"2016-08-02 23:39:29.000000000","message":"use single quotes for consistency","commit_id":"f69c7afbf2958d614cd3a6887d8b7c0ebd420946"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"700bf9195c55ab0dd72aa425d75b9c76d163dd3d","unresolved":false,"context_lines":[{"line_number":336,"context_line":"        host \u003d ds_obj.Datastore.choose_host(hosts)"},{"line_number":337,"context_line":""},{"line_number":338,"context_line":"        lst_properties \u003d [\"name\", \"summary.config.port\"]"},{"line_number":339,"context_line":"        props \u003d self._session._call_method(vutil, \"get_object_properties_dict\","},{"line_number":340,"context_line":"                                           host, lst_properties)"},{"line_number":341,"context_line":"        host_name \u003d props.get(\u0027name\u0027)"},{"line_number":342,"context_line":"        port \u003d props.get(\u0027summary.config.port\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"bacf61ea_58cf49b3","line":339,"range":{"start_line":339,"start_character":50,"end_line":339,"end_character":78},"updated":"2016-08-02 23:39:29.000000000","message":"single quotes","commit_id":"f69c7afbf2958d614cd3a6887d8b7c0ebd420946"},{"author":{"_account_id":16929,"name":"Giridhar Jayavelu","email":"gjayavelu@vmware.com","username":"gjayavelu"},"change_message_id":"ba6c19ca9436b516f17aabe9ce453b64afac1fe7","unresolved":false,"context_lines":[{"line_number":403,"context_line":"            dc_path \u003d vutil.get_inventory_path(session.vim, vi.dc_info.ref)"},{"line_number":404,"context_line":""},{"line_number":405,"context_line":"            host \u003d self._session._host"},{"line_number":406,"context_line":"            port \u003d self._session._port"},{"line_number":407,"context_line":"            cookies \u003d session.vim.client.options.transport.cookiejar"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"        images.fetch_image("}],"source_content_type":"text/x-python","patch_set":6,"id":"bacf61ea_9840a109","line":406,"updated":"2016-08-02 23:38:54.000000000","message":"nit: session variable from L386 can be used here.","commit_id":"f69c7afbf2958d614cd3a6887d8b7c0ebd420946"}]}
