)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"e2d8e1f4d8117c6e212ba49fe058b511f7ee6189","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Positive tests for the cinder CLI commands which check actions with"},{"line_number":10,"context_line":"volume create command like create volume from snapshot,"},{"line_number":11,"context_line":"create volume from volume, creata volume from image etc."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I77912d413ac061eb8376233dfef772c55265d135"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bacf61ea_6fddcc41","line":11,"range":{"start_line":11,"start_character":41,"end_line":11,"end_character":51},"updated":"2016-08-04 12:16:28.000000000","message":"Tests volumes from image will be placed in current patch?","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"e2d8e1f4d8117c6e212ba49fe058b511f7ee6189","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Positive tests for the cinder CLI commands which check actions with"},{"line_number":10,"context_line":"volume create command like create volume from snapshot,"},{"line_number":11,"context_line":"create volume from volume, creata volume from image etc."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I77912d413ac061eb8376233dfef772c55265d135"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bacf61ea_4fe0c8fa","line":11,"range":{"start_line":11,"start_character":27,"end_line":11,"end_character":33},"updated":"2016-08-04 12:16:28.000000000","message":"create","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"},{"author":{"_account_id":20675,"name":"Sofiia Andriichenko","email":"sofiia.andriichenko@gmail.com","username":"SofiiaAndriichenko"},"change_message_id":"32fde72af5e54599f7bff1f2d4a78cf66f433b37","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Positive tests for the cinder CLI commands which check actions with"},{"line_number":10,"context_line":"volume create command like create volume from snapshot,"},{"line_number":11,"context_line":"create volume from volume, creata volume from image etc."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I77912d413ac061eb8376233dfef772c55265d135"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9ad45d7e_13830fce","line":11,"range":{"start_line":11,"start_character":41,"end_line":11,"end_character":51},"in_reply_to":"9ad45d7e_d3d21734","updated":"2016-08-08 11:49:23.000000000","message":"Done","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"a33e447123121badc6207998b0cb3adb55f2f87a","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Positive tests for the cinder CLI commands which check actions with"},{"line_number":10,"context_line":"volume create command like create volume from snapshot,"},{"line_number":11,"context_line":"create volume from volume, creata volume from image etc."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I77912d413ac061eb8376233dfef772c55265d135"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9ad45d7e_d3d21734","line":11,"range":{"start_line":11,"start_character":41,"end_line":11,"end_character":51},"in_reply_to":"bacf61ea_177ea569","updated":"2016-08-08 11:43:55.000000000","message":"Than you should remove mention about them from commit message.","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"},{"author":{"_account_id":20675,"name":"Sofiia Andriichenko","email":"sofiia.andriichenko@gmail.com","username":"SofiiaAndriichenko"},"change_message_id":"a3c4ced08db8352b29f329b1bb5280da0f60fea7","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Positive tests for the cinder CLI commands which check actions with"},{"line_number":10,"context_line":"volume create command like create volume from snapshot,"},{"line_number":11,"context_line":"create volume from volume, creata volume from image etc."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I77912d413ac061eb8376233dfef772c55265d135"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bacf61ea_177ea569","line":11,"range":{"start_line":11,"start_character":41,"end_line":11,"end_character":51},"in_reply_to":"bacf61ea_6fddcc41","updated":"2016-08-08 11:17:00.000000000","message":"No, tests volumes from image will not be placed in current patch","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"}],"cinderclient/tests/functional/test_volume_create_cli.py":[{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"e2d8e1f4d8117c6e212ba49fe058b511f7ee6189","unresolved":false,"context_lines":[{"line_number":41,"context_line":"class CinderVolumeTests(base.ClientTestBase):"},{"line_number":42,"context_line":"    \"\"\"Check of cinder volume create commands.\"\"\""},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def test_volume_create_from_snapshot(self):"},{"line_number":45,"context_line":"        volume \u003d self.object_create(\u0027volume\u0027, params\u003d\u00271\u0027)"},{"line_number":46,"context_line":"        snapshot \u003d self.object_create(\u0027snapshot\u0027, params\u003dvolume[\u0027id\u0027])"},{"line_number":47,"context_line":"        volume_from_snapshot \u003d self.object_create(\u0027volume\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_6ff30c0b","line":44,"updated":"2016-08-04 12:16:28.000000000","message":"Please add docstring describing test step by step.","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"},{"author":{"_account_id":20675,"name":"Sofiia Andriichenko","email":"sofiia.andriichenko@gmail.com","username":"SofiiaAndriichenko"},"change_message_id":"a3c4ced08db8352b29f329b1bb5280da0f60fea7","unresolved":false,"context_lines":[{"line_number":41,"context_line":"class CinderVolumeTests(base.ClientTestBase):"},{"line_number":42,"context_line":"    \"\"\"Check of cinder volume create commands.\"\"\""},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def test_volume_create_from_snapshot(self):"},{"line_number":45,"context_line":"        volume \u003d self.object_create(\u0027volume\u0027, params\u003d\u00271\u0027)"},{"line_number":46,"context_line":"        snapshot \u003d self.object_create(\u0027snapshot\u0027, params\u003dvolume[\u0027id\u0027])"},{"line_number":47,"context_line":"        volume_from_snapshot \u003d self.object_create(\u0027volume\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad45d7e_bc62d514","line":44,"in_reply_to":"bacf61ea_6ff30c0b","updated":"2016-08-08 11:17:00.000000000","message":"Done","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"e2d8e1f4d8117c6e212ba49fe058b511f7ee6189","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        cinder_list \u003d self.cinder(\u0027list\u0027)"},{"line_number":56,"context_line":"        self.assertIn(volume_from_snapshot[\u0027id\u0027], cinder_list)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def test_volume_create_from_volume(self):"},{"line_number":59,"context_line":"        volume \u003d self.object_create(\u0027volume\u0027, params\u003d\u00271\u0027)"},{"line_number":60,"context_line":"        volume_from_volume \u003d self.object_create(\u0027volume\u0027,"},{"line_number":61,"context_line":"                                         params\u003d\u0027--source-volid {0} 1\u0027."}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_4fcd68b6","line":58,"updated":"2016-08-04 12:16:28.000000000","message":"And here docstring with steps too.","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"},{"author":{"_account_id":20675,"name":"Sofiia Andriichenko","email":"sofiia.andriichenko@gmail.com","username":"SofiiaAndriichenko"},"change_message_id":"a3c4ced08db8352b29f329b1bb5280da0f60fea7","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        cinder_list \u003d self.cinder(\u0027list\u0027)"},{"line_number":56,"context_line":"        self.assertIn(volume_from_snapshot[\u0027id\u0027], cinder_list)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def test_volume_create_from_volume(self):"},{"line_number":59,"context_line":"        volume \u003d self.object_create(\u0027volume\u0027, params\u003d\u00271\u0027)"},{"line_number":60,"context_line":"        volume_from_volume \u003d self.object_create(\u0027volume\u0027,"},{"line_number":61,"context_line":"                                         params\u003d\u0027--source-volid {0} 1\u0027."}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad45d7e_9c671125","line":58,"in_reply_to":"bacf61ea_4fcd68b6","updated":"2016-08-08 11:17:00.000000000","message":"Done","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"e2d8e1f4d8117c6e212ba49fe058b511f7ee6189","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        self.assertIn(volume_from_snapshot[\u0027id\u0027], cinder_list)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def test_volume_create_from_volume(self):"},{"line_number":59,"context_line":"        volume \u003d self.object_create(\u0027volume\u0027, params\u003d\u00271\u0027)"},{"line_number":60,"context_line":"        volume_from_volume \u003d self.object_create(\u0027volume\u0027,"},{"line_number":61,"context_line":"                                         params\u003d\u0027--source-volid {0} 1\u0027."},{"line_number":62,"context_line":"                                         format(volume[\u0027id\u0027]))"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_cf6798b5","line":59,"range":{"start_line":59,"start_character":8,"end_line":59,"end_character":57},"updated":"2016-08-04 12:16:28.000000000","message":"Possibly better to move initial volume creation to setUp() method.","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"},{"author":{"_account_id":20675,"name":"Sofiia Andriichenko","email":"sofiia.andriichenko@gmail.com","username":"SofiiaAndriichenko"},"change_message_id":"a3c4ced08db8352b29f329b1bb5280da0f60fea7","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        self.assertIn(volume_from_snapshot[\u0027id\u0027], cinder_list)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def test_volume_create_from_volume(self):"},{"line_number":59,"context_line":"        volume \u003d self.object_create(\u0027volume\u0027, params\u003d\u00271\u0027)"},{"line_number":60,"context_line":"        volume_from_volume \u003d self.object_create(\u0027volume\u0027,"},{"line_number":61,"context_line":"                                         params\u003d\u0027--source-volid {0} 1\u0027."},{"line_number":62,"context_line":"                                         format(volume[\u0027id\u0027]))"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad45d7e_dc7189e2","line":59,"range":{"start_line":59,"start_character":8,"end_line":59,"end_character":57},"in_reply_to":"bacf61ea_cf6798b5","updated":"2016-08-08 11:17:00.000000000","message":"Done","commit_id":"92fe5398abeeb7596736f3bc90dc01b805061463"},{"author":{"_account_id":17270,"name":"Sergii Turivnyi","email":"sturivnyi@mirantis.com","username":"sturivnyi"},"change_message_id":"72b27debc29c0e97e0a9631f3348aed868d1c5c6","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    def test_volume_create_from_snapshot(self):"},{"line_number":48,"context_line":"        \u0027\u0027\u0027Test steps:"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"        1) create volume"},{"line_number":51,"context_line":"        2) create snapshot"},{"line_number":52,"context_line":"        3) create volume from snapshot"},{"line_number":53,"context_line":"        4) check that volume from snapshot has been successfully created"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_d05fad73","line":50,"range":{"start_line":50,"start_character":11,"end_line":50,"end_character":24},"updated":"2016-08-08 11:30:39.000000000","message":"I feel would be better to add `in Setup()`. Something like that: 1) create volume in Setup()","commit_id":"7ed4d1f448687ded23d43c7f6e010532e0b232f1"},{"author":{"_account_id":20675,"name":"Sofiia Andriichenko","email":"sofiia.andriichenko@gmail.com","username":"SofiiaAndriichenko"},"change_message_id":"496ba9fb9c8a9af3beaeb615d199a509da54357f","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    def test_volume_create_from_snapshot(self):"},{"line_number":48,"context_line":"        \u0027\u0027\u0027Test steps:"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"        1) create volume"},{"line_number":51,"context_line":"        2) create snapshot"},{"line_number":52,"context_line":"        3) create volume from snapshot"},{"line_number":53,"context_line":"        4) check that volume from snapshot has been successfully created"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_130caf93","line":50,"range":{"start_line":50,"start_character":11,"end_line":50,"end_character":24},"in_reply_to":"9ad45d7e_d05fad73","updated":"2016-08-08 11:49:32.000000000","message":"Done","commit_id":"7ed4d1f448687ded23d43c7f6e010532e0b232f1"},{"author":{"_account_id":17270,"name":"Sergii Turivnyi","email":"sturivnyi@mirantis.com","username":"sturivnyi"},"change_message_id":"72b27debc29c0e97e0a9631f3348aed868d1c5c6","unresolved":false,"context_lines":[{"line_number":64,"context_line":"    def test_volume_create_from_volume(self):"},{"line_number":65,"context_line":"        \u0027\u0027\u0027Test steps:"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        1) create volume"},{"line_number":68,"context_line":"        2) create volume from volume"},{"line_number":69,"context_line":"        3) check that volume from volume has been successfully created"},{"line_number":70,"context_line":"        \u0027\u0027\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_105ae583","line":67,"range":{"start_line":67,"start_character":18,"end_line":67,"end_character":24},"updated":"2016-08-08 11:30:39.000000000","message":"ditto","commit_id":"7ed4d1f448687ded23d43c7f6e010532e0b232f1"},{"author":{"_account_id":20675,"name":"Sofiia Andriichenko","email":"sofiia.andriichenko@gmail.com","username":"SofiiaAndriichenko"},"change_message_id":"496ba9fb9c8a9af3beaeb615d199a509da54357f","unresolved":false,"context_lines":[{"line_number":64,"context_line":"    def test_volume_create_from_volume(self):"},{"line_number":65,"context_line":"        \u0027\u0027\u0027Test steps:"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        1) create volume"},{"line_number":68,"context_line":"        2) create volume from volume"},{"line_number":69,"context_line":"        3) check that volume from volume has been successfully created"},{"line_number":70,"context_line":"        \u0027\u0027\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_3323b307","line":67,"range":{"start_line":67,"start_character":18,"end_line":67,"end_character":24},"in_reply_to":"9ad45d7e_105ae583","updated":"2016-08-08 11:49:32.000000000","message":"Done","commit_id":"7ed4d1f448687ded23d43c7f6e010532e0b232f1"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"647dbddbf56e5f32582fa8a4c47a2d9cedeeabb3","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        self.volume \u003d self.object_create(\u0027volume\u0027, params\u003d\u00271\u0027)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def test_volume_create_from_snapshot(self):"},{"line_number":48,"context_line":"        \u0027\u0027\u0027Test steps:"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"        1) create volume in Setup()"},{"line_number":51,"context_line":"        2) create snapshot"}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_6600cbab","line":48,"range":{"start_line":48,"start_character":8,"end_line":48,"end_character":11},"updated":"2016-08-08 11:59:58.000000000","message":"Triple double-quotes should be here.","commit_id":"f0480148a384bf4395a5b1d835e26bc9f9b1cd16"}]}