)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"edf9d5f0e0beb914a026c561ae47558bf8cec08b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Johnson koil raj \u003cjohnson.raj@hp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-01-31 16:15:51 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"VMware: Set target Resource pool for Volume clone"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"A volume is cloned during create volume from another volume, snapshot"},{"line_number":10,"context_line":"or image.Currently, the target resource pool is unset while calling"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"da86d52c_fdecc6c9","line":7,"updated":"2015-02-02 12:34:51.000000000","message":"Maybe a more informative first line of the commit message:\n\n VMware: Fix missing target resource pool","commit_id":"12a001f10e722768581ea2567aedcfa0be6439bc"},{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"edf9d5f0e0beb914a026c561ae47558bf8cec08b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"VMware: Set target Resource pool for Volume clone"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"A volume is cloned during create volume from another volume, snapshot"},{"line_number":10,"context_line":"or image.Currently, the target resource pool is unset while calling"},{"line_number":11,"context_line":"vCenter CloneVM_Task API."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"da86d52c_1df8ba7b","line":9,"updated":"2015-02-02 12:34:51.000000000","message":"Better rewording:\n\n The volume is cloned while creating it from another volume, snapshot or image.","commit_id":"12a001f10e722768581ea2567aedcfa0be6439bc"},{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"edf9d5f0e0beb914a026c561ae47558bf8cec08b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"VMware: Set target Resource pool for Volume clone"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"A volume is cloned during create volume from another volume, snapshot"},{"line_number":10,"context_line":"or image.Currently, the target resource pool is unset while calling"},{"line_number":11,"context_line":"vCenter CloneVM_Task API."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"So when a clone happens accross clusters in a vCenter clone fails."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"da86d52c_3df57e83","line":10,"updated":"2015-02-02 12:34:51.000000000","message":"Nit: space before \u0027Currently\u0027","commit_id":"12a001f10e722768581ea2567aedcfa0be6439bc"},{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"edf9d5f0e0beb914a026c561ae47558bf8cec08b","unresolved":false,"context_lines":[{"line_number":10,"context_line":"or image.Currently, the target resource pool is unset while calling"},{"line_number":11,"context_line":"vCenter CloneVM_Task API."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"So when a clone happens accross clusters in a vCenter clone fails."},{"line_number":14,"context_line":"This patch fixes the problem by setting the target resource pool"},{"line_number":15,"context_line":"(returned by datastore selection logic) while invoking clone."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"da86d52c_bd308e57","line":13,"updated":"2015-02-02 12:34:51.000000000","message":"Typo: across","commit_id":"12a001f10e722768581ea2567aedcfa0be6439bc"}],"cinder/volume/drivers/vmware/volumeops.py":[{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"f9d357e11a7356dfa250efc960dbb889731797cd","unresolved":false,"context_lines":[{"line_number":1001,"context_line":""},{"line_number":1002,"context_line":"    def _get_clone_spec(self, datastore, disk_move_type, snapshot, backing,"},{"line_number":1003,"context_line":"                        disk_type, host\u003dNone, resource_pool\u003dNone):"},{"line_number":1004,"context_line":"        \"\"\"Get the clone spec."},{"line_number":1005,"context_line":""},{"line_number":1006,"context_line":"        :param datastore: Reference to datastore"},{"line_number":1007,"context_line":"        :param disk_move_type: Disk move type"}],"source_content_type":"text/x-python","patch_set":7,"id":"baa041b7_a04e046d","line":1004,"updated":"2015-04-06 04:34:38.000000000","message":"JFYI, Docstring specification say that the comment should not end with a fullstop http://docs.ckan.org/en/latest/contributing/python.html","commit_id":"0df929e60c9053240b792dea7ed7e2c10de69fc1"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"4a80d4ef513a88f076a2d6c99b06ad95a17d8466","unresolved":false,"context_lines":[{"line_number":1001,"context_line":""},{"line_number":1002,"context_line":"    def _get_clone_spec(self, datastore, disk_move_type, snapshot, backing,"},{"line_number":1003,"context_line":"                        disk_type, host\u003dNone, resource_pool\u003dNone):"},{"line_number":1004,"context_line":"        \"\"\"Get the clone spec."},{"line_number":1005,"context_line":""},{"line_number":1006,"context_line":"        :param datastore: Reference to datastore"},{"line_number":1007,"context_line":"        :param disk_move_type: Disk move type"}],"source_content_type":"text/x-python","patch_set":7,"id":"7aaa499b_b4cead81","line":1004,"in_reply_to":"baa041b7_a04e046d","updated":"2015-04-11 03:57:14.000000000","message":"That is not what is documented in the OpenStack hacking rules:\n\nhttp://docs.openstack.org/developer/hacking/#docstrings\n\nAccording to this, the first line of a multiple lines docstring should end with a period.","commit_id":"0df929e60c9053240b792dea7ed7e2c10de69fc1"}]}