)]}'
{"id":"openstack%2Fpython-cinderclient~362711","triplet_id":"openstack%2Fpython-cinderclient~master~I2c800099e8ae707135417f9821f14d1a9e69586e","project":"openstack/python-cinderclient","branch":"master","topic":"bug/1618360","hashtags":[],"change_id":"I2c800099e8ae707135417f9821f14d1a9e69586e","subject":"Wrap volume_backup\u0027s update function with api_version","status":"MERGED","created":"2016-08-30 08:53:32.000000000","updated":"2016-09-01 02:23:39.000000000","submitted":"2016-09-01 02:23:39.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"5cb883851b1a3b853124007b95a946655b48b04d","_number":362711,"virtual_id_number":362711,"owner":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":2,"date":"2016-09-01 02:23:39.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"}],"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":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":2,"date":"2016-09-01 01:15:39.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2016-08-31 21:43:12.000000000","_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"}],"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":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":1,"date":"2016-09-01 01:15:39.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"}],"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":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-08-31 21:43:12.000000000","updated_by":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"reviewer":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"state":"REVIEWER"},{"updated":"2016-09-01 01:15:39.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":"2016-09-01 02:23:39.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"bb581aab180b4bc279473a5c8d94f19e644947ca","author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"date":"2016-08-30 08:53:32.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"ce28df6528f37277bca93c251b1189b716a4d180","author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"date":"2016-08-30 09:40:31.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"0ada05b3f6ad8732b5e1dabd5d39046adad30ac2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-30 10:43:02.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/11/362711/2/check/gate-python-cinderclient-docs-ubuntu-xenial/fa7a0b2//doc/build/html/ : SUCCESS in 2m 24s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/11/362711/2/check/gate-python-cinderclient-pep8-ubuntu-xenial/6e9ab47/ : SUCCESS in 3m 23s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/11/362711/2/check/gate-python-cinderclient-python27-ubuntu-xenial/81f1a03/ : SUCCESS in 3m 45s\n- gate-python-cinderclient-python34 http://logs.openstack.org/11/362711/2/check/gate-python-cinderclient-python34/a548aba/ : SUCCESS in 2m 48s\n- gate-python-cinderclient-python35-nv http://logs.openstack.org/11/362711/2/check/gate-python-cinderclient-python35-nv/5c1433d/ : SUCCESS in 5m 48s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/11/362711/2/check/gate-tempest-dsvm-neutron-src-python-cinderclient/0551394/ : SUCCESS in 1h 01m 35s\n- gate-python-cinderclient-pypy http://logs.openstack.org/11/362711/2/check/gate-python-cinderclient-pypy/6d2630e/ : SUCCESS in 2m 48s (non-voting)\n- gate-cinderclient-dsvm-functional http://logs.openstack.org/11/362711/2/check/gate-cinderclient-dsvm-functional/7df6f39/ : SUCCESS in 14m 04s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/11/362711/2/check/gate-cinderclient-dsvm-functional-identity-v3-only-nv/afce359/ : SUCCESS in 22m 13s (non-voting)\n- gate-python-cinderclient-pylint http://logs.openstack.org/11/362711/2/check/gate-python-cinderclient-pylint/fe679c5/ : SUCCESS in 4m 25s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"c4fbb004f594b2df449b046138855712c67aee1a","author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"date":"2016-08-31 21:43:12.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"7dd0f8c92c83f63f5e1fa29ae3b171d64650623c","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2016-09-01 01:15:39.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nI think lib users need to be more aware of what versions they are using, more than CLI users, but this is good to have.","accounts_in_message":[],"_revision_number":2},{"id":"84c013dd7659753b900ad6686b64788ceaf3bb5b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-01 01:15:50.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"aa496f48226dc61d3b5211b1b840d53d54547839","date":"2016-09-01 02:23:39.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2},{"id":"278e3058a1143055ba61634f59c5f0e64c516a5f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-01 02:23:39.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/11/362711/2/gate/gate-python-cinderclient-docs-ubuntu-xenial/be3dc23//doc/build/html/ : SUCCESS in 3m 26s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/11/362711/2/gate/gate-python-cinderclient-pep8-ubuntu-xenial/ff50ce0/ : SUCCESS in 2m 21s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/11/362711/2/gate/gate-python-cinderclient-python27-ubuntu-xenial/28a09b4/ : SUCCESS in 2m 44s\n- gate-python-cinderclient-python34 http://logs.openstack.org/11/362711/2/gate/gate-python-cinderclient-python34/dfda97a/ : SUCCESS in 3m 33s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/11/362711/2/gate/gate-tempest-dsvm-neutron-src-python-cinderclient/0c81fba/ : SUCCESS in 1h 05m 09s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"654f71208b3a270df0c7c9d66e2f90d3ecc4c162","revisions":{"c4fbf2beb696555c9fdbc76aef9970cda45c7aa9":{"kind":"REWORK","_number":1,"created":"2016-08-30 08:53:32.000000000","uploader":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"ref":"refs/changes/11/362711/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/11/362711/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/11/362711/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/11/362711/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/11/362711/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/11/362711/1"}}},"commit":{"parents":[{"commit":"4f794410f1912090b85ec6332ea82feca8b60634","subject":"Merge \"Replace functions \u0027Dict.get\u0027 and \u0027del\u0027 with \u0027Dict.pop\u0027\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/4f794410f1912090b85ec6332ea82feca8b60634"}]}],"author":{"name":"Cao Shufeng","email":"caosf.fnst@cn.fujitsu.com","date":"2016-08-30 11:38:14.000000000","tz":-240},"committer":{"name":"Cao Shufeng","email":"caosf.fnst@cn.fujitsu.com","date":"2016-08-30 11:39:44.000000000","tz":-240},"subject":"Wrap volume_backup\u0027s update function with api_version","message":"Wrap volume_backup\u0027s update function with api_version\n\nVolume backup\u0027s update[1] function only supports 3.9 micro version or higher.\nSo it should be wrapped like like this one[2].\n\n[1]: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volume_backups.py#L134\n[2]: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/services.py#L82\n\nChange-Id: I2c800099e8ae707135417f9821f14d1a9e69586e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c4fbf2beb696555c9fdbc76aef9970cda45c7aa9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c4fbf2beb696555c9fdbc76aef9970cda45c7aa9"}]},"branch":"refs/heads/master"},"654f71208b3a270df0c7c9d66e2f90d3ecc4c162":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2016-08-30 09:40:31.000000000","uploader":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"ref":"refs/changes/11/362711/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/11/362711/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/11/362711/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/11/362711/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/11/362711/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/11/362711/2"}}},"commit":{"parents":[{"commit":"4f794410f1912090b85ec6332ea82feca8b60634","subject":"Merge \"Replace functions \u0027Dict.get\u0027 and \u0027del\u0027 with \u0027Dict.pop\u0027\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/4f794410f1912090b85ec6332ea82feca8b60634"}]}],"author":{"name":"Cao Shufeng","email":"caosf.fnst@cn.fujitsu.com","date":"2016-08-30 11:38:14.000000000","tz":-240},"committer":{"name":"Cao Shufeng","email":"caosf.fnst@cn.fujitsu.com","date":"2016-08-30 12:27:12.000000000","tz":-240},"subject":"Wrap volume_backup\u0027s update function with api_version","message":"Wrap volume_backup\u0027s update function with api_version\n\nVolume backup\u0027s update function only supports 3.9 microversion\nor higher. So it should be wrapped like like this one[1],\notherwise these fuctions may be exposed to lib users with old\nmicroversion.\n\n[1]: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/services.py#L82\n\nChange-Id: I2c800099e8ae707135417f9821f14d1a9e69586e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/654f71208b3a270df0c7c9d66e2f90d3ecc4c162"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/654f71208b3a270df0c7c9d66e2f90d3ecc4c162"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}