)]}'
{"rally/plugins/openstack/scenarios/cinder/volumes.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"59cc8fb2a48b5f995b605f62a82d31a522317267","unresolved":false,"context_lines":[{"line_number":588,"context_line":"        :param kwargs: optional args to create a volume"},{"line_number":589,"context_line":"        \"\"\""},{"line_number":590,"context_line":"        create_snapshot_kwargs \u003d create_snapshot_kwargs or {}"},{"line_number":591,"context_line":"        src_volume \u003d random.choice(self.context[\"tenant\"][\"volumes\"])"},{"line_number":592,"context_line":""},{"line_number":593,"context_line":"        snapshot \u003d self._create_snapshot(src_volume[\"id\"],"},{"line_number":594,"context_line":"                                         **create_snapshot_kwargs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_bfa6a7be","line":591,"updated":"2016-05-26 11:38:22.000000000","message":"You need to add required volume context validator here","commit_id":"7d511ca570c29e7b6a0210b34b196f788f5eddd1"},{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"a89d59db8c9837a08f1ace053fb44ab1b055d0b4","unresolved":false,"context_lines":[{"line_number":588,"context_line":"        :param kwargs: optional args to create a volume"},{"line_number":589,"context_line":"        \"\"\""},{"line_number":590,"context_line":"        create_snapshot_kwargs \u003d create_snapshot_kwargs or {}"},{"line_number":591,"context_line":"        src_volume \u003d random.choice(self.context[\"tenant\"][\"volumes\"])"},{"line_number":592,"context_line":""},{"line_number":593,"context_line":"        snapshot \u003d self._create_snapshot(src_volume[\"id\"],"},{"line_number":594,"context_line":"                                         **create_snapshot_kwargs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_e2b6d6da","line":591,"in_reply_to":"bab6814e_bfa6a7be","updated":"2016-05-26 12:00:01.000000000","message":"The validator is line-575. @validation.required_contexts(\"volumes\")","commit_id":"7d511ca570c29e7b6a0210b34b196f788f5eddd1"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"5042eb1497a83ff03f2012d8fd58812dcf7bffab","unresolved":false,"context_lines":[{"line_number":588,"context_line":"        :param kwargs: optional args to create a volume"},{"line_number":589,"context_line":"        \"\"\""},{"line_number":590,"context_line":"        create_snapshot_kwargs \u003d create_snapshot_kwargs or {}"},{"line_number":591,"context_line":"        src_volume \u003d random.choice(self.context[\"tenant\"][\"volumes\"])"},{"line_number":592,"context_line":""},{"line_number":593,"context_line":"        snapshot \u003d self._create_snapshot(src_volume[\"id\"],"},{"line_number":594,"context_line":"                                         **create_snapshot_kwargs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_85bef4ab","line":591,"in_reply_to":"bab6814e_e2b6d6da","updated":"2016-05-26 12:18:47.000000000","message":"Sorry for inattention","commit_id":"7d511ca570c29e7b6a0210b34b196f788f5eddd1"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"80a5c0ec503890de7a35e996d5576efd76e29f29","unresolved":false,"context_lines":[{"line_number":592,"context_line":"                                    **kwargs):"},{"line_number":593,"context_line":"        \"\"\"Create a volume-snapshot, then create a volume from this snapshot."},{"line_number":594,"context_line":""},{"line_number":595,"context_line":"        :param force: when set to True, allows snapshot of a volume when"},{"line_number":596,"context_line":"                      the volume is attached to an instance"},{"line_number":597,"context_line":"        :param do_delete: if True, a snapshot and a volume will"},{"line_number":598,"context_line":"                          be deleted after creation."}],"source_content_type":"text/x-python","patch_set":4,"id":"9abb7d3a_ad824f50","line":595,"updated":"2016-05-30 11:18:07.000000000","message":"This parameter is actually missed in arguments","commit_id":"ef8382e8b37537d1e09e21b343c80c309f7ae845"},{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"dee77f53ede44b2e0d122a76034adfd509f70588","unresolved":false,"context_lines":[{"line_number":592,"context_line":"                                    **kwargs):"},{"line_number":593,"context_line":"        \"\"\"Create a volume-snapshot, then create a volume from this snapshot."},{"line_number":594,"context_line":""},{"line_number":595,"context_line":"        :param force: when set to True, allows snapshot of a volume when"},{"line_number":596,"context_line":"                      the volume is attached to an instance"},{"line_number":597,"context_line":"        :param do_delete: if True, a snapshot and a volume will"},{"line_number":598,"context_line":"                          be deleted after creation."}],"source_content_type":"text/x-python","patch_set":4,"id":"9abb7d3a_f39f1601","line":595,"in_reply_to":"9abb7d3a_ad824f50","updated":"2016-05-30 12:11:27.000000000","message":"Done","commit_id":"ef8382e8b37537d1e09e21b343c80c309f7ae845"}],"tests/unit/plugins/openstack/scenarios/cinder/test_volumes.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"80a5c0ec503890de7a35e996d5576efd76e29f29","unresolved":false,"context_lines":[{"line_number":553,"context_line":"            \"uuid\", **create_snapshot_args)"},{"line_number":554,"context_line":"        scenario._create_volume.assert_called_once_with("},{"line_number":555,"context_line":"            1, snapshot_id\u003dfake_snapshot.id, fakearg\u003d\"f\")"},{"line_number":556,"context_line":"        scenario._delete_snapshot.assert_called_once_with(fake_snapshot)"},{"line_number":557,"context_line":"        scenario._delete_volume.assert_called_once_with(fake_volume)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9abb7d3a_2da49fa4","line":556,"updated":"2016-05-30 11:18:07.000000000","message":"These two assertions (mocks _delete_snapshot and _delete_volume) should be placed at L542 because they are actually related to L537, then these two mocks should be reset and here at L556 they should be checked that they are not called (because we have do_delete\u003dFalse at L547)","commit_id":"ef8382e8b37537d1e09e21b343c80c309f7ae845"},{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"dee77f53ede44b2e0d122a76034adfd509f70588","unresolved":false,"context_lines":[{"line_number":553,"context_line":"            \"uuid\", **create_snapshot_args)"},{"line_number":554,"context_line":"        scenario._create_volume.assert_called_once_with("},{"line_number":555,"context_line":"            1, snapshot_id\u003dfake_snapshot.id, fakearg\u003d\"f\")"},{"line_number":556,"context_line":"        scenario._delete_snapshot.assert_called_once_with(fake_snapshot)"},{"line_number":557,"context_line":"        scenario._delete_volume.assert_called_once_with(fake_volume)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9abb7d3a_ce298939","line":556,"in_reply_to":"9abb7d3a_2da49fa4","updated":"2016-05-30 12:11:27.000000000","message":"Done. Thank you.","commit_id":"ef8382e8b37537d1e09e21b343c80c309f7ae845"}]}