)]}'
{"id":"openstack%2Fpython-cinderclient~478447","triplet_id":"openstack%2Fpython-cinderclient~master~I2056b8d94bbf29d2f6943de7612954147a0e8cde","project":"openstack/python-cinderclient","branch":"master","topic":"bp/volume-delete-parameters","hashtags":[],"change_id":"I2056b8d94bbf29d2f6943de7612954147a0e8cde","subject":"Support \"--force\" param to delete volume in v3.23","status":"ABANDONED","created":"2017-06-28 09:19:59.000000000","updated":"2018-01-18 00:13:30.000000000","total_comment_count":17,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"6297f56f1042665c42f36b7a2d122026782bcda6","_number":478447,"virtual_id_number":478447,"owner":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"date":"2017-11-07 14:19:03.000000000","_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":1,"date":"2017-10-26 03:43:57.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":25388,"name":"liuyamin","email":"liuyamin@fiberhome.com","username":"liuyamin"},{"date":"2018-01-18 00:13:11.000000000","_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":20382,"name":"YuYang","email":"yuyangwang1985@gmail.com","username":"YuYangWang"},{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"disliked":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"all":[{"value":0,"date":"2017-10-26 13:25:23.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":-1,"date":"2017-11-16 23:37:20.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2017-11-03 10:56:06.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"value":-1,"date":"2017-11-18 00:53:00.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":25388,"name":"liuyamin","email":"liuyamin@fiberhome.com","username":"liuyamin"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},{"value":-1,"date":"2017-11-29 23:16:49.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":20382,"name":"YuYang","email":"yuyangwang1985@gmail.com","username":"YuYangWang"},{"value":-1,"date":"2017-11-07 10:16:44.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":25388,"name":"liuyamin","email":"liuyamin@fiberhome.com","username":"liuyamin"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":20382,"name":"YuYang","email":"yuyangwang1985@gmail.com","username":"YuYangWang"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":25388,"name":"liuyamin","email":"liuyamin@fiberhome.com","username":"liuyamin"},{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":20382,"name":"YuYang","email":"yuyangwang1985@gmail.com","username":"YuYangWang"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":20382,"name":"YuYang","email":"yuyangwang1985@gmail.com","username":"YuYangWang"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"_account_id":25388,"name":"liuyamin","email":"liuyamin@fiberhome.com","username":"liuyamin"},{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-07-05 06:01:05.000000000","updated_by":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"reviewer":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"state":"REVIEWER"},{"updated":"2017-07-05 06:16:43.000000000","updated_by":{"_account_id":20382,"name":"YuYang","email":"yuyangwang1985@gmail.com","username":"YuYangWang"},"reviewer":{"_account_id":20382,"name":"YuYang","email":"yuyangwang1985@gmail.com","username":"YuYangWang"},"state":"REVIEWER"},{"updated":"2017-07-26 05:53:57.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2017-08-08 08:30:11.000000000","updated_by":{"_account_id":25388,"name":"liuyamin","email":"liuyamin@fiberhome.com","username":"liuyamin"},"reviewer":{"_account_id":25388,"name":"liuyamin","email":"liuyamin@fiberhome.com","username":"liuyamin"},"state":"REVIEWER"},{"updated":"2017-10-26 03:43:57.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2017-10-26 13:25:23.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2017-11-03 10:56:06.000000000","updated_by":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"reviewer":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"state":"REVIEWER"},{"updated":"2017-11-07 10:16:44.000000000","updated_by":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"reviewer":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"state":"REVIEWER"},{"updated":"2017-11-07 14:19:03.000000000","updated_by":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"reviewer":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"state":"REVIEWER"},{"updated":"2017-11-16 23:37:20.000000000","updated_by":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"reviewer":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"state":"REVIEWER"},{"updated":"2017-11-18 00:53:00.000000000","updated_by":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"reviewer":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"state":"REVIEWER"},{"updated":"2017-11-29 23:16:49.000000000","updated_by":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"reviewer":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"state":"REVIEWER"}],"messages":[{"id":"7ae76b60ad07363e3508d6601d4ea20ac9014293","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-06-28 09:19:59.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"2ddbc19b59649231cf80a0e66320b8daf3a24fcd","author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"date":"2017-06-28 10:00:57.000000000","message":"Patch Set 1: Code-Review-1\n\n(4 comments)","accounts_in_message":[],"_revision_number":1},{"id":"9ddd65c6b8633fab9f42cb87c0cf3f6a354f060a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-06-28 10:40:07.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/47/478447/1/check/gate-python-cinderclient-docs-ubuntu-xenial/bc51c4e//doc/build/html/ : SUCCESS in 3m 11s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/47/478447/1/check/gate-python-cinderclient-pep8-ubuntu-xenial/a4770af/ : SUCCESS in 4m 17s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/47/478447/1/check/gate-python-cinderclient-python27-ubuntu-xenial/246f68d/ : SUCCESS in 3m 21s\n- gate-python-cinderclient-python35 http://logs.openstack.org/47/478447/1/check/gate-python-cinderclient-python35/01aeb17/ : SUCCESS in 3m 16s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/47/478447/1/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/579f51a/ : SUCCESS in 1h 17m 33s\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/47/478447/1/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/f8a6807/ : SUCCESS in 16m 12s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/47/478447/1/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/7f3fa1d/ : SUCCESS in 23m 02s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/47/478447/1/check/gate-python-cinderclient-pylint-ubuntu-xenial/b25ae86/ : FAILURE in 4m 29s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"72907367765bb3d4c8288eb3b508654422d55999","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-06-28 14:23:20.000000000","message":"Patch Set 1: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":1},{"id":"d7deb53b4228b1f6936fb25349a0cfc4f12643ab","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-06-28 14:51:01.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"4fce7bcadadf64e29bf1f45c16d63ddc47e3d815","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-06-28 19:40:52.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"2426b479de14e37013c4cc3b7160fcfeeb1dd0cd","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-06-29 08:19:25.000000000","message":"Patch Set 2: Published edit on patch set 1","accounts_in_message":[],"_revision_number":2},{"id":"6a6f8f14b42dc291d060fb28a6f448eb2a9f9836","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-06-29 09:29:59.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/47/478447/2/check/gate-python-cinderclient-docs-ubuntu-xenial/6ba057a//doc/build/html/ : SUCCESS in 4m 30s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/47/478447/2/check/gate-python-cinderclient-pep8-ubuntu-xenial/4dcedc0/ : SUCCESS in 3m 59s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/47/478447/2/check/gate-python-cinderclient-python27-ubuntu-xenial/e22a90b/ : FAILURE in 4m 02s\n- gate-python-cinderclient-python35 http://logs.openstack.org/47/478447/2/check/gate-python-cinderclient-python35/1974427/ : FAILURE in 4m 30s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/47/478447/2/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/6df5e9a/ : SUCCESS in 1h 09m 07s\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/47/478447/2/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/e29ab6c/ : SUCCESS in 15m 29s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/47/478447/2/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/c6735fd/ : SUCCESS in 22m 28s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/47/478447/2/check/gate-python-cinderclient-pylint-ubuntu-xenial/fc16d3f/ : FAILURE in 3m 35s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"72f20ec7c59b29a6d893942a76d10591802f3bb6","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-06-29 09:35:36.000000000","message":"Patch Set 3: Published edit on patch set 2","accounts_in_message":[],"_revision_number":3},{"id":"92971e7bdce7043a08d3e05a7f6c942efc16bb07","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-06-29 10:58:33.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/47/478447/3/check/gate-python-cinderclient-docs-ubuntu-xenial/3e8bafc//doc/build/html/ : SUCCESS in 4m 02s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/47/478447/3/check/gate-python-cinderclient-pep8-ubuntu-xenial/1ba8788/ : SUCCESS in 4m 38s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/47/478447/3/check/gate-python-cinderclient-python27-ubuntu-xenial/0dd8324/ : FAILURE in 5m 53s\n- gate-python-cinderclient-python35 http://logs.openstack.org/47/478447/3/check/gate-python-cinderclient-python35/302fd67/ : SUCCESS in 3m 56s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/47/478447/3/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/fa152d7/ : SUCCESS in 1h 18m 11s\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/47/478447/3/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/d76849f/ : SUCCESS in 25m 10s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/47/478447/3/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/ee63b3a/ : SUCCESS in 27m 35s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/47/478447/3/check/gate-python-cinderclient-pylint-ubuntu-xenial/3da33a3/ : FAILURE in 4m 16s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"2a5b076ec0958ea6e92dac7f2e453fc00c5d6852","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-06-29 11:44:20.000000000","message":"Patch Set 4: Published edit on patch set 3","accounts_in_message":[],"_revision_number":4},{"id":"6c54164240822cd1feef46bea7e08e22900dc9e8","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-06-29 11:48:29.000000000","message":"Patch Set 5: Published edit on patch set 4","accounts_in_message":[],"_revision_number":5},{"id":"87f739e47403b90138096698e8404e2a37d02236","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-06-29 13:03:35.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/47/478447/5/check/gate-python-cinderclient-docs-ubuntu-xenial/f0df129//doc/build/html/ : SUCCESS in 4m 47s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/47/478447/5/check/gate-python-cinderclient-pep8-ubuntu-xenial/3de50ab/ : SUCCESS in 5m 57s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/47/478447/5/check/gate-python-cinderclient-python27-ubuntu-xenial/7a9f068/ : SUCCESS in 8m 24s\n- gate-python-cinderclient-python35 http://logs.openstack.org/47/478447/5/check/gate-python-cinderclient-python35/54fd6c5/ : SUCCESS in 9m 38s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/47/478447/5/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/f71ec43/ : SUCCESS in 1h 11m 46s\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/47/478447/5/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/96dfaf1/ : SUCCESS in 26m 51s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/47/478447/5/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/b4fc8a7/ : SUCCESS in 26m 42s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/47/478447/5/check/gate-python-cinderclient-pylint-ubuntu-xenial/bc42661/ : FAILURE in 6m 27s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"31b4287f7866f84984802bf004c500edadf15478","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-06-29 22:12:20.000000000","message":"Patch Set 5: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":5},{"id":"a818947fee7a59400afc712554f08abeb6c99a0d","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-06-29 22:20:23.000000000","message":"Patch Set 6: Published edit on patch set 5","accounts_in_message":[],"_revision_number":6},{"id":"0bc6008266a8209b5a96cbee214b48590daac33b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-06-29 23:43:03.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/47/478447/6/check/gate-python-cinderclient-docs-ubuntu-xenial/3ebb2ec//doc/build/html/ : SUCCESS in 3m 25s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/47/478447/6/check/gate-python-cinderclient-pep8-ubuntu-xenial/126398c/ : SUCCESS in 4m 07s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/47/478447/6/check/gate-python-cinderclient-python27-ubuntu-xenial/a54a035/ : SUCCESS in 4m 34s\n- gate-python-cinderclient-python35 http://logs.openstack.org/47/478447/6/check/gate-python-cinderclient-python35/bd664b9/ : SUCCESS in 4m 05s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/47/478447/6/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/2164d28/ : SUCCESS in 1h 22m 09s\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/47/478447/6/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/70e06b0/ : SUCCESS in 16m 53s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/47/478447/6/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/2becfc5/ : SUCCESS in 29m 09s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/47/478447/6/check/gate-python-cinderclient-pylint-ubuntu-xenial/2ae76bd/ : FAILURE in 2m 54s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"4251e2d6b4c829815adadbcc5d5961983d68c986","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-07-02 02:18:00.000000000","message":"Topic changed from bug/1700927 to bp/volume-delete-parameters","accounts_in_message":[],"_revision_number":6},{"id":"69ecfb23dbf98e93d208d0a0b8bdaa3538e09a8d","author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"date":"2017-07-05 06:01:05.000000000","message":"Patch Set 6: Code-Review-1\n\nNeed to fix the pylint errors by disable the function redefine errors","accounts_in_message":[],"_revision_number":6},{"id":"08fb32cc0aab7d117d93abd4d57f3f78e0ab32be","author":{"_account_id":20382,"name":"YuYang","email":"yuyangwang1985@gmail.com","username":"YuYangWang"},"date":"2017-07-05 06:16:43.000000000","message":"Patch Set 6: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"4cec5fedd6eb914ee644c2c672026248ba03ba72","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-07-05 09:33:15.000000000","message":"Patch Set 7: Published edit on patch set 6","accounts_in_message":[],"_revision_number":7},{"id":"7ddd7ab572e71c54cf8ef28266710bb868446737","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-07-05 11:00:12.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/47/478447/7/check/gate-python-cinderclient-docs-ubuntu-xenial/b568407//doc/build/html/ : SUCCESS in 5m 07s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/47/478447/7/check/gate-python-cinderclient-pep8-ubuntu-xenial/0493f4e/ : SUCCESS in 3m 07s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/47/478447/7/check/gate-python-cinderclient-python27-ubuntu-xenial/7cf29f4/ : SUCCESS in 2m 37s\n- gate-python-cinderclient-python35 http://logs.openstack.org/47/478447/7/check/gate-python-cinderclient-python35/c43fcaf/ : SUCCESS in 3m 03s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/47/478447/7/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/3ac41f8/ : SUCCESS in 1h 15m 50s\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/47/478447/7/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/978292f/ : SUCCESS in 23m 32s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/47/478447/7/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/ed17236/ : SUCCESS in 22m 37s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/47/478447/7/check/gate-python-cinderclient-pylint-ubuntu-xenial/c8a8747/ : FAILURE in 3m 46s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"d48b4882e6618d64de8e8c150495297779b93725","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-07-06 00:14:06.000000000","message":"Patch Set 7: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":7},{"id":"a11b49cb04be80e3f3527824e1f964414f60a5dc","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-07-06 09:19:55.000000000","message":"Patch Set 7:\n\n\u003e Need to fix the pylint errors by disable the function redefine\n \u003e errors\n\nHow to fix this? ^_^","accounts_in_message":[],"_revision_number":7},{"id":"9e3eb923cc5eb99d12cd6338e2db272542d6b755","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-07-06 21:35:25.000000000","message":"Patch Set 7:\n\n\u003e\u003e    Need to fix the pylint errors \n\u003e How to fix this? ^_^\n\nJust ignore them for now.  This happens because we have duplicate method names on purpose.  They will not appear in other test runs after this patch is merged.","accounts_in_message":[],"_revision_number":7},{"id":"08fb2eb5e5c85b3bb5783f8e1aac876e8d4bc7e2","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-07-06 21:37:16.000000000","message":"Patch Set 7: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":7},{"id":"147488e5e966ffb8dee51a93c09788bff3fa2829","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-07-07 07:12:23.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"1139e9199a37d455478d34d0ee0a88c6f8a55cd2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-07-07 08:27:00.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/47/478447/8/check/gate-python-cinderclient-docs-ubuntu-xenial/96e9ac0//doc/build/html/ : SUCCESS in 2m 22s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/47/478447/8/check/gate-python-cinderclient-pep8-ubuntu-xenial/6a1563d/ : SUCCESS in 2m 51s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/47/478447/8/check/gate-python-cinderclient-python27-ubuntu-xenial/3a0840b/ : SUCCESS in 3m 19s\n- gate-python-cinderclient-python35 http://logs.openstack.org/47/478447/8/check/gate-python-cinderclient-python35/ff2458f/ : SUCCESS in 4m 09s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/47/478447/8/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/3958303/ : SUCCESS in 1h 11m 54s\n- gate-python-cinderclient-releasenotes http://docs-draft.openstack.org/47/478447/8/check/gate-python-cinderclient-releasenotes/08bb9a2//releasenotes/build/html/ : SUCCESS in 3m 37s\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/47/478447/8/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/6c47cd4/ : SUCCESS in 28m 23s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/47/478447/8/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/e4dfb15/ : SUCCESS in 27m 06s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/47/478447/8/check/gate-python-cinderclient-pylint-ubuntu-xenial/7836b1e/ : FAILURE in 5m 09s (non-voting)","accounts_in_message":[],"_revision_number":8},{"id":"74d557a6aeaafbdd29390ce3e55b365604b32e1e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-07-26 00:26:46.000000000","message":"Patch Set 8: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":8},{"id":"3d459a0d8121935f9c8f9b103ee7bb929772e379","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-07-26 04:37:56.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"b34cc673c50c4c5947ac6dce0e0d784ebc582eee","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-07-26 05:53:57.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/47/478447/9/check/gate-python-cinderclient-docs-ubuntu-xenial/62e3e23//doc/build/html/ : SUCCESS in 3m 01s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/47/478447/9/check/gate-python-cinderclient-pep8-ubuntu-xenial/c4017a6/ : SUCCESS in 3m 42s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/47/478447/9/check/gate-python-cinderclient-python27-ubuntu-xenial/19e71aa/ : SUCCESS in 3m 23s\n- gate-python-cinderclient-python35 http://logs.openstack.org/47/478447/9/check/gate-python-cinderclient-python35/584248a/ : SUCCESS in 2m 32s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/47/478447/9/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/1037434/ : SUCCESS in 1h 13m 48s\n- gate-python-cinderclient-releasenotes http://docs-draft.openstack.org/47/478447/9/check/gate-python-cinderclient-releasenotes/94bc1f1//releasenotes/build/html/ : SUCCESS in 2m 35s\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/47/478447/9/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/d3322b2/ : SUCCESS in 15m 19s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/47/478447/9/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/8758c5f/ : SUCCESS in 15m 34s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/47/478447/9/check/gate-python-cinderclient-pylint-ubuntu-xenial/764a1bf/ : FAILURE in 3m 12s (non-voting)","accounts_in_message":[],"_revision_number":9},{"id":"1ba307ea171b958eab695db507b05942c77b1542","author":{"_account_id":25388,"name":"liuyamin","email":"liuyamin@fiberhome.com","username":"liuyamin"},"date":"2017-08-08 08:30:11.000000000","message":"Patch Set 9: Code-Review+1\n\nLGTM","accounts_in_message":[],"_revision_number":9},{"id":"12bc8a818d04730aec69300299e6b46ccaa93822","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2017-08-15 17:12:23.000000000","message":"Patch Set 9: Code-Review+2","accounts_in_message":[],"_revision_number":9},{"id":"b53caf03e6737d5ffec00508025ba0734ca4f168","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2017-08-23 14:20:51.000000000","message":"Patch Set 9: Code-Review-2\n\nWe have force-delete command, adding these options to delete instead to force-delete will not help our usability.\n\nIf we want to stop using force-delete in favor of using delete --force then that\u0027s a broader discussion.","accounts_in_message":[],"_revision_number":9},{"id":"800557e0187523bdbdbafbcebc4414247d685bcb","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-08-23 14:42:52.000000000","message":"Patch Set 9: Code-Review+2\n\nGorka: This needs to be added to this command to allow deletion with both cascade and force, which our API now allows.","accounts_in_message":[],"_revision_number":9},{"id":"2cd2c88bf7685f0617511ad24cbcb0879565575e","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-08-23 14:58:18.000000000","message":"Patch Set 9:\n\nWe also did have some discussion about moving away from force-delete here: https://review.openstack.org/#/c/320052/1/specs/newton/delete-parameters.rst","accounts_in_message":[],"_revision_number":9},{"id":"b13dc19a403cff46ce970eb03b4b50aaac8a4cb1","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-08-25 09:16:31.000000000","message":"Patch Set 9:\n\n\u003e We have force-delete command, adding these options to delete\n \u003e instead to force-delete will not help our usability.\n \u003e \n \u003e If we want to stop using force-delete in favor of using delete\n \u003e --force then that\u0027s a broader discussion.\n\nGorka,\nNow, both cascade and force params have been implemented for deleting a volume in Cinder.\nI think using the params to delete is much better, we can delete a volume whaterver the force param or the cascade param or even the future new added params by using different params.\nAlso, this can lessen our Cinder API management.","accounts_in_message":[],"_revision_number":9},{"id":"be2bb5a87253f9878245ab45a72bd62c273b567a","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2017-08-25 12:51:56.000000000","message":"Patch Set 9: Code-Review-1\n\nOK, so the commit message needs some work, because what it\u0027s saying is incorrect, but the bigger problem (and I hope this doesn\u0027t merge now that I\u0027ve removed the -2) is that in terms of usability this is a nightmare.\n\nIf you are running Newton and using v3 api you cannot do:\n\n cinder delete --force\n\nYo somehow have to know that you have to do\n\n cinder force-delete\n\nBut if you are running Ocata or later you can use any of those two commands.\n\nI believe we need a consistent mechanism that will work regardless of the v3 microversion and will only fail if you try to do --cascade and --force on the same command prior to 3.23.","accounts_in_message":[],"_revision_number":9},{"id":"8bd566d61aea3b238c85606d4324c6f93166e46a","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2017-09-07 22:32:56.000000000","message":"Patch Set 9: Code-Review+2\n\nI\u0027d like to see this implemented and the force-delete command removed entirely.  If we deprecate it now, we can remove it in a year","accounts_in_message":[],"_revision_number":9},{"id":"557077bf2937e8b26a08ccb40bb26c4ae173e58e","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2017-09-08 09:06:11.000000000","message":"Patch Set 9:\n\nSo apparently everyone is cool with merging something that is fundamentally broken from a usability point of view...","accounts_in_message":[],"_revision_number":9},{"id":"55e20a20c5e25d8574b9e0f82e8d30b5fe05a057","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-10-25 17:54:22.000000000","message":"Patch Set 9:\n\nI\u0027m not sure how this is broken usability-wise.  What I had in mind was jgriffith\u0027s plan above: migrate toward this and ditch \"force-delete\", because it\u0027s not needed as a separate operation.  This isn\u0027t any more complex than most other changes that occur with microversions and API changes.","accounts_in_message":[],"_revision_number":9},{"id":"de7956d7f088efa2f18568dcce26202d1963542b","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-10-26 02:22:30.000000000","message":"Patch Set 10: Commit message was updated.","accounts_in_message":[],"_revision_number":10},{"id":"319a0c7145e1bb870c536beb56fba690941d991d","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-10-26 03:43:57.000000000","message":"Patch Set 10: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/47/478447/10/check/openstack-tox-pep8/6d0d615/ : SUCCESS in 4m 37s\n- openstack-tox-py27 http://logs.openstack.org/47/478447/10/check/openstack-tox-py27/a70695a/ : SUCCESS in 5m 37s\n- openstack-tox-py35 http://logs.openstack.org/47/478447/10/check/openstack-tox-py35/d807e47/ : SUCCESS in 4m 35s\n- build-openstack-sphinx-docs http://logs.openstack.org/47/478447/10/check/build-openstack-sphinx-docs/8c9239e/html/ : SUCCESS in 4m 24s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/47/478447/10/check/legacy-tempest-dsvm-neutron-src/05aa06e/ : SUCCESS in 1h 19m 57s\n- build-openstack-releasenotes http://logs.openstack.org/47/478447/10/check/build-openstack-releasenotes/998eece/html/ : SUCCESS in 4m 30s\n- legacy-cinderclient-dsvm-functional http://logs.openstack.org/47/478447/10/check/legacy-cinderclient-dsvm-functional/c2d3f0e/ : FAILURE in 25m 28s (non-voting)\n- legacy-cinderclient-dsvm-functional-identity-v3-only http://logs.openstack.org/47/478447/10/check/legacy-cinderclient-dsvm-functional-identity-v3-only/90f1694/ : FAILURE in 28m 12s (non-voting)\n- openstack-tox-pylint http://logs.openstack.org/47/478447/10/check/openstack-tox-pylint/6c49ad1/ : FAILURE in 5m 05s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"64da6aecb8e4774d3b38c4f29a724c826ea09283","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2017-10-26 13:25:23.000000000","message":"Patch Set 10: Code-Review+2\n\nI agree with Eric\u0027s last comment. I\u0027d like to see us go this direction.","accounts_in_message":[],"_revision_number":10},{"id":"a29dfdf0292165045ad280b4722595fe95511f87","author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"date":"2017-11-03 10:56:06.000000000","message":"Patch Set 10: Code-Review+2","accounts_in_message":[],"_revision_number":10},{"id":"f66923f082fbd4eb21fe54c87fa53bc2a4cd486e","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2017-11-07 10:16:44.000000000","message":"Patch Set 10: Code-Review-1\n\nI insist this is completely wrong from a user\u0027s point of view.  It should always be possible to call \"cinder delete --force\" in the command line, regardless of what API endpoint ends up being called.\n\nThe user calls \"cinder delete --force\" and cinderclient should either call the old \"os-force_delete\" method if API is older than v3.23 and call delete with the \"force\" parameter set to \"true\" otherwise.\n\nThis would provide a consistent interface instead of making the user know when it\u0027s ok to call \"cinder delete --force\".\n\nWith this patch if you are running Newton and using v3 api you cannot do:\n cinder delete --force\nYou somehow have to know that you have to do\n cinder force-delete\nBut if you are running Ocata or later you can use any of those two commands.","accounts_in_message":[],"_revision_number":10},{"id":"cb1e3c1c1ed1b7ec142b8aea15517e968a5d2e4d","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-11-07 14:19:03.000000000","message":"Patch Set 10:\n\n\u003e I insist this is completely wrong from a user\u0027s point of view.  It\n \u003e should always be possible to call \"cinder delete --force\"\n\nI agree, but, what we were previously doing was also wrong (having a separate force-delete verb), and this is going in the right direction, so I still think it\u0027s a win.\n\nHow bad this approach is depends a lot on your guess as to how often people would combine a Queens cinderclient with a Newton deployment.  I don\u0027t know the answer to that.\n\n \u003e The user calls \"cinder delete --force\" and cinderclient should\n \u003e either call the old \"os-force_delete\" method if API is older than\n \u003e v3.23 and call delete with the \"force\" parameter set to \"true\"\n \u003e otherwise.\n\nCan this be mapped just by checking cs.api_version in the do_delete method?\n\nI don\u0027t mind trying to add this if we have some infrastructure in place to do it.","accounts_in_message":[],"_revision_number":10},{"id":"1dfe53aa8d5eecb1675dcd5b3a22df98e2f741c0","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2017-11-08 09:56:55.000000000","message":"Patch Set 10:\n\n\u003e \u003e I insist this is completely wrong from a user\u0027s point of view. \n \u003e It\n \u003e \u003e should always be possible to call \"cinder delete --force\"\n \u003e \n \u003e I agree, but, what we were previously doing was also wrong (having\n \u003e a separate force-delete verb), and this is going in the right\n \u003e direction, so I still think it\u0027s a win.\n \u003e \n\nIn my experience it pays to do it right the first time, as it\u0027s unlikely that somebody else will improve this afterwards.\n\n \u003e How bad this approach is depends a lot on your guess as to how\n \u003e often people would combine a Queens cinderclient with a Newton\n \u003e deployment.  I don\u0027t know the answer to that.\n \u003e \n\nThere are other cases were this would come into play, as you could be using Queens and call the client with the vanilla v3.0 microversion.\n\n\n \u003e \u003e The user calls \"cinder delete --force\" and cinderclient should\n \u003e \u003e either call the old \"os-force_delete\" method if API is older than\n \u003e \u003e v3.23 and call delete with the \"force\" parameter set to \"true\"\n \u003e \u003e otherwise.\n \u003e \n \u003e Can this be mapped just by checking cs.api_version in the do_delete\n \u003e method?\n \u003e \n\nThat was my suggestion, you always accept --force and then decide which REST API method to call.","accounts_in_message":[],"_revision_number":10},{"id":"36d6d2981162be0e1545df9bbc01073c643f6888","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2017-11-09 10:36:42.000000000","message":"Patch Set 10:\n\nThis issue should be discussed in the weekly meeting. I think we had better provide a unique interface for deleting whihcever API version, the previous implements for --force and force-delete needs to be merged for one API.","accounts_in_message":[],"_revision_number":10},{"id":"664993aca6db4e33ca9d3c3f0316d5e4a074cb3e","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2017-11-16 23:37:20.000000000","message":"Patch Set 10: Code-Review-1\n\nI\u0027d agree with Gorka, additionally the seperate `force-delete` command is silly and frankly it doesn\u0027t work anyway... but that\u0027s another rant.\n\nI like the idea of just adding `--force` to the client and having it \"do the right thing\".  It\u0027s a much more sane user experience and not nearly as maddening as trying to juggle/deal with 100 microversions.","accounts_in_message":[],"_revision_number":10},{"id":"05a863794a999ae78c833c24815920dc7d3c5093","author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"date":"2017-11-18 00:53:00.000000000","message":"Patch Set 10: Code-Review-1\n\nAgree with john and gorka. Not sure how else the weekly meeting discussion will go.","accounts_in_message":[],"_revision_number":10},{"id":"ed29fbeedb74f7adc82cd43dbea8bd0275a0059a","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2017-11-29 21:56:21.000000000","message":"Removed Code-Review+2 by Sean McGinnis \u003csean.mcginnis@gmail.com\u003e\n","accounts_in_message":[],"_revision_number":10},{"id":"b6da4a9484c0db68317953f6efeecbf11a41c43c","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2017-11-29 22:59:02.000000000","message":"Patch Set 10:\n\nJust to clarify, my thinking was that the --force flag is great (how a new verb `force-delete` made it in is maddening), but point is/was don\u0027t mess with versions in the client like this.\n\nJust add the --force to both V2 and V3 shell and if the servers API Version (or MV) isn\u0027t available, be smart in the client and sort it out.  The point being don\u0027t make the client more convoluted and difficult to use than necessary, and most of all don\u0027t introduce a guessing game for users to figure out what they can and can\u0027t do.\n\nIf you were to modify this to just support \"cinder delete --force\" in both V2 and V3.x and hide the versioning stuff it would in my opinion fix some of our past mistakes here and I would be happy to up-vote and argue for it to be merged.","accounts_in_message":[],"_revision_number":10},{"id":"ff3b64fa9099823bf64fadab913eb0c3a1e7dd32","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2017-11-29 23:16:49.000000000","message":"Patch Set 10: Code-Review-1\n\nI think we should go with what John, Gorka and Mike are suggesting.  No hurry here, lets do it right.","accounts_in_message":[],"_revision_number":10},{"id":"77460a03c4a803f88d915d4eaa67b932ade244f3","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2018-01-17 23:48:19.000000000","message":"Patch Set 10:\n\nShould this change be abandoned?","accounts_in_message":[],"_revision_number":10},{"id":"61b7519d0cb5d5299388c460559e747322a76e63","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2018-01-18 00:13:11.000000000","message":"Patch Set 10:\n\n\u003e Should this change be abandoned?\n\nYes, I will abondon it right now.","accounts_in_message":[],"_revision_number":10},{"id":"6297f56f1042665c42f36b7a2d122026782bcda6","author":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"date":"2018-01-18 00:13:30.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":10}],"current_revision_number":10,"current_revision":"39ea690d7ea2da047c8bedb0fa84910a188bce7b","revisions":{"8aceb76dd42e083c18a2440ba047f46b16af0ab7":{"kind":"REWORK","_number":1,"created":"2017-06-28 09:19:59.000000000","uploader":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"ref":"refs/changes/47/478447/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/47/478447/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/1"}}},"commit":{"parents":[{"commit":"b910f5bea33cbccca25b008b3b03dc5dce27245a","subject":"Unicode value support for \"--filters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b910f5bea33cbccca25b008b3b03dc5dce27245a"}]}],"author":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-06-28 09:09:37.000000000","tz":480},"committer":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-06-28 09:09:37.000000000","tz":480},"subject":"Fix force delete volume in v3.23","message":"Fix force delete volume in v3.23\n\nCinder support force delete volumes, However, in cinderclient, no comand\nline support this feature. This patch is to add force delete in cinderclient.\n\n  # cinder help delete\n  usage: cinder delete [--cascade \u003cFalse|True\u003e] [--force \u003cFalse|True\u003e]\n                       \u003cvolume\u003e [\u003cvolume\u003e ...]\n  Removes one or more volumes.\n  Positional arguments:\n    \u003cvolume\u003e Name or ID of volume or volumes to delete.\n  Optional arguments:\n    --cascade \u003cFalse|True\u003e\n             Remove any snapshots along with volume. Default\u003dFalse.\n    --force \u003cFalse|True\u003e Force to delete the volume. Default\u003dFalse.\n\nAdd the force param and it will be passed through cinderclient to Cinder API,\nif the version is v3.23, then the param force will take effect.\n\nChange-Id: I2056b8d94bbf29d2f6943de7612954147a0e8cde\nCloses-Bug: #1700927\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8aceb76dd42e083c18a2440ba047f46b16af0ab7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8aceb76dd42e083c18a2440ba047f46b16af0ab7"}]},"branch":"refs/heads/master"},"f202738c08c1b71b06e3a389e9f4bb21f7d11f1f":{"kind":"REWORK","_number":2,"created":"2017-06-29 08:19:25.000000000","uploader":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"ref":"refs/changes/47/478447/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/47/478447/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/2"}}},"commit":{"parents":[{"commit":"b910f5bea33cbccca25b008b3b03dc5dce27245a","subject":"Unicode value support for \"--filters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b910f5bea33cbccca25b008b3b03dc5dce27245a"}]}],"author":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-06-28 09:09:37.000000000","tz":480},"committer":{"name":"junbo.li","email":"junbo85.li@gmail.com","date":"2017-06-29 08:19:02.000000000","tz":0},"subject":"Add force delete volume in v3.23","message":"Add force delete volume in v3.23\n\nCinder supports force delete volumes, However, in cinderclient, no comand\nline support this feature. This patch is to add force delete in cinderclient.\n\n  # cinder help delete\n  usage: cinder delete [--cascade \u003cFalse|True\u003e] [--force \u003cFalse|True\u003e]\n                       \u003cvolume\u003e [\u003cvolume\u003e ...]\n  Removes one or more volumes.\n  Positional arguments:\n    \u003cvolume\u003e Name or ID of volume or volumes to delete.\n  Optional arguments:\n    --cascade \u003cFalse|True\u003e\n             Remove any snapshots along with volume. Default\u003dFalse.\n    --force \u003cFalse|True\u003e Force to delete the volume. Default\u003dFalse.\n\nAdd the force param and it will be passed through cinderclient to Cinder API,\nif the version is v3.23, then the param force will take effect.\n\nImplements: bp volume-delete-parameters\nChange-Id: I2056b8d94bbf29d2f6943de7612954147a0e8cde\n\n\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/f202738c08c1b71b06e3a389e9f4bb21f7d11f1f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/f202738c08c1b71b06e3a389e9f4bb21f7d11f1f"}]},"branch":"refs/heads/master"},"bcd0715a26dbe9fb2cfb6525ca818780fa10e4d6":{"kind":"REWORK","_number":3,"created":"2017-06-29 09:35:36.000000000","uploader":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"ref":"refs/changes/47/478447/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/47/478447/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/3"}}},"commit":{"parents":[{"commit":"b910f5bea33cbccca25b008b3b03dc5dce27245a","subject":"Unicode value support for \"--filters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b910f5bea33cbccca25b008b3b03dc5dce27245a"}]}],"author":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-06-28 09:09:37.000000000","tz":480},"committer":{"name":"junbo.li","email":"junbo85.li@gmail.com","date":"2017-06-29 09:35:27.000000000","tz":0},"subject":"Add force delete volume in v3.23","message":"Add force delete volume in v3.23\n\nCinder supports force delete volumes, However, in cinderclient, no comand\nline support this feature. This patch is to add force delete in cinderclient.\n\n  # cinder help delete\n  usage: cinder delete [--cascade \u003cFalse|True\u003e] [--force \u003cFalse|True\u003e]\n                       \u003cvolume\u003e [\u003cvolume\u003e ...]\n  Removes one or more volumes.\n  Positional arguments:\n    \u003cvolume\u003e Name or ID of volume or volumes to delete.\n  Optional arguments:\n    --cascade \u003cFalse|True\u003e\n             Remove any snapshots along with volume. Default\u003dFalse.\n    --force \u003cFalse|True\u003e Force to delete the volume. Default\u003dFalse.\n\nAdd the force param and it will be passed through cinderclient to Cinder API,\nif the version is v3.23, then the param force will take effect.\n\nImplements: bp volume-delete-parameters\nChange-Id: I2056b8d94bbf29d2f6943de7612954147a0e8cde\n\n\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/bcd0715a26dbe9fb2cfb6525ca818780fa10e4d6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/bcd0715a26dbe9fb2cfb6525ca818780fa10e4d6"}]},"branch":"refs/heads/master"},"b69c09788d8288178206ad5ece903d01815bac3d":{"kind":"REWORK","_number":4,"created":"2017-06-29 11:44:20.000000000","uploader":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"ref":"refs/changes/47/478447/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/47/478447/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/4"}}},"commit":{"parents":[{"commit":"b910f5bea33cbccca25b008b3b03dc5dce27245a","subject":"Unicode value support for \"--filters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b910f5bea33cbccca25b008b3b03dc5dce27245a"}]}],"author":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-06-28 09:09:37.000000000","tz":480},"committer":{"name":"junbo.li","email":"junbo85.li@gmail.com","date":"2017-06-29 11:43:42.000000000","tz":0},"subject":"Add force delete volume in v3.23","message":"Add force delete volume in v3.23\n\nCinder supports force delete volumes, However, in cinderclient, no comand\nline support this feature. This patch is to add force delete in cinderclient.\n\n  # cinder help delete\n  usage: cinder delete [--cascade \u003cFalse|True\u003e] [--force \u003cFalse|True\u003e]\n                       \u003cvolume\u003e [\u003cvolume\u003e ...]\n  Removes one or more volumes.\n  Positional arguments:\n    \u003cvolume\u003e Name or ID of volume or volumes to delete.\n  Optional arguments:\n    --cascade \u003cFalse|True\u003e\n             Remove any snapshots along with volume. Default\u003dFalse.\n    --force \u003cFalse|True\u003e Force to delete the volume. Default\u003dFalse.\n\nAdd the force param and it will be passed through cinderclient to Cinder API,\nif the version is v3.23, then the param force will take effect.\n\nImplements: bp volume-delete-parameters\nChange-Id: I2056b8d94bbf29d2f6943de7612954147a0e8cde\n\n\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b69c09788d8288178206ad5ece903d01815bac3d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b69c09788d8288178206ad5ece903d01815bac3d"}]},"branch":"refs/heads/master"},"8b0217dea7b291307bc254e5eef7462cab1ca75f":{"kind":"REWORK","_number":5,"created":"2017-06-29 11:48:29.000000000","uploader":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"ref":"refs/changes/47/478447/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/47/478447/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/5"}}},"commit":{"parents":[{"commit":"b910f5bea33cbccca25b008b3b03dc5dce27245a","subject":"Unicode value support for \"--filters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b910f5bea33cbccca25b008b3b03dc5dce27245a"}]}],"author":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-06-28 09:09:37.000000000","tz":480},"committer":{"name":"junbo.li","email":"junbo85.li@gmail.com","date":"2017-06-29 11:48:19.000000000","tz":0},"subject":"Add force delete volume in v3.23","message":"Add force delete volume in v3.23\n\nCinder supports force delete volumes, However, in cinderclient, no comand\nline support this feature. This patch is to add force delete in cinderclient.\n\n  # cinder help delete\n  usage: cinder delete [--cascade \u003cFalse|True\u003e] [--force \u003cFalse|True\u003e]\n                       \u003cvolume\u003e [\u003cvolume\u003e ...]\n  Removes one or more volumes.\n  Positional arguments:\n    \u003cvolume\u003e Name or ID of volume or volumes to delete.\n  Optional arguments:\n    --cascade \u003cFalse|True\u003e\n             Remove any snapshots along with volume. Default\u003dFalse.\n    --force \u003cFalse|True\u003e Force to delete the volume. Default\u003dFalse.\n\nAdd the force param and it will be passed through cinderclient to Cinder API,\nif the version is v3.23, then the param force will take effect.\n\nImplements: bp volume-delete-parameters\nChange-Id: I2056b8d94bbf29d2f6943de7612954147a0e8cde\n\n\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8b0217dea7b291307bc254e5eef7462cab1ca75f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8b0217dea7b291307bc254e5eef7462cab1ca75f"}]},"branch":"refs/heads/master"},"398884113cff6064559a7b4e5515d9e54b2338be":{"kind":"REWORK","_number":6,"created":"2017-06-29 22:20:23.000000000","uploader":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"ref":"refs/changes/47/478447/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/47/478447/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/6"}}},"commit":{"parents":[{"commit":"b910f5bea33cbccca25b008b3b03dc5dce27245a","subject":"Unicode value support for \"--filters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b910f5bea33cbccca25b008b3b03dc5dce27245a"}]}],"author":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-06-28 09:09:37.000000000","tz":480},"committer":{"name":"junbo.li","email":"junbo85.li@gmail.com","date":"2017-06-29 22:19:42.000000000","tz":0},"subject":"Add force delete volume in v3.23","message":"Add force delete volume in v3.23\n\nCinder supports force delete volumes, However, in cinderclient, no comand\nline support this feature. This patch is to add force delete in cinderclient.\n\n  # cinder help delete\n  usage: cinder delete [--cascade \u003cFalse|True\u003e] [--force \u003cFalse|True\u003e]\n                       \u003cvolume\u003e [\u003cvolume\u003e ...]\n  Removes one or more volumes.\n  Positional arguments:\n    \u003cvolume\u003e Name or ID of volume or volumes to delete.\n  Optional arguments:\n    --cascade \u003cFalse|True\u003e\n             Remove any snapshots along with volume. Default\u003dFalse.\n    --force \u003cFalse|True\u003e Force to delete the volume. Default\u003dFalse.\n\nAdd the force param and it will be passed through cinderclient to Cinder API,\nif the version is v3.23, then the param force will take effect.\n\nImplements: bp volume-delete-parameters\nChange-Id: I2056b8d94bbf29d2f6943de7612954147a0e8cde\n\n\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/398884113cff6064559a7b4e5515d9e54b2338be"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/398884113cff6064559a7b4e5515d9e54b2338be"}]},"branch":"refs/heads/master"},"a821365c155e23c0e422cef4a9ff9f7bf87e0c0d":{"kind":"REWORK","_number":7,"created":"2017-07-05 09:33:15.000000000","uploader":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"ref":"refs/changes/47/478447/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/47/478447/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/7"}}},"commit":{"parents":[{"commit":"b910f5bea33cbccca25b008b3b03dc5dce27245a","subject":"Unicode value support for \"--filters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b910f5bea33cbccca25b008b3b03dc5dce27245a"}]}],"author":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-06-28 09:09:37.000000000","tz":480},"committer":{"name":"junbo.li","email":"junbo85.li@gmail.com","date":"2017-07-05 09:32:58.000000000","tz":0},"subject":"Add force delete volume in v3.23","message":"Add force delete volume in v3.23\n\nCinder supports force delete volumes, However, in cinderclient, no comand\nline support this feature. This patch is to add force delete in cinderclient.\n\n  # cinder help delete\n  usage: cinder delete [--cascade \u003cFalse|True\u003e] [--force \u003cFalse|True\u003e]\n                       \u003cvolume\u003e [\u003cvolume\u003e ...]\n  Removes one or more volumes.\n  Positional arguments:\n    \u003cvolume\u003e Name or ID of volume or volumes to delete.\n  Optional arguments:\n    --cascade \u003cFalse|True\u003e\n             Remove any snapshots along with volume. Default\u003dFalse.\n    --force \u003cFalse|True\u003e Force to delete the volume. Default\u003dFalse.\n\nAdd the force param and it will be passed through cinderclient to Cinder API,\nif the version is v3.23, then the param force will take effect.\n\nImplements: bp volume-delete-parameters\nChange-Id: I2056b8d94bbf29d2f6943de7612954147a0e8cde\n\n\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a821365c155e23c0e422cef4a9ff9f7bf87e0c0d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a821365c155e23c0e422cef4a9ff9f7bf87e0c0d"}]},"branch":"refs/heads/master"},"4ca90a9304d8131bffe6660d7a81c58d829a4dab":{"kind":"REWORK","_number":8,"created":"2017-07-07 07:12:23.000000000","uploader":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"ref":"refs/changes/47/478447/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/47/478447/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/8"}}},"commit":{"parents":[{"commit":"0fdd41d84ccfa16245d60617a4099e120f6100cf","subject":"Merge \"Fix output of update command\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/0fdd41d84ccfa16245d60617a4099e120f6100cf"}]}],"author":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-07-07 07:10:55.000000000","tz":480},"committer":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-07-07 07:10:55.000000000","tz":480},"subject":"Add force delete volume in v3.23","message":"Add force delete volume in v3.23\n\nCinder supports force delete volumes, However, in cinderclient, no comand\nline support this feature. This patch is to add force delete in cinderclient.\n\n  # cinder help delete\n  usage: cinder delete [--cascade] [--force] \u003cvolume\u003e [\u003cvolume\u003e ...]\n  Removes one or more volumes.\n  Positional arguments:\n    \u003cvolume\u003e   Name or ID of volume to delete.\n  Optional arguments:\n    --cascade  Remove any snapshots along with volume.\n    --force    Volume force delete. (Supported by API version 3.23 and later)\n\nAdd the force param and it will be passed through cinderclient to Cinder API,\nif the version is v3.23, then the param force will take effect.\n\nImplements: bp volume-delete-parameters\nChange-Id: I2056b8d94bbf29d2f6943de7612954147a0e8cde\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/4ca90a9304d8131bffe6660d7a81c58d829a4dab"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/4ca90a9304d8131bffe6660d7a81c58d829a4dab"}]},"branch":"refs/heads/master"},"82fc90d58d84f16d93498b57fbf501e0dae3c68f":{"kind":"REWORK","_number":9,"created":"2017-07-26 04:37:56.000000000","uploader":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"ref":"refs/changes/47/478447/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/47/478447/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/9"}}},"commit":{"parents":[{"commit":"c73828c5cede28488d68eadf06d2e4f0454495d9","subject":"Merge \"Fix support for Unicode value filters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c73828c5cede28488d68eadf06d2e4f0454495d9"}]}],"author":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-07-26 04:35:52.000000000","tz":480},"committer":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-07-26 04:37:07.000000000","tz":480},"subject":"Add force delete volume in v3.23","message":"Add force delete volume in v3.23\n\nCinder supports force delete volumes, However, in cinderclient, no comand\nline support this feature. This patch is to add force delete in cinderclient.\n\n  # cinder help delete\n  usage: cinder delete [--cascade] [--force] \u003cvolume\u003e [\u003cvolume\u003e ...]\n  Removes one or more volumes.\n  Positional arguments:\n    \u003cvolume\u003e   Name or ID of volume to delete.\n  Optional arguments:\n    --cascade  Remove any snapshots along with volume.\n    --force    Volume force delete. (Supported by API version 3.23 and later)\n\nAdd the force param and it will be passed through cinderclient to Cinder API,\nif the version is v3.23, then the param force will take effect.\n\nImplements: bp volume-delete-parameters\nChange-Id: I2056b8d94bbf29d2f6943de7612954147a0e8cde\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/82fc90d58d84f16d93498b57fbf501e0dae3c68f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/82fc90d58d84f16d93498b57fbf501e0dae3c68f"}]},"branch":"refs/heads/master"},"39ea690d7ea2da047c8bedb0fa84910a188bce7b":{"kind":"NO_CODE_CHANGE","_number":10,"created":"2017-10-26 02:22:30.000000000","uploader":{"_account_id":25456,"name":"junboli","email":"junbo85.li@gmail.com","username":"junboli"},"ref":"refs/changes/47/478447/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/47/478447/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/47/478447/10"}}},"commit":{"parents":[{"commit":"c73828c5cede28488d68eadf06d2e4f0454495d9","subject":"Merge \"Fix support for Unicode value filters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c73828c5cede28488d68eadf06d2e4f0454495d9"}]}],"author":{"name":"junboli","email":"junbo85.li@gmail.com","date":"2017-07-26 04:35:52.000000000","tz":480},"committer":{"name":"junbo.li","email":"junbo85.li@gmail.com","date":"2017-10-26 02:22:22.000000000","tz":0},"subject":"Support \"--force\" param to delete volume in v3.23","message":"Support \"--force\" param to delete volume in v3.23\n\nCinder supports delete volumes by using force param, However, in\ncinderclient, no comand line support this feature. This patch is\nto add \"--force\" param to delete in cinderclient.\n\nAdd the \"--force\" param and it will be passed through cinderclient\nto Cinder API, if the version is v3.23, then the force param would\ntake effect.\n\n\nImplements: bp volume-delete-parameters\nChange-Id: I2056b8d94bbf29d2f6943de7612954147a0e8cde\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/39ea690d7ea2da047c8bedb0fa84910a188bce7b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/39ea690d7ea2da047c8bedb0fa84910a188bce7b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}