)]}'
{"id":"openstack%2Fpython-openstackclient~372712","triplet_id":"openstack%2Fpython-openstackclient~stable%2Fnewton~Ia4eae350e6904c9eb2c8507d9b3429fe52418726","project":"openstack/python-openstackclient","branch":"stable/newton","topic":"bug/1619274","hashtags":[],"change_id":"Ia4eae350e6904c9eb2c8507d9b3429fe52418726","subject":"Defer auth prompting until it is actually needed","status":"MERGED","created":"2016-09-19 19:14:43.000000000","updated":"2016-12-09 23:25:13.000000000","submitted":"2016-12-09 23:25:13.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"fb4a927ff03460da3c5838d7e7bcc110ead99035","_number":372712,"virtual_id_number":372712,"owner":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},{"value":2,"date":"2016-12-09 23:25:13.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"value":0,"_account_id":15699,"name":"sonu","email":"sonu.bhumca11@gmail.com","username":"sonu.kumar"},{"value":0,"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},{"value":0,"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":16282,"name":"YaZug","email":"jschluet@redhat.com","username":"yazug"}],"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":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":2,"date":"2016-12-09 22:32:34.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"value":0,"_account_id":15699,"name":"sonu","email":"sonu.bhumca11@gmail.com","username":"sonu.kumar"},{"value":0,"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},{"value":0,"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},{"value":2,"date":"2016-12-09 21:34:30.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":16282,"name":"YaZug","email":"jschluet@redhat.com","username":"yazug"}],"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":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":1,"date":"2016-12-09 22:32:34.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"value":0,"_account_id":15699,"name":"sonu","email":"sonu.bhumca11@gmail.com","username":"sonu.kumar"},{"value":0,"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},{"value":0,"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":16282,"name":"YaZug","email":"jschluet@redhat.com","username":"yazug"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},{"_account_id":15699,"name":"sonu","email":"sonu.bhumca11@gmail.com","username":"sonu.kumar"},{"_account_id":16282,"name":"YaZug","email":"jschluet@redhat.com","username":"yazug"},{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-09-20 14:05:56.000000000","updated_by":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"reviewer":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"state":"REVIEWER"},{"updated":"2016-09-21 20:13:17.000000000","updated_by":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"reviewer":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"state":"REVIEWER"},{"updated":"2016-09-23 06:06:40.000000000","updated_by":{"_account_id":15699,"name":"sonu","email":"sonu.bhumca11@gmail.com","username":"sonu.kumar"},"reviewer":{"_account_id":15699,"name":"sonu","email":"sonu.bhumca11@gmail.com","username":"sonu.kumar"},"state":"REVIEWER"},{"updated":"2016-10-07 13:56:47.000000000","updated_by":{"_account_id":16282,"name":"YaZug","email":"jschluet@redhat.com","username":"yazug"},"reviewer":{"_account_id":16282,"name":"YaZug","email":"jschluet@redhat.com","username":"yazug"},"state":"REVIEWER"},{"updated":"2016-10-18 00:52:22.000000000","updated_by":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"reviewer":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"state":"REVIEWER"},{"updated":"2016-10-18 02:14:45.000000000","updated_by":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"reviewer":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"state":"REVIEWER"},{"updated":"2016-10-21 13:33:51.000000000","updated_by":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"reviewer":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"state":"REVIEWER"},{"updated":"2016-11-11 03:11:32.000000000","updated_by":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"reviewer":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"state":"REVIEWER"},{"updated":"2016-12-09 21:34:30.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2016-12-09 22:32:34.000000000","updated_by":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"reviewer":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"state":"REVIEWER"},{"updated":"2016-12-09 23:25:13.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"ecd140f0a6a1f097d0ec785e5138aeebde6eedc7","author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"date":"2016-09-19 19:14:43.000000000","message":"Patch Set 1: Cherry Picked from branch master.","accounts_in_message":[],"_revision_number":1},{"id":"aac97d0ad76f3139a0a2a3b5da5b4949a04d3f3c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-19 20:25:27.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/12/372712/1/check/gate-python-openstackclient-docs-ubuntu-xenial/135ab93//doc/build/html/ : SUCCESS in 2m 50s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/12/372712/1/check/gate-python-openstackclient-pep8-ubuntu-xenial/e06562a/ : SUCCESS in 2m 41s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/12/372712/1/check/gate-python-openstackclient-python27-ubuntu-xenial/cb16273/ : SUCCESS in 3m 16s\n- gate-python-openstackclient-python34 http://logs.openstack.org/12/372712/1/check/gate-python-openstackclient-python34/65ece16/ : SUCCESS in 2m 36s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/12/372712/1/check/gate-python-openstackclient-python35-nv/3e3e695/ : SUCCESS in 3m 17s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/12/372712/1/check/gate-tempest-dsvm-neutron-src-python-openstackclient/d9657ad/ : SUCCESS in 1h 07m 36s\n- check-osc-plugins http://logs.openstack.org/12/372712/1/check/check-osc-plugins/23b1dda/ : SUCCESS in 5m 28s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/12/372712/1/check/gate-osc-dsvm-functional/13144c0/ : SUCCESS in 23m 39s","accounts_in_message":[],"_revision_number":1},{"id":"2fee1c0904bc270d376e595b6f38a665487bd476","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-09-19 21:07:38.000000000","message":"Patch Set 1: Code-Review+1\n\njames, is there a reason you cannot use the latest osc release? 3.2.0?\n\nwe typically don\u0027t backport fixes, or release new osc versions for stable branches because we\u0027ve never had a need to. what is the use case you are trying to solve? does tripleO only use the osc version mentioned in upper-constraints?","accounts_in_message":[],"_revision_number":1},{"id":"8e5159e69fa5e2061333099db3402bae4c916498","author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"date":"2016-09-20 12:30:36.000000000","message":"Patch Set 1:\n\n\u003e james, is there a reason you cannot use the latest osc release?\n \u003e 3.2.0?\n \u003e \n \u003e we typically don\u0027t backport fixes, or release new osc versions for\n \u003e stable branches because we\u0027ve never had a need to. what is the use\n \u003e case you are trying to solve? does tripleO only use the osc version\n \u003e mentioned in upper-constraints?\n\nhey steve, I proposed this backport because when TripleO deploys the Newton release, we use python-openstackclient from stable/newton as well.\n\nWe only use the version in u-c when we deploy using master.\n\nPresently, those are equivalent (pinned at 3.2.0) anyway. But even if a new release is done from master that includes this fix and upper-constraints is pinned to that new release, TripleO would not pick it up because we use the latest release/tag on stable/newton.\n\nThis is how RDO is configured to consume python-openstackclient for Newton as well. If that\u0027s not correct, we could investigate changing that.","accounts_in_message":[],"_revision_number":1},{"id":"6faa4c4c753d4ae386cc6307ea88114bdcd18b34","author":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"date":"2016-09-20 14:05:56.000000000","message":"Patch Set 1: Code-Review+2\n\nIf fixes are not backported what\u0027s the use of stable branches?\n\n3.2.0 release is used in RDO as it is on stable/newton branch but this fix is on master only.","accounts_in_message":[],"_revision_number":1},{"id":"2110c0624c59a35325ba3250edbfa5ebf9c3d95d","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2016-09-21 20:03:50.000000000","message":"Patch Set 1:\n\nThe stable branches exist as a requirement of the release process, not because we want them.  We have never had a policy of backporting bug fixes, it has always been critical security fixes only.","accounts_in_message":[],"_revision_number":1},{"id":"9b431c0b088f39862d9536d3da600008b1bcb346","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2016-09-21 20:13:17.000000000","message":"Patch Set 1: Code-Review+1\n\nI\u0027m confused. When I read the doc:\nhttp://docs.openstack.org/project-team-guide/stable-branches.html\n\nit says:\n\"Phase I (first 6 months): All bugfixes (which meet the criteria described below) are appropriate\".\nand \"It’s nevertheless allowed to backport fixes for other bugs if their safety can be easily proved.\"\n\nI think the this patch is a bugfix that meets the criteria that is documented.","accounts_in_message":[],"_revision_number":1},{"id":"fe5fe0b2dccd11a1dc74c248b09ebd11542f8196","author":{"_account_id":15699,"name":"sonu","email":"sonu.bhumca11@gmail.com","username":"sonu.kumar"},"date":"2016-09-23 06:06:40.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"e59917f90a11c994ddf2ab6c0ed4055def4af2a2","author":{"_account_id":16282,"name":"YaZug","email":"jschluet@redhat.com","username":"yazug"},"date":"2016-10-07 13:56:47.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"2fbd002f521b0baf3fd302346d3c9d704b248227","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2016-10-18 00:52:22.000000000","message":"Patch Set 1:\n\nStable branches exist so we can backport and release bug fixes with patch releases. You can\u0027t always jump to the latest version on master and use that on stable because of the creep in dependencies, and we don\u0027t test that way either because of upper-constraints and minimums in g-r on master that differ from stable branches.","accounts_in_message":[],"_revision_number":1},{"id":"134c41a15602e9dc11a48ca4751456074af2e120","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2016-10-18 00:55:56.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"eefb0439a2d2ac2b1a84660bcf25e483bd7588f7","author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"date":"2016-10-18 02:14:45.000000000","message":"Patch Set 1:\n\n\u003e \u003e james, is there a reason you cannot use the latest osc release?\n \u003e \u003e 3.2.0?\n \u003e \u003e\n \u003e \u003e we typically don\u0027t backport fixes, or release new osc versions\n \u003e for\n \u003e \u003e stable branches because we\u0027ve never had a need to. what is the\n \u003e use\n \u003e \u003e case you are trying to solve? does tripleO only use the osc\n \u003e version\n \u003e \u003e mentioned in upper-constraints?\n \u003e \n \u003e hey steve, I proposed this backport because when TripleO deploys\n \u003e the Newton release, we use python-openstackclient from\n \u003e stable/newton as well.\n \u003e \n \u003e We only use the version in u-c when we deploy using master.\n \u003e \n \u003e Presently, those are equivalent (pinned at 3.2.0) anyway. But even\n \u003e if a new release is done from master that includes this fix and\n \u003e upper-constraints is pinned to that new release, TripleO would not\n \u003e pick it up because we use the latest release/tag on stable/newton.\n \u003e \n \u003e This is how RDO is configured to consume python-openstackclient for\n \u003e Newton as well. If that\u0027s not correct, we could investigate\n \u003e changing that.\n\nIdeally TripleO would use the u-c version listed in all stable branches as well as master.  Thats what we test/gate/validate with so you\u0027d want to have a really good reason to do something different.\n\nHaving said that As you point out TripleO would still like to see this land on the newton branch","accounts_in_message":[],"_revision_number":1},{"id":"675d05a4b29316d224cec0331e98cae57fe11701","author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"date":"2016-10-18 02:18:35.000000000","message":"Patch Set 1:\n\n\u003e The stable branches exist as a requirement of the release process,\n \u003e not because we want them.  We have never had a policy of\n \u003e backporting bug fixes, it has always been critical security fixes\n \u003e only.\n\nIt\u0027s be great to reconsider that view  As OpenStackClient is used my more projects they\u0027re going to hit bugs that affect the stable branches and as Matt points out right now the way we handle that is by releasing from those stable branches as consumers can\u0027t always install from master.","accounts_in_message":[],"_revision_number":1},{"id":"317f13ffb5bcd7fd673fefa756eca78351378cff","author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"date":"2016-10-18 02:37:10.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"65f895cfbf4df1b5a744ee3d6ed506820f7dde9c","author":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"date":"2016-10-21 13:33:51.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"ac466104a20192ade58a38d33f202fbdd368174e","author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"date":"2016-11-11 03:11:32.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"67463280c76cca32f38d1847c6acbfaa14959094","author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"date":"2016-11-16 21:34:36.000000000","message":"Patch Set 1:\n\ni don\u0027t know how to move this forward in a meaningful way. upper-constraints.txt from the stable/newton branch of the requirements repo is 3.2.0, so we still do not have this fix in openstackclient for newton.\n\nis updating upper-constraints on stable/newton to a later release from master a path forward?","accounts_in_message":[],"_revision_number":1},{"id":"f3e434e0f366704aa9c62f1023dc36296269b91d","author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"date":"2016-11-28 05:01:30.000000000","message":"Patch Set 1:\n\n\u003e i don\u0027t know how to move this forward in a meaningful way.\n \u003e upper-constraints.txt from the stable/newton branch of the\n \u003e requirements repo is 3.2.0, so we still do not have this fix in\n \u003e openstackclient for newton.\n \u003e \n \u003e is updating upper-constraints on stable/newton to a later release\n \u003e from master a path forward?\n\nYou can\u0027t use a release from master in the newton branch.  A quick check shows the osc master[1] and newton[2] have differing requirements for oslo.utils and the constrained versions differ (see below) so installing osc from master in newton will result in resource conflicts\n\n Constraints\n 1.0.0-58-g2919572     : oslo.utils\u003d\u003d\u003d3.18.0\n origin/master         : oslo.utils\u003d\u003d\u003d3.18.0\n origin/stable/newton  : oslo.utils\u003d\u003d\u003d3.16.0\n origin/stable/mitaka  : oslo.utils\u003d\u003d\u003d3.8.0\n origin/stable/liberty : oslo.utils\u003d\u003d\u003d2.5.0\n\nThere are 2 things I\u0027d like to understand:\n1) why tripleo tracks branches they way you describe instead of just using the branch specific upper-constraints.txt\n2) The openstackclient\u0027s view on stable branches.\n\nI know it\u0027s at a slightly awkward time but It\u0027s be great to discuss this either in the requirements meeting[3] or at another mutually agreeable time/place\n\nIdeally there\u0027d be a osc 3.2.1 release including this change, but that isn\u0027t my call to make.\n\n[1] http://git.openstack.org/cgit/openstack/python-openstackclient/tree/requirements.txt#n13\n[2] http://git.openstack.org/cgit/openstack/python-openstackclient/tree/requirements.txt?h\u003dstable/newton#n13\n[3] http://eavesdrop.openstack.org/#Requirements_Team_Meeting","accounts_in_message":[],"_revision_number":1},{"id":"f4214e75032e93547f12ca589dca57dbffea2c78","author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"date":"2016-11-28 11:57:28.000000000","message":"Patch Set 1:\n\n\u003e \u003e i don\u0027t know how to move this forward in a meaningful way.\n \u003e \u003e upper-constraints.txt from the stable/newton branch of the\n \u003e \u003e requirements repo is 3.2.0, so we still do not have this fix in\n \u003e \u003e openstackclient for newton.\n \u003e \u003e\n \u003e \u003e is updating upper-constraints on stable/newton to a later release\n \u003e \u003e from master a path forward?\n \u003e \n \u003e You can\u0027t use a release from master in the newton branch.\n\nTripleO (and myself) aren\u0027t the ones proposing we do that. python-openstackclient is saying they don\u0027t want to do this backport and release from newton. Yet, this bug is pretty much a blocker for using openstackclient from newton.\n\n \u003e There are 2 things I\u0027d like to understand:\n \u003e 1) why tripleo tracks branches they way you describe instead of\n \u003e just using the branch specific upper-constraints.txt\n\nAgain, that is what TripleO would prefer to do. It\u0027s the whole reason I\u0027m trying to backport this patch. However, that version in upper-constraints from stable/newton is 3.2.0, which does not contain this patch I\u0027m trying to backport (obviously, since this proposed backport has not merged).\n\nSo, we can\u0027t use it.\n\nThe openstackclient team recommended we use the latest osc release (see the reviews on this patch).","accounts_in_message":[],"_revision_number":1},{"id":"c1bfb3dbef8983d52371024cc2be2ac6afc3ae98","author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"date":"2016-12-09 13:32:08.000000000","message":"Abandoned\n\nwent in a different direction","accounts_in_message":[],"_revision_number":1},{"id":"024052da7304800522d6ce9b41c1180d3685dfb7","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2016-12-09 16:38:21.000000000","message":"Patch Set 1:\n\nJames, ironically enough it was probably this abandonment that prompted folk to push me over the line to merging these backports.  I still hate it, and will mutter under my breath, but if you would like to un-abandon this we\u0027ll get it in.","accounts_in_message":[],"_revision_number":1},{"id":"23791331f9a7d65f859bba6d617d69f32ed86c3d","author":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"date":"2016-12-09 18:04:05.000000000","message":"Restored\n\nRestoring to get usable stable/newton release of openstackclient","accounts_in_message":[],"_revision_number":1},{"id":"c7d9ea4253bde399886655e4bcaf5204ec5153c2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-09 18:04:22.000000000","message":"Patch Set 1: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"10c68056941f13ef8ace804638ccd9108853a4b9","author":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"date":"2016-12-09 18:09:11.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"9709ce87200a05e3af4b190116be1c53e89304db","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-09 19:27:01.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/12/372712/2/check/gate-python-openstackclient-docs-ubuntu-xenial/0232217//doc/build/html/ : SUCCESS in 2m 39s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/12/372712/2/check/gate-python-openstackclient-pep8-ubuntu-xenial/7070a64/ : SUCCESS in 5m 10s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/12/372712/2/check/gate-python-openstackclient-python27-ubuntu-xenial/cc92c80/ : SUCCESS in 3m 51s\n- gate-python-openstackclient-python34 http://logs.openstack.org/12/372712/2/check/gate-python-openstackclient-python34/5a73584/ : SUCCESS in 2m 23s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/12/372712/2/check/gate-python-openstackclient-python35-nv/789e131/ : SUCCESS in 3m 31s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-openstackclient-ubuntu-xenial http://logs.openstack.org/12/372712/2/check/gate-tempest-dsvm-neutron-src-python-openstackclient-ubuntu-xenial/677566a/ : SUCCESS in 1h 16m 53s\n- check-osc-plugins http://logs.openstack.org/12/372712/2/check/check-osc-plugins/3f4d820/ : FAILURE in 3m 44s (non-voting)\n- gate-osc-dsvm-functional-ubuntu-xenial http://logs.openstack.org/12/372712/2/check/gate-osc-dsvm-functional-ubuntu-xenial/2b39dca/ : SUCCESS in 14m 45s\n- gate-tripleo-ci-centos-7-nonha-multinode-nv http://logs.openstack.org/12/372712/2/check/gate-tripleo-ci-centos-7-nonha-multinode-nv/993cae1/ : FAILURE in 7m 43s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"bfd62c4365cc69a44f55f9b1c4debf21a4d955cc","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2016-12-09 21:34:30.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"655a6abf078066f41a721d18cbe2ea161f03f291","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-12-09 22:32:34.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"b9237cdb540b03e67cdc3dc17fb713d5411f809c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-09 22:32:44.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"b8a6dda2bda178c3aafccb840bdc516dc8378c99","date":"2016-12-09 23:25:13.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2},{"id":"9f7a574d736ecf92737887be8b457adfa9c83e59","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-09 23:25:13.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/12/372712/2/gate/gate-python-openstackclient-docs-ubuntu-xenial/0ac444d//doc/build/html/ : SUCCESS in 2m 35s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/12/372712/2/gate/gate-python-openstackclient-pep8-ubuntu-xenial/6fb837a/ : SUCCESS in 3m 01s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/12/372712/2/gate/gate-python-openstackclient-python27-ubuntu-xenial/c6d27eb/ : SUCCESS in 2m 20s\n- gate-python-openstackclient-python34 http://logs.openstack.org/12/372712/2/gate/gate-python-openstackclient-python34/ee21a61/ : SUCCESS in 2m 21s\n- gate-tempest-dsvm-neutron-src-python-openstackclient-ubuntu-xenial http://logs.openstack.org/12/372712/2/gate/gate-tempest-dsvm-neutron-src-python-openstackclient-ubuntu-xenial/e2c4b49/ : SUCCESS in 51m 54s\n- gate-osc-dsvm-functional-ubuntu-xenial http://logs.openstack.org/12/372712/2/gate/gate-osc-dsvm-functional-ubuntu-xenial/c43fbcc/ : SUCCESS in 23m 16s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"9484cd3850594ef6cb62d3c57dc5ec402305975c","revisions":{"a2cd3a7e29955bac69cf4492a3d159bff8db598d":{"kind":"REWORK","_number":1,"created":"2016-09-19 19:14:43.000000000","uploader":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"ref":"refs/changes/12/372712/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/12/372712/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/12/372712/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/12/372712/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/12/372712/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/12/372712/1"}}},"commit":{"parents":[{"commit":"b3ce8364f54c0e229d04ecc6e21b4d7819df736e","subject":"Update .gitreview for stable/newton","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/b3ce8364f54c0e229d04ecc6e21b4d7819df736e"}]}],"author":{"name":"Dean Troyer","email":"dtroyer@gmail.com","date":"2016-09-01 15:54:29.000000000","tz":-300},"committer":{"name":"James Slagle","email":"jslagle@redhat.com","date":"2016-09-19 19:14:43.000000000","tz":0},"subject":"Defer auth prompting until it is actually needed","message":"Defer auth prompting until it is actually needed\n\nAuth option prompting happens waaaay to early in the default\nos-client-config flow, we need to defer it until adter the commands\nhave been parsed.  This is why ClientManager.setup_auth() exists,\nas it is not called until the first attempt to connect to a server\noccurs.  Commands that do not require authentication never hit this.\n\nAlso, required options were not being enforced.  By doing this we handle\nwhen no authentication info is present, we fail on missing auth-url rather\nthan attempt to prompt for a password (default auth is password).\n\nCloses-Bug: 1619274\nChange-Id: Ia4eae350e6904c9eb2c8507d9b3429fe52418726\n(cherry picked from commit 14dbfe44741b65c9e0514a34669f52de8629b1c0)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/a2cd3a7e29955bac69cf4492a3d159bff8db598d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/a2cd3a7e29955bac69cf4492a3d159bff8db598d"}]},"branch":"refs/heads/stable/newton"},"9484cd3850594ef6cb62d3c57dc5ec402305975c":{"kind":"REWORK","_number":2,"created":"2016-12-09 18:09:11.000000000","uploader":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"ref":"refs/changes/12/372712/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/12/372712/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/12/372712/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/12/372712/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/12/372712/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/12/372712/2"}}},"commit":{"parents":[{"commit":"23f6681de615973b7cf54036b607d9a80da6d929","subject":"Merge \"Use project_domain_id only in password auth\" into stable/newton","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/23f6681de615973b7cf54036b607d9a80da6d929"}]}],"author":{"name":"Dean Troyer","email":"dtroyer@gmail.com","date":"2016-09-01 15:54:29.000000000","tz":-300},"committer":{"name":"Alan Pevec","email":"alan.pevec@redhat.com","date":"2016-12-09 18:07:52.000000000","tz":60},"subject":"Defer auth prompting until it is actually needed","message":"Defer auth prompting until it is actually needed\n\nAuth option prompting happens waaaay to early in the default\nos-client-config flow, we need to defer it until adter the commands\nhave been parsed.  This is why ClientManager.setup_auth() exists,\nas it is not called until the first attempt to connect to a server\noccurs.  Commands that do not require authentication never hit this.\n\nAlso, required options were not being enforced.  By doing this we handle\nwhen no authentication info is present, we fail on missing auth-url rather\nthan attempt to prompt for a password (default auth is password).\n\nCloses-Bug: 1619274\nChange-Id: Ia4eae350e6904c9eb2c8507d9b3429fe52418726\n(cherry picked from commit 14dbfe44741b65c9e0514a34669f52de8629b1c0)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/9484cd3850594ef6cb62d3c57dc5ec402305975c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/9484cd3850594ef6cb62d3c57dc5ec402305975c"}]},"branch":"refs/heads/stable/newton"}},"requirements":[],"submit_records":[],"submit_requirements":[]}