)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11418,"name":"Shay Halsband","email":"shay.halsband@emc.com","username":"shyh"},"change_message_id":"976788749111fd00363a5c4b20118a898fd9814b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"XtremIO: extending volume in create_volume_from_snapshot"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In create_volume_from_snapshot, If the original volume is larger then the requested snapshot we extend the volume to the original size."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #XtremIO: fix fetching FC targets from X2 array"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f515b1d_b1a9be90","line":9,"updated":"2017-10-02 06:54:45.000000000","message":"you should split this line to fit the width limit","commit_id":"af4f64eae209b04949f5bf6be5106fb447842829"},{"author":{"_account_id":11418,"name":"Shay Halsband","email":"shay.halsband@emc.com","username":"shyh"},"change_message_id":"976788749111fd00363a5c4b20118a898fd9814b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In create_volume_from_snapshot, If the original volume is larger then the requested snapshot we extend the volume to the original size."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #XtremIO: fix fetching FC targets from X2 array"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Previous versions of XtremIO had the type as part of the target name, in"},{"line_number":14,"context_line":"X2 this convention was changed and a driver updated is required to query"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f515b1d_11989272","line":11,"range":{"start_line":11,"start_character":11,"end_line":11,"end_character":59},"updated":"2017-10-02 06:54:45.000000000","message":"specify the bug number not it\u0027s title","commit_id":"af4f64eae209b04949f5bf6be5106fb447842829"},{"author":{"_account_id":11418,"name":"Shay Halsband","email":"shay.halsband@emc.com","username":"shyh"},"change_message_id":"976788749111fd00363a5c4b20118a898fd9814b","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #XtremIO: fix fetching FC targets from X2 array"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Previous versions of XtremIO had the type as part of the target name, in"},{"line_number":14,"context_line":"X2 this convention was changed and a driver updated is required to query"},{"line_number":15,"context_line":"the type from the target object."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I1a1c3a8ff3fb0196e8bffb9a4978e7368eeeacec"},{"line_number":18,"context_line":"Closes-Bug: #1702458"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f515b1d_7197c649","line":18,"range":{"start_line":13,"start_character":2,"end_line":18,"end_character":20},"updated":"2017-10-02 06:54:45.000000000","message":"this isn\u0027t relevant, make sure you didn\u0027t ammend and overwritten a previose commit","commit_id":"af4f64eae209b04949f5bf6be5106fb447842829"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"e8ad25aac006e75bfc21e547ed510b50cc4ce682","unresolved":false,"context_lines":[{"line_number":7,"context_line":"XtremIO: extending volume in create_volume_from_snapshot"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In create_volume_from_snapshot, If the original volume is larger"},{"line_number":10,"context_line":"then the requested snapshot we extend the volume to the original"},{"line_number":11,"context_line":"size."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1560650"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3f4b6375_d81e2b73","line":10,"updated":"2017-10-31 20:49:02.000000000","message":"This description is actually a little turned around, but the code appears to be correct.","commit_id":"50a4c376811c18684c8cada7a0c0c0cc39696942"}],"cinder/tests/unit/volume/drivers/dell_emc/test_xtremio.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"06d138727165163c2be37bd2c4d535c66ed76803","unresolved":false,"context_lines":[{"line_number":449,"context_line":"        self.driver.create_snapshot(self.data.test_snapshot)"},{"line_number":450,"context_line":"        self.driver.create_volume_from_snapshot(self.data.test_volume2,"},{"line_number":451,"context_line":"                                                self.data.test_snapshot)"},{"line_number":452,"context_line":""},{"line_number":453,"context_line":"    def test_volume_from_snapshot_and_resize(self, req):"},{"line_number":454,"context_line":"        req.side_effect \u003d xms_request"},{"line_number":455,"context_line":"        xms_data[\u0027volumes\u0027] \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f4b6375_59f3ec74","line":452,"updated":"2017-10-25 22:44:57.000000000","message":"Need a bad path test case as well.","commit_id":"89fb0b1653ac2c24ee0fb1c1f909bbbdd520e811"}],"cinder/volume/drivers/dell_emc/xtremio.py":[{"author":{"_account_id":11418,"name":"Shay Halsband","email":"shay.halsband@emc.com","username":"shyh"},"change_message_id":"976788749111fd00363a5c4b20118a898fd9814b","unresolved":false,"context_lines":[{"line_number":491,"context_line":"        self.client.create_snapshot(snapshot_id, volume[\u0027id\u0027])"},{"line_number":492,"context_line":""},{"line_number":493,"context_line":"        # resolve snapshot volume."},{"line_number":494,"context_line":"        try:"},{"line_number":495,"context_line":"            snapshot_vol \u003d self.client.req(\u0027volumes\u0027, name\u003dvolume[\u0027id\u0027])[\u0027content\u0027]"},{"line_number":496,"context_line":"        except Exception:"},{"line_number":497,"context_line":"            LOG.error(\u0027fails to resolve volume %s, reverting.\u0027)"},{"line_number":498,"context_line":"            self.delete_volume(volume)"},{"line_number":499,"context_line":"            raise"},{"line_number":500,"context_line":""},{"line_number":501,"context_line":"        # extend the snapped volume if requested size is larger then original"},{"line_number":502,"context_line":"        if volume[\u0027size\u0027] \u003e snapshot[\u0027volume_size\u0027]:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_f1a3b6b0","line":499,"range":{"start_line":494,"start_character":8,"end_line":499,"end_character":17},"updated":"2017-10-02 06:54:45.000000000","message":"what is this code for?","commit_id":"af4f64eae209b04949f5bf6be5106fb447842829"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"06d138727165163c2be37bd2c4d535c66ed76803","unresolved":false,"context_lines":[{"line_number":495,"context_line":"            try:"},{"line_number":496,"context_line":"                self.extend_volume(volume, volume[\u0027size\u0027])"},{"line_number":497,"context_line":"            except Exception:"},{"line_number":498,"context_line":"                LOG.error(\u0027failes to extend volume %s, \u0027"},{"line_number":499,"context_line":"                          \u0027reverting volume from snapshot operation\u0027,"},{"line_number":500,"context_line":"                          volume[\u0027id\u0027])"},{"line_number":501,"context_line":"                # remove the volume in case resize failed"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f4b6375_794908b5","line":498,"range":{"start_line":498,"start_character":27,"end_line":498,"end_character":33},"updated":"2017-10-25 22:44:57.000000000","message":"Failed","commit_id":"89fb0b1653ac2c24ee0fb1c1f909bbbdd520e811"}]}