)]}'
{"id":"openstack%2Fpython-cinderclient~38002","triplet_id":"openstack%2Fpython-cinderclient~master~If4858dc17cedf027112defb935016137727681cc","project":"openstack/python-cinderclient","branch":"master","topic":"bug/1014689","hashtags":[],"change_id":"If4858dc17cedf027112defb935016137727681cc","subject":"Add evaluation of --force parameter when creating snapshots","status":"MERGED","created":"2013-07-20 00:01:48.000000000","updated":"2013-08-03 00:09:23.000000000","submitted":"2013-07-31 01:58:54.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"a61260f0b1536e5f8f68cf5a3b4dd430b43f674e","_number":38002,"virtual_id_number":38002,"owner":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},{"value":0,"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},{"value":2,"date":"2013-07-31 01:58:54.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":0,"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},{"value":0,"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"all":[{"value":0,"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},{"value":0,"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-07-30 23:35:00.000000000","_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":0,"date":"2013-07-28 16:28:03.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},{"value":2,"date":"2013-07-30 06:41:18.000000000","_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"}],"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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"all":[{"value":0,"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},{"value":0,"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-07-30 23:35:00.000000000","_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":0,"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},{"value":0,"date":"2013-07-30 06:41:18.000000000","_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},{"value":0,"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":0,"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},{"value":0,"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"}],"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":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"},{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-07-21 12:17:02.000000000","updated_by":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"reviewer":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"state":"REVIEWER"},{"updated":"2013-07-21 18:45:23.000000000","updated_by":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"reviewer":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"state":"REVIEWER"},{"updated":"2013-07-22 14:33:06.000000000","updated_by":{"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},"reviewer":{"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},"state":"REVIEWER"},{"updated":"2013-07-28 15:45:19.000000000","updated_by":{"_account_id":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"},"reviewer":{"_account_id":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"},"state":"REVIEWER"},{"updated":"2013-07-30 06:41:18.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":"2013-07-30 23:35:00.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":"2013-07-31 01:58:54.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"0ebaef04e0e01d71d3da472c59aea938915a5740","author":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"date":"2013-07-20 00:04:49.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"2ec4f2d19ea632bc6c94ec69cadf4f487d7e89c0","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-07-20 00:04:55.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":2},{"id":"ea89c15e02a7345033b2fd503d518a3b6bc981a0","author":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"date":"2013-07-20 00:05:31.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"0ac7e3ea9fbfe0cd2a4bff75be0ba9d1720b07f7","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-07-20 00:05:37.000000000","message":"Patch Set 3:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":3},{"id":"46e3f797a3e6d2bb175c87bac2972e506fcfcf8c","author":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"date":"2013-07-20 00:07:53.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"ebbb38c7adc0dfb29cce1f9ec4f886360c839a6e","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-07-20 00:07:58.000000000","message":"Patch Set 4:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":4},{"id":"ae5ae91e51d3b401a5fa6744577b9d50b9fbb622","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-20 00:52:47.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/02/38002/4/check/gate-python-cinderclient-pep8/575 : SUCCESS in 1m 09s\n- http://docs-draft.openstack.org/02/38002/4/check/gate-python-cinderclient-docs/539/doc/build/html/ : SUCCESS in 51s\n- http://logs.openstack.org/02/38002/4/check/gate-python-cinderclient-python26/571 : SUCCESS in 1m 09s\n- http://logs.openstack.org/02/38002/4/check/gate-python-cinderclient-python27/569 : SUCCESS in 46s\n- http://logs.openstack.org/02/38002/4/check/gate-tempest-devstack-vm-full/34020 : SUCCESS in 44m 28s\n- http://logs.openstack.org/02/38002/4/check/gate-tempest-devstack-vm-cells-full/1673 : FAILURE in 13m 53s (non-voting)\n- http://logs.openstack.org/02/38002/4/check/gate-tempest-devstack-vm-testr-full/524 : SUCCESS in 26m 23s (non-voting)\n- http://logs.openstack.org/02/38002/4/check/gate-tempest-devstack-vm-postgres-full/33710 : SUCCESS in 44m 50s\n- http://logs.openstack.org/02/38002/4/check/gate-tempest-devstack-vm-neutron/5804 : SUCCESS in 17m 47s (non-voting)\n- http://logs.openstack.org/02/38002/4/check/gate-grenade-devstack-vm/19740 : SUCCESS in 26m 25s\n","accounts_in_message":[],"_revision_number":4},{"id":"7c4e06028c40db9b70728df78612b55fd77116b0","author":{"_account_id":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"},"date":"2013-07-24 14:00:08.000000000","message":"Patch Set 4: I would prefer that you didn\u0027t merge this\n\nLooking at https://github.com/openstack/cinder/commit/c678b045c9839ed30e13fe5e7e655e3cb2e2c3f9 -- do we really want to have many (identical?) is_valid_boolstr implementations spread across cinder, cinderclient, nova, etc.?\n\nAlso, wouldn\u0027t it be better to not allow any params for --force on the shell client and just pass True to the API, in case of \"--force\" existance?","accounts_in_message":[],"_revision_number":4},{"id":"f7d7d07b5044e2ade0ef06a33ecd17230efad83e","author":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"date":"2013-07-24 14:49:06.000000000","message":"Patch Set 4:\n\nGood call... Will change this soon","accounts_in_message":[],"_revision_number":4},{"id":"b60332deb9eee6a45783ab9efdc3dc1bd147d50c","author":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"date":"2013-07-28 15:29:18.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"84f7e46c54cf70b5b2247b0102088c640a1f7218","author":{"_account_id":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"},"date":"2013-07-28 15:43:49.000000000","message":"Patch Set 5: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nIn addition to just using action\u003d\"store_true\" for --force, this should probably also be fixed for v2/shell.py, to keep things consistent?","accounts_in_message":[],"_revision_number":5},{"id":"3890883b02512843cf9242d0b9ec93d58ee8ca95","author":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"date":"2013-07-28 15:44:13.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"1ed72107525068ea7e1c5be9b4b55b824b7ddd5c","author":{"_account_id":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"},"date":"2013-07-28 15:45:19.000000000","message":"Patch Set 6: I would prefer that you didn\u0027t merge this\n\nIMHO my comment from revision #8 is still holding true ;)","accounts_in_message":[],"_revision_number":6},{"id":"fbcec712488874b29bbdcae7ef81392799b10c22","author":{"_account_id":1119,"name":"Christoph Thiel","email":"ct@kki.org","username":"cthiel-suse"},"date":"2013-07-28 15:46:10.000000000","message":"Patch Set 6:\n\nLast comment should have read: patch set #5 (not #8 :)) of course.","accounts_in_message":[],"_revision_number":6},{"id":"cdcbe9b011d930da8afb021bff583dc71ecedf0a","author":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"date":"2013-07-28 15:53:08.000000000","message":"Patch Set 6:\n\nSo altering --force to take no arguments would break the API. I don\u0027t think we can do this without a discussion with the maintainers.\nHowever I removed the dedicated method to verify the force argument like Christoph suggested.","accounts_in_message":[],"_revision_number":6},{"id":"f0c605a162a634f76aa30ddf5b27e74a227eb53f","author":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"date":"2013-07-28 16:20:46.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"0077b8ed6b037cd7a1d1cde8b09e3bf2036baf1e","author":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"date":"2013-07-28 16:28:03.000000000","message":"Patch Set 7:\n\nSo we don\u0027t need to convert the string to a boolean on the python-cinderclient side. All we need to do is verify the string is of an acceptable value.\ncinder-api already has strutils.bool_from_string(force) that takes the passed force string and converts it to a boolean","accounts_in_message":[],"_revision_number":7},{"id":"c46d299bdced0240ce05e516eb70e25c5482ba30","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-28 17:16:11.000000000","message":"Patch Set 7: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/02/38002/7/check/gate-python-cinderclient-pep8/602 : SUCCESS in 39s\n- http://docs-draft.openstack.org/02/38002/7/check/gate-python-cinderclient-docs/566/doc/build/html/ : SUCCESS in 46s\n- http://logs.openstack.org/02/38002/7/check/gate-python-cinderclient-python26/598 : SUCCESS in 1m 02s\n- http://logs.openstack.org/02/38002/7/check/gate-python-cinderclient-python27/596 : SUCCESS in 46s\n- http://logs.openstack.org/02/38002/7/check/gate-tempest-devstack-vm-full/36756 : SUCCESS in 55m 21s\n- http://logs.openstack.org/02/38002/7/check/gate-tempest-devstack-vm-cells-full/3533 : FAILURE in 15m 25s (non-voting)\n- http://logs.openstack.org/02/38002/7/check/gate-tempest-devstack-vm-testr-full/2357 : FAILURE in 32m 25s (non-voting)\n- http://logs.openstack.org/02/38002/7/check/gate-tempest-devstack-vm-postgres-full/36433 : SUCCESS in 46m 52s\n- http://logs.openstack.org/02/38002/7/check/gate-tempest-devstack-vm-neutron/8800 : FAILURE in 16m 15s (non-voting)\n- http://logs.openstack.org/02/38002/7/check/gate-grenade-devstack-vm/22203 : FAILURE in 18m 48s\n","accounts_in_message":[],"_revision_number":7},{"id":"8e3a549d2ed031ce92492a9f20bd63ebcfaa23f8","author":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"date":"2013-07-29 08:44:43.000000000","message":"Patch Set 7:\n\nrecheck no bug","accounts_in_message":[],"_revision_number":7},{"id":"b13cfa2ddd51c0fe6835593b7f38a9d4967cc0db","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-29 09:29:09.000000000","message":"Patch Set 7: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/02/38002/7/check/gate-python-cinderclient-pep8/604 : SUCCESS in 45s\n- http://docs-draft.openstack.org/02/38002/7/check/gate-python-cinderclient-docs/568/doc/build/html/ : SUCCESS in 47s\n- http://logs.openstack.org/02/38002/7/check/gate-python-cinderclient-python26/600 : SUCCESS in 1m 22s\n- http://logs.openstack.org/02/38002/7/check/gate-python-cinderclient-python27/598 : SUCCESS in 50s\n- http://logs.openstack.org/02/38002/7/check/gate-tempest-devstack-vm-full/36889 : SUCCESS in 44m 21s\n- http://logs.openstack.org/02/38002/7/check/gate-tempest-devstack-vm-cells-full/3652 : FAILURE in 15m 58s (non-voting)\n- http://logs.openstack.org/02/38002/7/check/gate-tempest-devstack-vm-testr-full/2473 : SUCCESS in 28m 40s (non-voting)\n- http://logs.openstack.org/02/38002/7/check/gate-tempest-devstack-vm-postgres-full/36566 : SUCCESS in 44m 21s\n- http://logs.openstack.org/02/38002/7/check/gate-tempest-devstack-vm-neutron/8942 : FAILURE in 18m 00s (non-voting)\n- http://logs.openstack.org/02/38002/7/check/gate-grenade-devstack-vm/22309 : SUCCESS in 22m 47s\n","accounts_in_message":[],"_revision_number":7},{"id":"7742890902964ee0d73d7f8da3543b85356ae188","author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"date":"2013-07-30 06:41:18.000000000","message":"Patch Set 7: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":7},{"id":"c22684bbb106d9ec38fce628cb2b9fc12bbab210","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2013-07-30 23:35:00.000000000","message":"Patch Set 7: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":7},{"id":"f596f8c3a08882df05290d821bbab065ade15aaa","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-30 23:35:01.000000000","message":"Patch Set 7: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":7},{"id":"5555dd67bd4c5d501f3546826481d20b11ba8907","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-31 01:58:54.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":7},{"id":"771868dd2ec7d433726838c94b147f450e66b3f9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-31 01:58:54.000000000","message":"Patch Set 7: Verified\n\nBuild succeeded.\n\n- http://docs-draft.openstack.org/02/38002/7/gate/gate-python-cinderclient-docs/578/doc/build/html/ : SUCCESS in 1m 07s\n- http://logs.openstack.org/02/38002/7/gate/gate-python-cinderclient-pep8/614 : SUCCESS in 57s\n- http://logs.openstack.org/02/38002/7/gate/gate-python-cinderclient-python26/610 : SUCCESS in 1m 13s\n- http://logs.openstack.org/02/38002/7/gate/gate-python-cinderclient-python27/608 : SUCCESS in 50s\n- http://logs.openstack.org/02/38002/7/gate/gate-tempest-devstack-vm-full/37582 : SUCCESS in 57m 51s\n- http://logs.openstack.org/02/38002/7/gate/gate-tempest-devstack-vm-postgres-full/37258 : SUCCESS in 50m 11s\n- http://logs.openstack.org/02/38002/7/gate/gate-tempest-devstack-vm-neutron/9716 : FAILURE in 21m 22s (non-voting)\n- http://logs.openstack.org/02/38002/7/gate/gate-grenade-devstack-vm/22927 : SUCCESS in 23m 33s\n","accounts_in_message":[],"_revision_number":7},{"id":"887dfce44ab5a380b08b485d15f7d2bc1fe89309","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2013-08-03 00:09:23.000000000","message":"Patch Set 7: Reverted\n\nThis patchset was reverted in change: Iaeab4e4335a94db6ca68950110bd9dd0f9d94b4f","accounts_in_message":[],"_revision_number":7}],"current_revision_number":7,"current_revision":"854740c4c793444c955180e7c0c96977e3e06c56","revisions":{"043a484d03ae4187e7c9e12d9974acd9e8a5b378":{"kind":"REWORK","_number":1,"created":"2013-07-20 00:01:48.000000000","uploader":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"ref":"refs/changes/02/38002/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/02/38002/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/1"}}},"commit":{"parents":[{"commit":"eaad8b0bd02316554cda6b4d5a0d7ead7c550649","subject":"Merge \"Revert \"Use exceptions from oslo\"\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/eaad8b0bd02316554cda6b4d5a0d7ead7c550649"}]}],"author":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:01:41.000000000","tz":0},"committer":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:01:41.000000000","tz":0},"subject":"Add evaluation of --force parameter when creating snapshots and raising a BadRequest if value is invalid","message":"Add evaluation of --force parameter when creating snapshots and raising\na BadRequest if value is invalid\n\nChange-Id: If4858dc17cedf027112defb935016137727681cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/043a484d03ae4187e7c9e12d9974acd9e8a5b378"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/043a484d03ae4187e7c9e12d9974acd9e8a5b378"}]},"branch":"refs/heads/master"},"77dcf68aef3f8673de9cd50b535acf1da56dc932":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-07-20 00:04:49.000000000","uploader":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"ref":"refs/changes/02/38002/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/02/38002/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/2"}}},"commit":{"parents":[{"commit":"eaad8b0bd02316554cda6b4d5a0d7ead7c550649","subject":"Merge \"Revert \"Use exceptions from oslo\"\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/eaad8b0bd02316554cda6b4d5a0d7ead7c550649"}]}],"author":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:01:41.000000000","tz":0},"committer":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:06:18.000000000","tz":0},"subject":"Add evaluation of --force parameter when creating snapshots and raising a BadRequest if value is invalid","message":"Add evaluation of --force parameter when creating snapshots and raising\na BadRequest if value is invalid\n\nFixes: bug #1014689\n\nChange-Id: If4858dc17cedf027112defb935016137727681cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/77dcf68aef3f8673de9cd50b535acf1da56dc932"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/77dcf68aef3f8673de9cd50b535acf1da56dc932"}]},"branch":"refs/heads/master"},"cf1c45f209e9645ba117d7a39796e73210dcc811":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2013-07-20 00:05:31.000000000","uploader":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"ref":"refs/changes/02/38002/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/02/38002/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/3"}}},"commit":{"parents":[{"commit":"eaad8b0bd02316554cda6b4d5a0d7ead7c550649","subject":"Merge \"Revert \"Use exceptions from oslo\"\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/eaad8b0bd02316554cda6b4d5a0d7ead7c550649"}]}],"author":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:01:41.000000000","tz":0},"committer":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:07:10.000000000","tz":0},"subject":"Add evaluation of --force parameter when creating snapshots and raising a BadRequest Exception  if value is invalid","message":"Add evaluation of --force parameter when creating snapshots\nand raising a BadRequest Exception  if value is invalid\n\nFixes: bug #1014689\n\nChange-Id: If4858dc17cedf027112defb935016137727681cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/cf1c45f209e9645ba117d7a39796e73210dcc811"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/cf1c45f209e9645ba117d7a39796e73210dcc811"}]},"branch":"refs/heads/master"},"f64623354a8176f0182ce4e0f41b19bd6bafc832":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2013-07-20 00:07:53.000000000","uploader":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"ref":"refs/changes/02/38002/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/02/38002/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/4"}}},"commit":{"parents":[{"commit":"eaad8b0bd02316554cda6b4d5a0d7ead7c550649","subject":"Merge \"Revert \"Use exceptions from oslo\"\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/eaad8b0bd02316554cda6b4d5a0d7ead7c550649"}]}],"author":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:01:41.000000000","tz":0},"committer":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:07:57.000000000","tz":0},"subject":"Add evaluation of --force parameter when creating snapshots","message":"Add evaluation of --force parameter when creating snapshots\n\nRaise BadRequest Exception if value of --force parameter in\nsnapshot-create is invalid\n\nFixes: bug #1014689\n\nChange-Id: If4858dc17cedf027112defb935016137727681cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/f64623354a8176f0182ce4e0f41b19bd6bafc832"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/f64623354a8176f0182ce4e0f41b19bd6bafc832"}]},"branch":"refs/heads/master"},"8c459e1da974e15a15b3d302bfea97f6f7eed738":{"kind":"REWORK","_number":5,"created":"2013-07-28 15:29:18.000000000","uploader":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"ref":"refs/changes/02/38002/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/02/38002/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/5"}}},"commit":{"parents":[{"commit":"878098d6129b7c606e4a55cde4f316d5da2d532d","subject":"Merge \"Add print to the upload-to-image command\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/878098d6129b7c606e4a55cde4f316d5da2d532d"}]}],"author":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:01:41.000000000","tz":0},"committer":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-28 15:26:11.000000000","tz":0},"subject":"Add evaluation of --force parameter when creating snapshots","message":"Add evaluation of --force parameter when creating snapshots\n\nRaise BadRequest Exception if value of --force parameter in\nsnapshot-create is invalid\n\nFixes: bug #1014689\n\nChange-Id: If4858dc17cedf027112defb935016137727681cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8c459e1da974e15a15b3d302bfea97f6f7eed738"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8c459e1da974e15a15b3d302bfea97f6f7eed738"}]},"branch":"refs/heads/master"},"01f28da5c43f5a5e2e2d9e3df0b69b2f81f50286":{"kind":"REWORK","_number":6,"created":"2013-07-28 15:44:13.000000000","uploader":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"ref":"refs/changes/02/38002/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/02/38002/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/6"}}},"commit":{"parents":[{"commit":"878098d6129b7c606e4a55cde4f316d5da2d532d","subject":"Merge \"Add print to the upload-to-image command\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/878098d6129b7c606e4a55cde4f316d5da2d532d"}]}],"author":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:01:41.000000000","tz":0},"committer":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-28 15:42:05.000000000","tz":0},"subject":"Add evaluation of --force parameter when creating snapshots","message":"Add evaluation of --force parameter when creating snapshots\n\nRaise BadRequest Exception if value of --force parameter in\nsnapshot-create is invalid\n\nFixes: bug #1014689\n\nChange-Id: If4858dc17cedf027112defb935016137727681cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/01f28da5c43f5a5e2e2d9e3df0b69b2f81f50286"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/01f28da5c43f5a5e2e2d9e3df0b69b2f81f50286"}]},"branch":"refs/heads/master"},"854740c4c793444c955180e7c0c96977e3e06c56":{"kind":"REWORK","_number":7,"created":"2013-07-28 16:20:46.000000000","uploader":{"_account_id":6743,"name":"Seif Lotfy","email":"s.lotfy@telekom.de","username":"seif"},"ref":"refs/changes/02/38002/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/02/38002/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/02/38002/7"}}},"commit":{"parents":[{"commit":"878098d6129b7c606e4a55cde4f316d5da2d532d","subject":"Merge \"Add print to the upload-to-image command\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/878098d6129b7c606e4a55cde4f316d5da2d532d"}]}],"author":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-18 18:01:41.000000000","tz":0},"committer":{"name":"Seif Lotfy","email":"s.lotfy@telekom.de","date":"2013-07-28 16:18:27.000000000","tz":0},"subject":"Add evaluation of --force parameter when creating snapshots","message":"Add evaluation of --force parameter when creating snapshots\n\nRaise BadRequest Exception if value of --force parameter in\nsnapshot-create is invalid\n\nFixes: bug #1014689\n\nChange-Id: If4858dc17cedf027112defb935016137727681cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/854740c4c793444c955180e7c0c96977e3e06c56"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/854740c4c793444c955180e7c0c96977e3e06c56"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}