)]}'
{"cinder/volume/drivers/emc/emc_vnx_cli.py":[{"author":{"_account_id":14274,"name":"XinXiaohui","email":"xiaohui.xin@intel.com","username":"XinXiaohui"},"change_message_id":"ba62b9d71d5676c38afdc6036031c4081d4d6ef5","unresolved":false,"context_lines":[{"line_number":2500,"context_line":"                                    \"%(out)s\"),"},{"line_number":2501,"context_line":"                                {\u0027name\u0027: volume[\u0027name\u0027], \u0027out\u0027: out})"},{"line_number":2502,"context_line":"                    self._client._wait_for_a_condition(self._client.wait_until_lun_ready_for_ops,"},{"line_number":2503,"context_line":"                                                       name\u003dvolume[\u0027name\u0027])"},{"line_number":2504,"context_line":"                    self._client.expand_lun_and_wait(volume[\u0027name\u0027], new_size)"},{"line_number":2505,"context_line":"                    ctxt.reraise \u003d False"},{"line_number":2506,"context_line":"                else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"dae33548_3bdd2415","line":2503,"range":{"start_line":2503,"start_character":68,"end_line":2503,"end_character":72},"updated":"2016-02-15 09:08:48.000000000","message":"The line is too long. Maybe it\u0027s the reason for your jenkin failure. Please try \"tox -e pep8\" to fix the issue.","commit_id":"74ad3e47f95d7a62c5f2c4b429d82548cfd8d952"},{"author":{"_account_id":10439,"name":"Xi Yang","email":"yang.xi@99cloud.net","username":"phenom"},"change_message_id":"70258b1343af03db8d4272d69fef89bd0fa91a54","unresolved":false,"context_lines":[{"line_number":1565,"context_line":"        return data[VNXLunProperties.LUN_OPERATION.key]"},{"line_number":1566,"context_line":""},{"line_number":1567,"context_line":"    def wait_until_lun_ready_for_ops(self, name):"},{"line_number":1568,"context_line":"        data \u003d self.get_lun_current_ops_state(name, poll\u003dTrue)"},{"line_number":1569,"context_line":"        return data \u003d\u003d \u0027None\u0027"},{"line_number":1570,"context_line":""},{"line_number":1571,"context_line":"    def get_pool(self,"}],"source_content_type":"text/x-python","patch_set":2,"id":"dae33548_b6275911","line":1568,"range":{"start_line":1568,"start_character":0,"end_line":1568,"end_character":8},"updated":"2016-02-16 09:08:36.000000000","message":"I believe you should consult VNX guys to do this change.\nThe code can not be removed, because the function \"is_lun_ready_for_ops\" is using polling\u003dFalse to get the lun\u0027s operation status and you change it to polling\u003dTrue. This will cause pefformance issue.","commit_id":"d3fcb7e2c2078ea2b647ec793688f750fab79946"}]}