)]}'
{"id":"openstack%2Fpython-cinderclient~49234","triplet_id":"openstack%2Fpython-cinderclient~master~I8b978a6ecd87b81c6ae34938e37623ad1cc33994","project":"openstack/python-cinderclient","branch":"master","topic":"bug/1233311","hashtags":[],"change_id":"I8b978a6ecd87b81c6ae34938e37623ad1cc33994","subject":"Override endpoint URL check for API version","status":"MERGED","created":"2013-10-01 18:46:20.000000000","updated":"2013-10-19 20:04:55.000000000","submitted":"2013-10-19 20:04:55.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"af46e4eeeedeec7ca97b763ca61ccf698ed6f26b","_number":49234,"virtual_id_number":49234,"owner":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-10-19 20:04:55.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6499,"name":"yehia-beyh","email":"yehia.beyh@hpe.com","username":"yehia-beyh"},{"value":0,"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},{"value":0,"_account_id":7482,"name":"Jenny Shieh","email":"jenny.shieh@hp.com","username":"jenny-shieh"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6499,"name":"yehia-beyh","email":"yehia.beyh@hpe.com","username":"yehia-beyh"},{"value":2,"date":"2013-10-19 19:22:16.000000000","_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":2,"date":"2013-10-17 23:50:59.000000000","_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},{"value":0,"_account_id":7482,"name":"Jenny Shieh","email":"jenny.shieh@hp.com","username":"jenny-shieh"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6499,"name":"yehia-beyh","email":"yehia.beyh@hpe.com","username":"yehia-beyh"},{"value":1,"date":"2013-10-19 19:22:16.000000000","_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"date":"2013-10-17 23:50:59.000000000","_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},{"value":0,"_account_id":7482,"name":"Jenny Shieh","email":"jenny.shieh@hp.com","username":"jenny-shieh"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6499,"name":"yehia-beyh","email":"yehia.beyh@hpe.com","username":"yehia-beyh"},{"value":0,"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},{"value":0,"_account_id":7482,"name":"Jenny Shieh","email":"jenny.shieh@hp.com","username":"jenny-shieh"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":6499,"name":"yehia-beyh","email":"yehia.beyh@hpe.com","username":"yehia-beyh"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":7482,"name":"Jenny Shieh","email":"jenny.shieh@hp.com","username":"jenny-shieh"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-10-09 16:15:07.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":"2013-10-09 19:51:53.000000000","updated_by":{"_account_id":7482,"name":"Jenny Shieh","email":"jenny.shieh@hp.com","username":"jenny-shieh"},"reviewer":{"_account_id":7482,"name":"Jenny Shieh","email":"jenny.shieh@hp.com","username":"jenny-shieh"},"state":"REVIEWER"},{"updated":"2013-10-09 19:52:36.000000000","updated_by":{"_account_id":6499,"name":"yehia-beyh","email":"yehia.beyh@hpe.com","username":"yehia-beyh"},"reviewer":{"_account_id":6499,"name":"yehia-beyh","email":"yehia.beyh@hpe.com","username":"yehia-beyh"},"state":"REVIEWER"},{"updated":"2013-10-17 03:16:11.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-10-17 23:50:59.000000000","updated_by":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"reviewer":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"state":"REVIEWER"},{"updated":"2013-10-19 19:22:16.000000000","updated_by":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"reviewer":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"state":"REVIEWER"},{"updated":"2013-10-19 20:04:55.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"69ee08692d4ab78d579b0a1e9c6d225970fe71ca","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-01 19:40:03.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-pep8/5484389 : SUCCESS in 48s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/34/49234/1/check/gate-python-cinderclient-docs/0603ed5/doc/build/html/ : SUCCESS in 57s\n- gate-python-cinderclient-python26 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python26/54caab3 : SUCCESS in 1m 01s\n- gate-python-cinderclient-python27 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python27/dff6fa3 : SUCCESS in 1m 07s\n- gate-python-cinderclient-python33 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python33/730d70a : FAILURE in 4m 02s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-full/4a5af84 : FAILURE in 47m 33s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-postgres-full/588d786 : SUCCESS in 43m 37s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-neutron/0932948 : SUCCESS in 26m 23s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/34/49234/1/check/gate-tempest-devstack-vm-large-ops/223bb9c : SUCCESS in 12m 15s\n- check-grenade-devstack-vm http://logs.openstack.org/34/49234/1/check/check-grenade-devstack-vm/ae49bf6 : SUCCESS in 36m 32s\n","accounts_in_message":[],"_revision_number":1},{"id":"c34f20a3c962051855497b6447b50656c1acb5a6","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-01 20:31:46.000000000","message":"Patch Set 1:\n\nrecheck bug 1233743","accounts_in_message":[],"_revision_number":1},{"id":"56c82d3757e3a73b79a2d7bd651c713966534bc1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-01 21:18:45.000000000","message":"Patch Set 1:\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-pep8/36f1c5c : SUCCESS in 57s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/34/49234/1/check/gate-python-cinderclient-docs/47c658f/doc/build/html/ : SUCCESS in 57s\n- gate-python-cinderclient-python26 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python26/293aac6 : SUCCESS in 1m 13s\n- gate-python-cinderclient-python27 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python27/3190314 : SUCCESS in 50s\n- gate-python-cinderclient-python33 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python33/a00ff0f : FAILURE in 4m 02s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-full/8007f7b : SUCCESS in 42m 16s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-postgres-full/bb9b5c5 : FAILURE in 41m 43s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-neutron/80f5360 : SUCCESS in 28m 14s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/34/49234/1/check/gate-tempest-devstack-vm-large-ops/4529163 : SUCCESS in 16m 26s\n- check-grenade-devstack-vm http://logs.openstack.org/34/49234/1/check/check-grenade-devstack-vm/43fce37 : SUCCESS in 36m 09s\n","accounts_in_message":[],"_revision_number":1},{"id":"8791f2dfa5a95a0b1fc3d042e502cf06f3112c26","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-01 23:46:06.000000000","message":"Patch Set 1:\n\nrecheck bug 1230354","accounts_in_message":[],"_revision_number":1},{"id":"4e840d3f92b8d4097dc8557cc5dc34d10ae5df19","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-02 00:51:52.000000000","message":"Patch Set 1:\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-pep8/a4870f2 : SUCCESS in 59s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/34/49234/1/check/gate-python-cinderclient-docs/96cfaf7/doc/build/html/ : SUCCESS in 4m 20s\n- gate-python-cinderclient-python26 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python26/aebce46 : SUCCESS in 1m 08s\n- gate-python-cinderclient-python27 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python27/3a3df9a : SUCCESS in 2m 01s\n- gate-python-cinderclient-python33 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python33/b936aa7 : FAILURE in 4m 06s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-full/f11776e : FAILURE in 51m 53s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-postgres-full/ef0c87c : FAILURE in 52m 35s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-neutron/6a3abf3 : FAILURE in 47m 37s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/34/49234/1/check/gate-tempest-devstack-vm-large-ops/8d26d58 : SUCCESS in 30m 45s\n- check-grenade-devstack-vm http://logs.openstack.org/34/49234/1/check/check-grenade-devstack-vm/378bd29 : SUCCESS in 43m 21s\n","accounts_in_message":[],"_revision_number":1},{"id":"732e019a3444f8785872d520865b45f7768d5b09","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-02 04:07:07.000000000","message":"Patch Set 1:\n\nrecheck bug 1223161","accounts_in_message":[],"_revision_number":1},{"id":"1ebe37ca4af959523820b609aca4710c8e6bfb6e","author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"date":"2013-10-02 04:38:28.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\nLooks OK to me, but not an expert on this.","accounts_in_message":[],"_revision_number":1},{"id":"a0ff41f41e70b2c6eeaa437606de6c23d0d21663","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-02 04:49:24.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-pep8/6a3ad91 : SUCCESS in 37s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/34/49234/1/check/gate-python-cinderclient-docs/d30027f/doc/build/html/ : SUCCESS in 54s\n- gate-python-cinderclient-python26 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python26/d09eda3 : SUCCESS in 1m 06s\n- gate-python-cinderclient-python27 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python27/3c99d9b : SUCCESS in 37s\n- gate-python-cinderclient-python33 http://logs.openstack.org/34/49234/1/check/gate-python-cinderclient-python33/4f4e10b : FAILURE in 4m 00s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-full/fd40247 : SUCCESS in 41m 37s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-postgres-full/6912623 : SUCCESS in 33m 40s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/34/49234/1/check/check-tempest-devstack-vm-neutron/89954f3 : SUCCESS in 22m 32s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/34/49234/1/check/gate-tempest-devstack-vm-large-ops/7223668 : SUCCESS in 11m 00s\n- check-grenade-devstack-vm http://logs.openstack.org/34/49234/1/check/check-grenade-devstack-vm/02ee696 : SUCCESS in 32m 31s\n","accounts_in_message":[],"_revision_number":1},{"id":"a0ed1b7c0c8e3fcd2f3c684e7e8c76fda843f228","author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"date":"2013-10-09 06:18:01.000000000","message":"Patch Set 1: No score\n\nhttps://review.openstack.org/47624  solves the same bug - need to choose the better solution","accounts_in_message":[],"_revision_number":1},{"id":"7b76d97ec2e2fcfc04650c272df61d00aa8b9f58","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-10-09 16:15:07.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"9057eacfb82a02e030306e6ff913ffac745b309a","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-09 19:41:49.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"e5ffd5100d4140c8443a2d9aaf7e9e8b6bff1b47","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-09 19:51:40.000000000","message":"Patch Set 1: (1 inline comment)\n\nA few notes here.  First, Eric, I implemented your recommendation.  Thanks for the comment.\n\nSecond, with regards to Avishay\u0027s question about which way to go, John Griffith suggested that we go with both solutions.  Update get_volume_api_version_from_endpoint via https://review.openstack.org/47624 and leave this change.  This way, overriding the OS_VOLUME_API_VERSION shouldn\u0027t be necessary, but if there is a customer using some custom URL that doesn\u0027t work with the search of the URL, they can easily work around it by setting the OS_VOLUME_API_VERSION in the environment.\n\nThere was concern raised that this may break the normal function of cinder client by having the environmental setting override something set on the cinder command line, but it appears to me that that is the way it is currently functioning.","accounts_in_message":[],"_revision_number":1},{"id":"c3792df7077b9d40bae4bfb2f7becb681b3ea129","author":{"_account_id":7482,"name":"Jenny Shieh","email":"jenny.shieh@hp.com","username":"jenny-shieh"},"date":"2013-10-09 19:55:01.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"4b739a05e5439de1e5b1fb32577a956aca18655d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-09 20:22:08.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/34/49234/2/check/gate-python-cinderclient-pep8/3017d85 : SUCCESS in 43s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/34/49234/2/check/gate-python-cinderclient-docs/4b66658/doc/build/html/ : SUCCESS in 1m 10s\n- gate-python-cinderclient-python26 http://logs.openstack.org/34/49234/2/check/gate-python-cinderclient-python26/8330224 : SUCCESS in 1m 11s\n- gate-python-cinderclient-python27 http://logs.openstack.org/34/49234/2/check/gate-python-cinderclient-python27/7054b9e : SUCCESS in 43s\n- gate-python-cinderclient-python33 http://logs.openstack.org/34/49234/2/check/gate-python-cinderclient-python33/3c7da9f : FAILURE in 4m 08s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/34/49234/2/check/check-tempest-devstack-vm-full/39470ab : SUCCESS in 39m 02s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/34/49234/2/check/check-tempest-devstack-vm-postgres-full/df8c335 : SUCCESS in 32m 07s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/34/49234/2/check/check-tempest-devstack-vm-neutron/ed5f573 : SUCCESS in 25m 52s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/34/49234/2/check/gate-tempest-devstack-vm-large-ops/4502c5d : SUCCESS in 10m 58s\n- check-grenade-devstack-vm http://logs.openstack.org/34/49234/2/check/check-grenade-devstack-vm/c34da12 : SUCCESS in 40m 12s\n","accounts_in_message":[],"_revision_number":2},{"id":"da45c790dc0334740793183485d0d598ac7a67ca","author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"date":"2013-10-10 01:09:26.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nThis change will totally make mgmt url checking function gone when OS_VOLUME_API_VERSION is set in the environment.","accounts_in_message":[],"_revision_number":2},{"id":"de8d8f8c6d1836d605e7a0befbb6aaa70a9ec0bc","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-10 02:00:22.000000000","message":"Patch Set 2:\n\nThanks for the feedback Winston.  As we discussed on IRC I will submit a patch tomorrow that only uses the environment variable as an override in the case that the URL parsing fails.","accounts_in_message":[],"_revision_number":2},{"id":"fd4ee709d8d6ab4d69a70d5db33296db0192b345","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-10 22:09:29.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"fc3d41e0818bd2f95e566b36dbcc8d17eea2234e","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-10 22:12:34.000000000","message":"Patch Set 3:\n\nNew patch set uploaded that only check for OS_VOLUME_API_VERSION if getting the version of the API fails.  It also, leave in place the functionality of checking the API version being used against what is set by default or set with --os-volume-api-version.  That way there won\u0027t be mismatches.","accounts_in_message":[],"_revision_number":3},{"id":"e4dba28afea0ec118cb58440e63beb94ed2c34ef","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-10 23:12:51.000000000","message":"Patch Set 3: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/34/49234/3/check/gate-python-cinderclient-pep8/2f4bb62 : SUCCESS in 36s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/34/49234/3/check/gate-python-cinderclient-docs/72e4dce/doc/build/html/ : SUCCESS in 35s\n- gate-python-cinderclient-python26 http://logs.openstack.org/34/49234/3/check/gate-python-cinderclient-python26/f9860f1 : SUCCESS in 41s\n- gate-python-cinderclient-python27 http://logs.openstack.org/34/49234/3/check/gate-python-cinderclient-python27/830aac8 : SUCCESS in 38s\n- gate-python-cinderclient-python33 http://logs.openstack.org/34/49234/3/check/gate-python-cinderclient-python33/cd79020 : FAILURE in 4m 02s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/34/49234/3/check/check-tempest-devstack-vm-full/9a0311d : FAILURE in 1h 01m 16s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/34/49234/3/check/check-tempest-devstack-vm-postgres-full/9a16dd3 : SUCCESS in 42m 02s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/34/49234/3/check/check-tempest-devstack-vm-neutron/95485b2 : SUCCESS in 26m 58s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/34/49234/3/check/gate-tempest-devstack-vm-large-ops/e0ad27b : SUCCESS in 23m 04s\n- check-grenade-devstack-vm http://logs.openstack.org/34/49234/3/check/check-grenade-devstack-vm/fcb3802 : SUCCESS in 38m 09s\n","accounts_in_message":[],"_revision_number":3},{"id":"60320c7e2a78c5f5343795bdb3e73b076544e4aa","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-10 23:45:15.000000000","message":"Patch Set 3:\n\nrecheck bug 1232283","accounts_in_message":[],"_revision_number":3},{"id":"db7dfeb9fbf547a24eb6c06203e973826b03283e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-11 00:29:01.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/34/49234/3/check/gate-python-cinderclient-pep8/800436c : SUCCESS in 49s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/34/49234/3/check/gate-python-cinderclient-docs/7f4cee5/doc/build/html/ : SUCCESS in 32s\n- gate-python-cinderclient-python26 http://logs.openstack.org/34/49234/3/check/gate-python-cinderclient-python26/3283b38 : SUCCESS in 1m 02s\n- gate-python-cinderclient-python27 http://logs.openstack.org/34/49234/3/check/gate-python-cinderclient-python27/1395fce : SUCCESS in 45s\n- gate-python-cinderclient-python33 http://logs.openstack.org/34/49234/3/check/gate-python-cinderclient-python33/a5e9b12 : FAILURE in 4m 04s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/34/49234/3/check/check-tempest-devstack-vm-full/4fc306a : SUCCESS in 41m 38s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/34/49234/3/check/check-tempest-devstack-vm-postgres-full/10c5b44 : SUCCESS in 43m 42s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/34/49234/3/check/check-tempest-devstack-vm-neutron/0b1b835 : SUCCESS in 25m 02s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/34/49234/3/check/gate-tempest-devstack-vm-large-ops/7896704 : SUCCESS in 16m 50s\n- check-grenade-devstack-vm http://logs.openstack.org/34/49234/3/check/check-grenade-devstack-vm/62aaea0 : SUCCESS in 23m 41s\n","accounts_in_message":[],"_revision_number":3},{"id":"05702527ecb19951a85c4a120937e0c23bba3dae","author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"date":"2013-10-12 08:23:25.000000000","message":"Patch Set 3:\n\nJay, thanks for making the change according to what we discussed.  I kept thinking if we should also allow user to work around the issue using \u0027--os-volume-api-version\u0027 optional argument when failed to extract API version from endpoint. It\u0027ll be handy. And the change could be like this: http://paste.openstack.org/show/48329/  Does that make sense to you?","accounts_in_message":[],"_revision_number":3},{"id":"7bd3f3069664f3816790ba34a09732a8d1a7bca2","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-16 23:04:46.000000000","message":"Patch Set 3:\n\nWinston, Sorry I haven\u0027t had a chance to respond to your idea in here.  Been a wild week.\n\nI had actually been thinking about doing something similar.  I like the idea.  Let me add that into this patch and see how it goes down.","accounts_in_message":[],"_revision_number":3},{"id":"8cebe9a7836374969f31ce2fffd18ee5b71976ee","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-17 02:57:55.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"02c493fa2fc9664dc980f81b95b1e546273c204e","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-17 02:59:38.000000000","message":"Patch Set 4:\n\nGreat idea Winston.  Now the cli does both what you and I were hoping for.  --os-volume-api-version is the master, followed by the ENV variable and then it falls back to the default if necessary.  Mismatches against the URL are still caught.  I like this solution.  Hope you do too.  :-)","accounts_in_message":[],"_revision_number":4},{"id":"eef92ffe1fd19cbb680c8bf384880c1600f5566a","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2013-10-17 03:16:11.000000000","message":"Patch Set 4: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nMinor nit inline regarding logger.warn and logger.warning","accounts_in_message":[],"_revision_number":4},{"id":"fc3bc230869eb1394c1c314208d70803446e4d5b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-17 03:52:20.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/34/49234/4/check/gate-python-cinderclient-pep8/ed5e72d : SUCCESS in 47s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/34/49234/4/check/gate-python-cinderclient-docs/2b433f3/doc/build/html/ : SUCCESS in 44s\n- gate-python-cinderclient-python26 http://logs.openstack.org/34/49234/4/check/gate-python-cinderclient-python26/19906e4 : SUCCESS in 54s\n- gate-python-cinderclient-python27 http://logs.openstack.org/34/49234/4/check/gate-python-cinderclient-python27/94e6a0f : SUCCESS in 50s\n- gate-python-cinderclient-python33 http://logs.openstack.org/34/49234/4/check/gate-python-cinderclient-python33/4f1ad0f : FAILURE in 4m 03s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/34/49234/4/check/check-tempest-devstack-vm-full/88f3077 : SUCCESS in 51m 42s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/34/49234/4/check/check-tempest-devstack-vm-postgres-full/a37800a : SUCCESS in 54m 21s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/34/49234/4/check/check-tempest-devstack-vm-neutron/e4c73c6 : SUCCESS in 18m 10s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/34/49234/4/check/gate-tempest-devstack-vm-large-ops/947c4c6 : SUCCESS in 15m 44s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/34/49234/4/check/gate-tempest-devstack-vm-neutron-large-ops/69f2d94 : SUCCESS in 16m 31s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/34/49234/4/check/check-grenade-devstack-vm/b1e5057 : SUCCESS in 32m 08s\n","accounts_in_message":[],"_revision_number":4},{"id":"47d1901d31c4c6ae6ad54615c2ce045dbf0c72f1","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-17 19:48:04.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"2cb4514a114c02d8474a20d5c445c51d1e01a9b7","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-17 19:48:27.000000000","message":"Patch Set 4: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"f07e4f4fe9dc18c5f3d6267fde2200d519cac2de","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-10-17 20:05:46.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"dcf5e881bd20841fdb5a67408448fb0fec80be7d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-17 20:45:56.000000000","message":"Patch Set 6: Works for me\n\nBuild succeeded.\n\n- gate-python-cinderclient-pep8 http://logs.openstack.org/34/49234/6/check/gate-python-cinderclient-pep8/3ade4c3 : SUCCESS in 41s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/34/49234/6/check/gate-python-cinderclient-docs/45088a6/doc/build/html/ : SUCCESS in 41s\n- gate-python-cinderclient-python26 http://logs.openstack.org/34/49234/6/check/gate-python-cinderclient-python26/78040a6 : SUCCESS in 51s\n- gate-python-cinderclient-python27 http://logs.openstack.org/34/49234/6/check/gate-python-cinderclient-python27/4370241 : SUCCESS in 36s\n- gate-python-cinderclient-python33 http://logs.openstack.org/34/49234/6/check/gate-python-cinderclient-python33/87979b6 : FAILURE in 4m 00s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/34/49234/6/check/check-tempest-devstack-vm-full/28bbcb2 : SUCCESS in 40m 06s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/34/49234/6/check/check-tempest-devstack-vm-postgres-full/fe54b35 : SUCCESS in 38m 34s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/34/49234/6/check/check-tempest-devstack-vm-neutron/ec8c165 : SUCCESS in 21m 26s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/34/49234/6/check/gate-tempest-devstack-vm-large-ops/4844797 : SUCCESS in 14m 09s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/34/49234/6/check/gate-tempest-devstack-vm-neutron-large-ops/8ce50ff : SUCCESS in 15m 59s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/34/49234/6/check/check-grenade-devstack-vm/c922269 : SUCCESS in 26m 18s\n","accounts_in_message":[],"_revision_number":6},{"id":"c290796fcff3dc3b1f15dc936cfd427a822fce4c","author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"date":"2013-10-17 23:50:59.000000000","message":"Patch Set 6: Looks good to me (core reviewer)\n\nLooks good to me, thanks for taking care of this.","accounts_in_message":[],"_revision_number":6},{"id":"ab4d16ba426fa21b7f1f7c04857538f45c467645","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-19 19:22:16.000000000","message":"Patch Set 6: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":6},{"id":"a5c5a7bc97ae6c43d722977518f26cf5bfaf98dc","author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"date":"2013-10-19 19:22:16.000000000","message":"Patch Set 6: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":6},{"id":"23d35535733926aacb323a47c1d67e04e0c2ec6d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-19 20:04:55.000000000","message":"Patch Set 6: Verified\n\nBuild succeeded.\n\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/34/49234/6/gate/gate-python-cinderclient-docs/c9662b9/doc/build/html/ : SUCCESS in 42s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/34/49234/6/gate/gate-python-cinderclient-pep8/e5d5280 : SUCCESS in 34s\n- gate-python-cinderclient-python26 http://logs.openstack.org/34/49234/6/gate/gate-python-cinderclient-python26/17d9854 : SUCCESS in 55s\n- gate-python-cinderclient-python27 http://logs.openstack.org/34/49234/6/gate/gate-python-cinderclient-python27/e40aba0 : SUCCESS in 55s\n- gate-tempest-devstack-vm-full http://logs.openstack.org/34/49234/6/gate/gate-tempest-devstack-vm-full/e31b42d : SUCCESS in 32m 16s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/34/49234/6/gate/gate-tempest-devstack-vm-postgres-full/41f96d3 : SUCCESS in 34m 37s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/34/49234/6/gate/gate-tempest-devstack-vm-neutron/a98ea8f : SUCCESS in 18m 31s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/34/49234/6/gate/gate-tempest-devstack-vm-large-ops/d103a4f : SUCCESS in 13m 33s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/34/49234/6/gate/gate-tempest-devstack-vm-neutron-large-ops/2780821 : SUCCESS in 15m 57s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/34/49234/6/gate/gate-grenade-devstack-vm/f73ad3d : SUCCESS in 24m 14s\n","accounts_in_message":[],"_revision_number":6},{"id":"9dc20c551aefea85f519b7eb88d62cf3d1defaeb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-19 20:04:55.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"84ea630be01cc8a847ba73daa90f609fcc5aa00b","revisions":{"ea3ecc617057db6f4bab605dbb60f4228545a2fe":{"kind":"REWORK","_number":1,"created":"2013-10-01 18:46:20.000000000","uploader":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"ref":"refs/changes/34/49234/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/34/49234/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/1"}}},"commit":{"parents":[{"commit":"665abd9acc4a266cf668c4562cf30fb85bf7cf2a","subject":"Merge \"Add volume name arguments\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/665abd9acc4a266cf668c4562cf30fb85bf7cf2a"}]}],"author":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-01 18:34:00.000000000","tz":-300},"committer":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-01 18:34:00.000000000","tz":-300},"subject":"Override endpoint URL check for API version","message":"Override endpoint URL check for API version\n\nget_volume_api_version_from_endpoint currently assumes that the\nmanagement_url will always be in the format of\nhttp://\u003cip\u003e:\u003cport\u003e/vX/\u003ctenant_id\u003e. Customers using an IaaS gateway, however,\nget a mangement_url in the form of\nhttps://\u003cip\u003e:\u003cport\u003e/\u003cresource\u003e/vX/\u003ctenant_id\u003e.  This causes the\nget_volume_api_version_from_endpoint to find the resource portion of the\nurl when looking for the Volume API version which makes the\ncinder client unusable.\n\nTo resolves this I have implemented a check for OS_VOLUME_API_VERSION being\nset in the environment.  If the variable is manually set we assume the\nuser has purposefully set the variable and use it as the os_volume_api_version,\nskipping the checking/parsing of the management_url.\n\nThere are proposals to fix this via changes to keystone in future releases,\nbut this provides a way for IaaS Gateway users to proceed with Havana.\n\nChange-Id: I8b978a6ecd87b81c6ae34938e37623ad1cc33994\nCloses-Bug: 1233311\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/ea3ecc617057db6f4bab605dbb60f4228545a2fe"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/ea3ecc617057db6f4bab605dbb60f4228545a2fe"}]},"branch":"refs/heads/master"},"34032e56ce2c5d66249e9d26c7d5062c85a30b0c":{"kind":"REWORK","_number":2,"created":"2013-10-09 19:41:49.000000000","uploader":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"ref":"refs/changes/34/49234/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/34/49234/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/2"}}},"commit":{"parents":[{"commit":"4a507601d7dded2efb1bd2e885155ba38db9538c","subject":"Update docs/index.rst with release info for 1.0.6","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/4a507601d7dded2efb1bd2e885155ba38db9538c"}]}],"author":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-01 18:34:00.000000000","tz":-300},"committer":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-09 19:41:46.000000000","tz":-300},"subject":"Override endpoint URL check for API version","message":"Override endpoint URL check for API version\n\nget_volume_api_version_from_endpoint currently assumes that the\nmanagement_url will always be in the format of\nhttp://\u003cip\u003e:\u003cport\u003e/vX/\u003ctenant_id\u003e. Customers using an IaaS gateway, however,\nget a mangement_url in the form of\nhttps://\u003cip\u003e:\u003cport\u003e/\u003cresource\u003e/vX/\u003ctenant_id\u003e.  This causes the\nget_volume_api_version_from_endpoint to find the resource portion of the\nurl when looking for the Volume API version which makes the\ncinder client unusable.\n\nTo resolves this I have implemented a check for OS_VOLUME_API_VERSION being\nset in the environment.  If the variable is manually set we assume the\nuser has purposefully set the variable and use it as the os_volume_api_version,\nskipping the checking/parsing of the management_url.\n\nThere are proposals to fix this via changes to keystone in future releases,\nbut this provides a way for IaaS Gateway users to proceed with Havana.\n\nChange-Id: I8b978a6ecd87b81c6ae34938e37623ad1cc33994\nCloses-Bug: 1233311\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/34032e56ce2c5d66249e9d26c7d5062c85a30b0c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/34032e56ce2c5d66249e9d26c7d5062c85a30b0c"}]},"branch":"refs/heads/master"},"1d811bd2fbd494420a941c004b9beb13f69fb168":{"kind":"REWORK","_number":3,"created":"2013-10-10 22:09:29.000000000","uploader":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"ref":"refs/changes/34/49234/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/34/49234/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/3"}}},"commit":{"parents":[{"commit":"4a507601d7dded2efb1bd2e885155ba38db9538c","subject":"Update docs/index.rst with release info for 1.0.6","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/4a507601d7dded2efb1bd2e885155ba38db9538c"}]}],"author":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-01 18:34:00.000000000","tz":-300},"committer":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-10 22:03:30.000000000","tz":-300},"subject":"Override endpoint URL check for API version","message":"Override endpoint URL check for API version\n\nget_volume_api_version_from_endpoint currently assumes that the\nmanagement_url will always be in the format of\nhttp://\u003cip\u003e:\u003cport\u003e/vX/\u003ctenant_id\u003e. Customers using a gateway, however,\ncan get a mangement_url in a different form like\nhttps://\u003cip\u003e:\u003cport\u003e/\u003cresource\u003e/vX/\u003ctenant_id\u003e.  This causes the\nget_volume_api_version_from_endpoint to find the resource portion of the\nurl when looking for the Volume API version which makes the\ncinder client unusable.\n\nTo resolve this I have implemented an exception handler in the case that\nget_volume_api_version_from_endpoint fails.  If the exception is thrown, I\ncheck for OS_VOLUME_API_VERSION in the environment and use the value of that\nenvironment variable, if it is set, to determine the API version to use.\n\nThere are proposals to fix this via changes to keystone in future releases,\nbut this provides a way for IaaS Gateway users to proceed with Havana.\n\nChange-Id: I8b978a6ecd87b81c6ae34938e37623ad1cc33994\nCloses-Bug: 1233311\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/1d811bd2fbd494420a941c004b9beb13f69fb168"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/1d811bd2fbd494420a941c004b9beb13f69fb168"}]},"branch":"refs/heads/master"},"b0904cf647fc5fc0907d9e296dd9525db7d79f06":{"kind":"REWORK","_number":4,"created":"2013-10-17 02:57:55.000000000","uploader":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"ref":"refs/changes/34/49234/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/34/49234/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/4"}}},"commit":{"parents":[{"commit":"873bed99a8019f87bbce3edf883153e7f67e875e","subject":"Merge \"Fix the failure of fetching the version in cinder endpoint\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/873bed99a8019f87bbce3edf883153e7f67e875e"}]}],"author":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-01 18:34:00.000000000","tz":-300},"committer":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-17 02:57:51.000000000","tz":-300},"subject":"Override endpoint URL check for API version","message":"Override endpoint URL check for API version\n\nget_volume_api_version_from_endpoint currently assumes that the\nmanagement_url will always be in the format of\nhttp://\u003cip\u003e:\u003cport\u003e/vX/\u003ctenant_id\u003e. Customers using a gateway, however,\ncan get a mangement_url in a different form like\nhttps://\u003cip\u003e:\u003cport\u003e/\u003cresource\u003e/vX/\u003ctenant_id\u003e.  This causes the\nget_volume_api_version_from_endpoint to find the resource portion of the\nurl when looking for the Volume API version which makes the\ncinder client unusable.\n\nTo resolve this I have implemented an exception handler in the case that\nget_volume_api_version_from_endpoint fails.  If the exception is thrown, I\ncheck for OS_VOLUME_API_VERSION in the environment and use the value of that\nenvironment variable, if it is set, to determine the API version to use.\n\nThere are proposals to fix this via changes to keystone in future releases,\nbut this provides a way for IaaS Gateway users to proceed with Havana.\n\nChange-Id: I8b978a6ecd87b81c6ae34938e37623ad1cc33994\nCloses-Bug: 1233311\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b0904cf647fc5fc0907d9e296dd9525db7d79f06"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b0904cf647fc5fc0907d9e296dd9525db7d79f06"}]},"branch":"refs/heads/master"},"747fb6671345556dced40568fdd02ad64d2349bc":{"kind":"REWORK","_number":5,"created":"2013-10-17 19:48:04.000000000","uploader":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"ref":"refs/changes/34/49234/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/34/49234/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/5"}}},"commit":{"parents":[{"commit":"342022b5cf02aa2fa08b0bcce3cd93aa126f54e4","subject":"Merge \"python3: iteration order of dict is unpredictable\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/342022b5cf02aa2fa08b0bcce3cd93aa126f54e4"}]}],"author":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-01 18:34:00.000000000","tz":-300},"committer":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-17 19:48:00.000000000","tz":-300},"subject":"Override endpoint URL check for API version","message":"Override endpoint URL check for API version\n\nget_volume_api_version_from_endpoint currently assumes that the\nmanagement_url will always be in the format of\nhttp://\u003cip\u003e:\u003cport\u003e/vX/\u003ctenant_id\u003e. Customers using a gateway, however,\ncan get a mangement_url in a different form like\nhttps://\u003cip\u003e:\u003cport\u003e/\u003cresource\u003e/vX/\u003ctenant_id\u003e.  This causes the\nget_volume_api_version_from_endpoint to find the resource portion of the\nurl when looking for the Volume API version which makes the\ncinder client unusable.\n\nTo resolve this I have implemented an exception handler in the case that\nget_volume_api_version_from_endpoint fails.  If the exception is thrown, I\ncheck for OS_VOLUME_API_VERSION in the environment and use the value of that\nenvironment variable, if it is set, to determine the API version to use.\n\nThere are proposals to fix this via changes to keystone in future releases,\nbut this provides a way for IaaS Gateway users to proceed with Havana.\n\nChange-Id: I8b978a6ecd87b81c6ae34938e37623ad1cc33994\nCloses-Bug: 1233311\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/747fb6671345556dced40568fdd02ad64d2349bc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/747fb6671345556dced40568fdd02ad64d2349bc"}]},"branch":"refs/heads/master"},"84ea630be01cc8a847ba73daa90f609fcc5aa00b":{"kind":"REWORK","_number":6,"created":"2013-10-17 20:05:46.000000000","uploader":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"ref":"refs/changes/34/49234/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/34/49234/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/34/49234/6"}}},"commit":{"parents":[{"commit":"342022b5cf02aa2fa08b0bcce3cd93aa126f54e4","subject":"Merge \"python3: iteration order of dict is unpredictable\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/342022b5cf02aa2fa08b0bcce3cd93aa126f54e4"}]}],"author":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-01 18:34:00.000000000","tz":-300},"committer":{"name":"Jay S. Bryant","email":"jsbryant@us.ibm.com","date":"2013-10-17 20:03:55.000000000","tz":-300},"subject":"Override endpoint URL check for API version","message":"Override endpoint URL check for API version\n\nget_volume_api_version_from_endpoint currently assumes that the\nmanagement_url will always be in the format of\nhttp://\u003cip\u003e:\u003cport\u003e/vX/\u003ctenant_id\u003e. Customers using a gateway, however,\ncan get a mangement_url in a different form like\nhttps://\u003cip\u003e:\u003cport\u003e/\u003cresource\u003e/vX/\u003ctenant_id\u003e.  This causes the\nget_volume_api_version_from_endpoint to find the resource portion of the\nurl when looking for the Volume API version which makes the\ncinder client unusable.\n\nTo resolve this I have implemented an exception handler in the case that\nget_volume_api_version_from_endpoint fails.  If the exception is thrown, I\ncheck for OS_VOLUME_API_VERSION in the environment and use the value of that\nenvironment variable, if it is set, to determine the API version to use.\n\nThere are proposals to fix this via changes to keystone in future releases,\nbut this provides a way for IaaS Gateway users to proceed with Havana.\n\nChange-Id: I8b978a6ecd87b81c6ae34938e37623ad1cc33994\nCloses-Bug: 1233311\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/84ea630be01cc8a847ba73daa90f609fcc5aa00b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/84ea630be01cc8a847ba73daa90f609fcc5aa00b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}