)]}'
{"id":"openstack%2Fpuppet-openstacklib~207873","triplet_id":"openstack%2Fpuppet-openstacklib~master~I63278fbb34d4a2e37861d0ec411226feb27ef9dc","project":"openstack/puppet-openstacklib","branch":"master","topic":"bug/1479879","hashtags":[],"change_id":"I63278fbb34d4a2e37861d0ec411226feb27ef9dc","subject":"Fix OS_AUTH_URL to adapt to v3/v2.0 split","status":"ABANDONED","created":"2015-07-31 12:49:35.000000000","updated":"2015-08-20 10:00:14.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"e59ac8dfb03c7cb151f273dae252ff3add2342df","_number":207873,"virtual_id_number":207873,"owner":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},{"value":1,"date":"2015-08-12 16:42:42.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"date":"2015-08-13 11:10:39.000000000","_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},{"_account_id":9387,"name":"Aleksandr Didenko","email":"adidenko@mirantis.com","username":"AD"},{"_account_id":13948,"name":"Bartłomiej Piotrowski","email":"bpiotrowski@mirantis.com","username":"bpiotrowski"},{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"all":[{"value":1,"date":"2015-08-13 14:12:43.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2015-08-12 17:32:09.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9387,"name":"Aleksandr Didenko","email":"adidenko@mirantis.com","username":"AD"},{"value":1,"date":"2015-08-12 18:29:42.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":13948,"name":"Bartłomiej Piotrowski","email":"bpiotrowski@mirantis.com","username":"bpiotrowski"},{"value":-1,"date":"2015-08-18 01:53:04.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},{"_account_id":9387,"name":"Aleksandr Didenko","email":"adidenko@mirantis.com","username":"AD"},{"_account_id":13948,"name":"Bartłomiej Piotrowski","email":"bpiotrowski@mirantis.com","username":"bpiotrowski"},{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"}],"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":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},{"_account_id":9387,"name":"Aleksandr Didenko","email":"adidenko@mirantis.com","username":"AD"},{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},{"_account_id":13948,"name":"Bartłomiej Piotrowski","email":"bpiotrowski@mirantis.com","username":"bpiotrowski"},{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-07-31 14:07:29.000000000","updated_by":{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},"reviewer":{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},"state":"REVIEWER"},{"updated":"2015-07-31 14:07:34.000000000","updated_by":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"reviewer":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"state":"REVIEWER"},{"updated":"2015-07-31 14:07:37.000000000","updated_by":{"_account_id":9387,"name":"Aleksandr Didenko","email":"adidenko@mirantis.com","username":"AD"},"reviewer":{"_account_id":9387,"name":"Aleksandr Didenko","email":"adidenko@mirantis.com","username":"AD"},"state":"REVIEWER"},{"updated":"2015-07-31 14:24:37.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":"2015-08-10 13:42:22.000000000","updated_by":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"reviewer":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"state":"REVIEWER"},{"updated":"2015-08-11 16:39:53.000000000","updated_by":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"reviewer":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"state":"REVIEWER"},{"updated":"2015-08-12 16:42:42.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2015-08-12 17:32:09.000000000","updated_by":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"reviewer":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"state":"REVIEWER"},{"updated":"2015-08-12 18:29:42.000000000","updated_by":{"_account_id":13948,"name":"Bartłomiej Piotrowski","email":"bpiotrowski@mirantis.com","username":"bpiotrowski"},"reviewer":{"_account_id":13948,"name":"Bartłomiej Piotrowski","email":"bpiotrowski@mirantis.com","username":"bpiotrowski"},"state":"REVIEWER"},{"updated":"2015-08-13 14:12:43.000000000","updated_by":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"reviewer":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"state":"REVIEWER"},{"updated":"2015-08-18 01:53:04.000000000","updated_by":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"reviewer":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"state":"REVIEWER"}],"messages":[{"id":"11f2ee293f8bc0ce50474222ce771c8c5cf8397d","author":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"date":"2015-07-31 12:49:35.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"64764f3b7bd6f16187db4492be3ed640074fafd6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-31 13:55:27.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-puppet-openstacklib-puppet-lint http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-lint/93d93f0/ : SUCCESS in 2m 38s\n- gate-puppet-openstacklib-puppet-syntax-3 http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-syntax-3/430d14e/ : SUCCESS in 2m 31s\n- gate-puppet-openstacklib-puppet-syntax-4 http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-syntax-4/38670fa/ : SUCCESS in 3m 27s\n- gate-puppet-openstacklib-puppet-unit-3.3 http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-unit-3.3/95300eb/ : SUCCESS in 4m 19s\n- gate-puppet-openstacklib-puppet-unit-3.4 http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-unit-3.4/f513ba5/ : SUCCESS in 3m 29s\n- gate-puppet-openstacklib-puppet-unit-3.6 http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-unit-3.6/50ef98c/ : SUCCESS in 3m 25s\n- gate-puppet-openstacklib-puppet-unit-3.7 http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-unit-3.7/8752b59/ : SUCCESS in 3m 20s\n- gate-puppet-openstacklib-puppet-unit-3.8 http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-unit-3.8/e296cbc/ : SUCCESS in 5m 50s\n- gate-puppet-openstacklib-puppet-unit-4.0 http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-unit-4.0/9575c90/ : SUCCESS in 5m 22s\n- gate-puppet-openstacklib-puppet-unit-latest http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-unit-latest/04aef4e/ : SUCCESS in 5m 28s (non-voting)\n- gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-trusty http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-trusty/d6d176e/ : SUCCESS in 5m 37s\n- gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-centos7 http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-centos7/49de476/ : SUCCESS in 6m 55s\n- gate-puppet-openstacklib-puppet-beaker-rspec-upgrade-dsvm-trusty http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-beaker-rspec-upgrade-dsvm-trusty/fbe2665/ : SUCCESS in 9m 40s (non-voting)\n- gate-puppet-openstacklib-puppet-beaker-rspec-upgrade-dsvm-centos7 http://logs.openstack.org/73/207873/1/check/gate-puppet-openstacklib-puppet-beaker-rspec-upgrade-dsvm-centos7/7a6771d/ : SUCCESS in 8m 56s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"ab38d8aea3593a9317d606e2178718143408cc7e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-31 14:04:13.000000000","message":"Patch Set 1:\n\nBuild succeeded (check-tripleo pipeline).\n\n- gate-tripleo-ironic-overcloud-f21puppet-nonha http://logs.openstack.org/73/207873/1/check-tripleo/gate-tripleo-ironic-overcloud-f21puppet-nonha/26ea569/ : SUCCESS in 1h 12m 49s","accounts_in_message":[],"_revision_number":1},{"id":"5637f1a314b63ee71f72280e0b55456f2c73744f","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2015-07-31 14:24:37.000000000","message":"Patch Set 1: Code-Review-1\n\nSince we don\u0027t have integration testing in place (yet, we are currently working on this).\n\nI would like you to use Depends-On in the Keystone patch so we can actually verify it does not break puppet-keystone which is the highest consumer of this code now.","accounts_in_message":[],"_revision_number":1},{"id":"1a14220288218e897efa529c338d17b96e173d8b","author":{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},"date":"2015-07-31 20:19:11.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"9c9ff35ab6c6232cea25450054a89abb24e1c405","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2015-08-04 15:19:01.000000000","message":"Patch Set 1:\n\nthis is failing on https://review.openstack.org/#/c/207890/\ndoes that mean this patch does not fix the bug?","accounts_in_message":[],"_revision_number":1},{"id":"71da00d2518f96ac511df3141d7dfa7efd9b3f56","author":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"date":"2015-08-06 05:16:29.000000000","message":"Patch Set 1: Code-Review-1\n\nThe Authentication URL doesn\u0027t need the version anmore.\n\nOnly the Service URL needs to have version 2.0 explicitly set (automatically replaced in v3)\n-------------------------------------------\n# openstack catalog show keystone -f value\nRegionOne\n  publicURL: http://127.0.0.1:5000\n  internalURL: http://192.168.65.11:5000\n  adminURL: http://127.0.0.1:35357/v2.0\n\nkeystone\nidentity\n-------------------------------------------\n\nTherefore there is no need for this patch.","accounts_in_message":[],"_revision_number":1},{"id":"c490a221f67d512fda90805ebbbc72725ee9f495","author":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"date":"2015-08-06 07:48:23.000000000","message":"Patch Set 1:\n\nGilles, I\u0027ll check and confirm that you are right. However, if AUTH_URL in openrc contains v2.0, the v3.0 commands are broken. Maybe we can at least strip the version off the URL still.","accounts_in_message":[],"_revision_number":1},{"id":"05aa2c10efab57affe25e223ee58c4c5132d9a21","author":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"date":"2015-08-07 02:35:39.000000000","message":"Patch Set 1:\n\nHi Matthew,\n\nI\u0027ve successfully tested os-auth-url without the version (v2.0 and v3)\n\nI\u0027ve also successfully tested against v3 the os-url, with v2.0, keystone hacks-out the version by replacing it with v3.\n\nThe long term goal is for Keystone to make the version completely transparent. Therefore that means removing the version number from openrc where needed.","accounts_in_message":[],"_revision_number":1},{"id":"2679615e237285bf268af366af30da8d413e0683","author":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"date":"2015-08-07 09:54:53.000000000","message":"Patch Set 1:\n\nI\u0027m afraid that it\u0027s not as transparent as I wish:\nhttp://paste.openstack.org/show/412156/\n\nIt seems that openstackclient fails to access v3 keystone API unless it is explicitly in the URL. Back to the drawing board. We need to get this right so that it behaves normally when using local connection and service token vs remote connections with openrc","accounts_in_message":[],"_revision_number":1},{"id":"b9ecbf7347aca99673ff1ff1626ce8ee9d5129c4","author":{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},"date":"2015-08-07 21:08:07.000000000","message":"Patch Set 1:\n\nI think the problem with puppet-keystone is that it uses token based auth with OS_URL (not OS_AUTH_URL) and OS_TOKEN (not username/password/etc.) instead of username/password based auth.  In this case, the `openstack` cli can\u0027t automatically figure out which API URL to use, so it falls back on /v2.0, even if you specify OS_IDENTITY_API_VERSION\u003d3.  This is why puppet-keystone (and probably any other puppet code or service that uses token based auth) must provide the correct \"/vN\" version suffix for the auth url.\n\nYou can use `openstack --debug user list ...`, or just go to the keystone logs, to see which URL it is trying to use - /v2.0/tokens, or /v3/auth/tokens.\n\nSo I think we do need a way for puppet-keystone (or other code that uses token based auth) to override whatever url is set in the environment, and force the use of \"/v3\".","accounts_in_message":[],"_revision_number":1},{"id":"44fb0fbfb58d62b95a1cb64ed1df372555be9459","author":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"date":"2015-08-10 10:15:29.000000000","message":"Patch Set 1:\n\nRichard, you\u0027re objecting to this patch because you believe openstackclient and python-keystone are capable of creating users/roles/endpoints using openrc alone as-is. Can you show me a configuration that works? This patch 1) solves a bug affecting my deployment configuration and 2) doesn\u0027t break anything. Maybe you can show me an example openrc that _does_ work. OS_URL is intended to work with OS_TOKEN, while OS_AUTH_URL works with credentials. They should both work.","accounts_in_message":[],"_revision_number":1},{"id":"d3a78f9122674e79bccc1e2040850c56d6663c14","author":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"date":"2015-08-10 10:53:13.000000000","message":"Patch Set 1:\n\n@Richard: example URLs for openstackclient v3 commands: http://paste.openstack.org/show/h8Ag4ZRuDeCjYumhC54E/","accounts_in_message":[],"_revision_number":1},{"id":"02bd5fd9d95cd5b354524b64141ccb3570b47bc2","author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"date":"2015-08-10 11:14:10.000000000","message":"Patch Set 1: Code-Review+1\n\nI have the same result as Matthew: http://paste.openstack.org/show/412345/\n\nThe same issue and for using v2 version via cli: http://paste.openstack.org/show/412346/\n\nI think, that api version shouldn\u0027t be set to v2. It should be replaced on proper version (via cli, if we have not passed version via cli, then should be default v2): http://paste.openstack.org/show/412347/\n\nActually, it seems like a bug not in puppet (client or keystone), but for me this patch is good.","accounts_in_message":[],"_revision_number":1},{"id":"bdf6f741be77d8f0c3a6ceff94d8ad8de37fe6b4","author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"date":"2015-08-10 11:47:30.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"481f8810ac290df7a1c269d0e825e2a3bd62cfaf","author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"date":"2015-08-10 13:42:22.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"b29e8d9ca66b34c49c2dd65fb3791ea90c48c8fa","author":{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},"date":"2015-08-10 14:49:19.000000000","message":"Patch Set 1: -Code-Review\n\n\u003e Richard, you\u0027re objecting to this patch because you believe openstackclient and python-keystone are capable of creating users/roles/endpoints using openrc alone as-is.\n\nNo.  I am not objecting to this patch.  I\u0027m just attempting to explain the behavior of openstackclient.  You\u0027ve already proven that having \"/v2.0\" in the URL from openrc will cause problems with v3 commands.\n\n\u003e Can you show me a configuration that works? This patch 1) solves a bug affecting my deployment configuration and 2) doesn\u0027t break anything. Maybe you can show me an example openrc that _does_ work. OS_URL is intended to work with OS_TOKEN, while OS_AUTH_URL works with credentials. They should both work.\n\nThe only configuration that works is having OS_URL or OS_AUTH_URL with \"/v3\" in openrc.","accounts_in_message":[],"_revision_number":1},{"id":"8857495148d773a5832f4a1377184d6287467e10","author":{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},"date":"2015-08-10 14:49:51.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"746abad805d3323197cd2f14a91af1cbcb85b049","author":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"date":"2015-08-12 16:19:42.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"5a0c27acbafb46abe2e543bf554bf29bedbeb4b6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-12 16:42:42.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-puppet-openstacklib-puppet-lint http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-lint/db76ca4/ : SUCCESS in 2m 32s\n- gate-puppet-openstacklib-puppet-syntax-3-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-syntax-3-dsvm-centos7/4a7efc6/ : SUCCESS in 2m 37s\n- gate-puppet-openstacklib-puppet-syntax-4-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-syntax-4-dsvm-centos7/0150d28/ : SUCCESS in 2m 51s\n- gate-puppet-openstacklib-puppet-unit-3.3-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-unit-3.3-dsvm-centos7/b2daa16/ : SUCCESS in 2m 59s\n- gate-puppet-openstacklib-puppet-unit-3.4-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-unit-3.4-dsvm-centos7/08e7c13/ : SUCCESS in 3m 39s\n- gate-puppet-openstacklib-puppet-unit-3.6-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-unit-3.6-dsvm-centos7/39281e7/ : SUCCESS in 3m 15s\n- gate-puppet-openstacklib-puppet-unit-3.7-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-unit-3.7-dsvm-centos7/97bb218/ : SUCCESS in 3m 20s\n- gate-puppet-openstacklib-puppet-unit-3.8-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-unit-3.8-dsvm-centos7/b294ed0/ : SUCCESS in 2m 55s\n- gate-puppet-openstacklib-puppet-unit-4.0-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-unit-4.0-dsvm-centos7/1a77f6d/ : SUCCESS in 2m 48s\n- gate-puppet-openstacklib-puppet-unit-latest-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-unit-latest-dsvm-centos7/f46f4c0/ : SUCCESS in 2m 55s\n- gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-trusty http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-trusty/db785e9/ : SUCCESS in 5m 43s\n- gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-beaker-rspec-dsvm-centos7/5452026/ : SUCCESS in 5m 28s\n- gate-puppet-openstacklib-puppet-beaker-rspec-upgrade-dsvm-trusty http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-beaker-rspec-upgrade-dsvm-trusty/2a546d1/ : FAILURE in 9m 49s (non-voting)\n- gate-puppet-openstacklib-puppet-beaker-rspec-upgrade-dsvm-centos7 http://logs.openstack.org/73/207873/2/check/gate-puppet-openstacklib-puppet-beaker-rspec-upgrade-dsvm-centos7/ca3242f/ : FAILURE in 8m 54s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"f8e90be4eb5f787d1b380b9e65b4d49d7df2ebff","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2015-08-12 17:32:09.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"4a2d413b6ede6f80085451a2af24194cc6880513","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-12 17:54:10.000000000","message":"Patch Set 2:\n\nBuild succeeded (check-tripleo pipeline).\n\n- gate-tripleo-ironic-overcloud-f21puppet-nonha http://logs.openstack.org/73/207873/2/check-tripleo/gate-tripleo-ironic-overcloud-f21puppet-nonha/17e14c4/ : SUCCESS in 1h 13m 36s","accounts_in_message":[],"_revision_number":2},{"id":"b55ca14c164f40c6bc5b86abbc77088f7bf85380","author":{"_account_id":13948,"name":"Bartłomiej Piotrowski","email":"bpiotrowski@mirantis.com","username":"bpiotrowski"},"date":"2015-08-12 18:29:42.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"70cfd2d0f2585ec3d2a7829e3335e09d72510417","author":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"date":"2015-08-13 05:29:24.000000000","message":"Patch Set 2: Code-Review-1\n\nHave you guys verified the identity endpoint?\n\nThis is working fine when there is no version specified in the URL suffix:\nhttp://paste.openstack.org/raw/412819/\n\nPlease make sure the endpoint is set up accordingly and display the catalog in any example you might provide.","accounts_in_message":[],"_revision_number":2},{"id":"9bb034f9007b413b36e86bc05ff7fca5c0f8fc57","author":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"date":"2015-08-13 11:10:39.000000000","message":"Patch Set 2:\n\nGilles, how are you creating these endpoints? If you use puppet-keystone, the API version is appended automatically.\n\nhttps://github.com/openstack/puppet-keystone/blob/master/manifests/endpoint.pp#L58-L64","accounts_in_message":[],"_revision_number":2},{"id":"8f9e2a299c2a226fe137c4ade2faad68e7b0bd81","author":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"date":"2015-08-13 11:25:01.000000000","message":"Patch Set 2: -Code-Review\n\nRight, we need to fix the public URL in the endpoint.pp to not contain the version!\n\n(I had my test setup with keystone_puppet but I played with the endpoint for the sake of this discussion!)","accounts_in_message":[],"_revision_number":2},{"id":"be81476d14f845feebdeede61c59f7fd41b471b8","author":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"date":"2015-08-13 13:18:53.000000000","message":"Patch Set 2:\n\nMaybe this https://review.openstack.org/212523/","accounts_in_message":[],"_revision_number":2},{"id":"6f8d5dd3dc91fda6c014702202d344e5016aad40","author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"date":"2015-08-13 14:12:43.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"5469238762882c57a760402acb9ba23888170cd7","author":{"_account_id":6525,"name":"Gilles Dubreuil","email":"gilles@redhat.com","username":"q-1illes-a"},"date":"2015-08-18 01:53:04.000000000","message":"Patch Set 2: Code-Review-1\n\nWith Liberty and python-opentstackclient (OSC) version being bumped up to 1.6.1+, the endpoints are now \u0027APIs version-less\u0027\n\nThis should be taken care with https://review.openstack.org/213957.\n\nMeanwhile that is not supported on Kilo (OSC version 1.0.4) nevertheless the two patches 213957 depends on can be backported.\n\nPlease abandon this one and lets take if from after the two backports are merged.\n\nPlease see also discussion on the following thread http://lists.openstack.org/pipermail/openstack-dev/2015-July/071004.html","accounts_in_message":[],"_revision_number":2},{"id":"8614b31f37be17b468a807ff19ccf2641ff618ae","author":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"date":"2015-08-20 10:00:14.000000000","message":"Abandoned\n\nAbandoning in lieu of Gilles\u0027s https://review.openstack.org/213957.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"f8ae6d950c0a4234f299152c89f8d14e94864c71","revisions":{"3b404c132b117dfcce0d6c01b24a97a48f99685c":{"kind":"REWORK","_number":1,"created":"2015-07-31 12:49:35.000000000","uploader":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"ref":"refs/changes/73/207873/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-openstacklib","ref":"refs/changes/73/207873/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/73/207873/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/73/207873/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/73/207873/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-openstacklib refs/changes/73/207873/1"}}},"commit":{"parents":[{"commit":"f754ef3bcf05177e50375c6464231214f66f83f8","subject":"Add a proxy inifile provider","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/f754ef3bcf05177e50375c6464231214f66f83f8"}]}],"author":{"name":"Matthew Mosesohn","email":"mmosesohn@mirantis.com","date":"2015-07-31 12:46:37.000000000","tz":180},"committer":{"name":"Matthew Mosesohn","email":"mmosesohn@mirantis.com","date":"2015-07-31 12:49:31.000000000","tz":180},"subject":"Fix OS_AUTH_URL to adapt to v3/v2.0 split","message":"Fix OS_AUTH_URL to adapt to v3/v2.0 split\n\nWithout this patch, openrc and env vars get\npassed directly through as-is without being\nmodified, but OS_API_IDENTITY_VERSION was,\ncausing breakage.\n\nChange-Id: I63278fbb34d4a2e37861d0ec411226feb27ef9dc\nRelated-Bug: #1479879\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/3b404c132b117dfcce0d6c01b24a97a48f99685c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/3b404c132b117dfcce0d6c01b24a97a48f99685c"}]},"branch":"refs/heads/master"},"f8ae6d950c0a4234f299152c89f8d14e94864c71":{"kind":"REWORK","_number":2,"created":"2015-08-12 16:19:42.000000000","uploader":{"_account_id":7195,"name":"Matthew Mosesohn","email":"raytrac3r@gmail.com","username":"mmosesohn"},"ref":"refs/changes/73/207873/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-openstacklib","ref":"refs/changes/73/207873/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/73/207873/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/73/207873/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/73/207873/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-openstacklib refs/changes/73/207873/2"}}},"commit":{"parents":[{"commit":"f754ef3bcf05177e50375c6464231214f66f83f8","subject":"Add a proxy inifile provider","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/f754ef3bcf05177e50375c6464231214f66f83f8"}]}],"author":{"name":"Matthew Mosesohn","email":"mmosesohn@mirantis.com","date":"2015-07-31 12:46:37.000000000","tz":180},"committer":{"name":"Matthew Mosesohn","email":"mmosesohn@mirantis.com","date":"2015-08-12 16:19:25.000000000","tz":180},"subject":"Fix OS_AUTH_URL to adapt to v3/v2.0 split","message":"Fix OS_AUTH_URL to adapt to v3/v2.0 split\n\nWithout this patch, openrc and env vars get\npassed directly through as-is without being\nmodified, but OS_API_IDENTITY_VERSION was,\ncausing breakage.\n\nChange-Id: I63278fbb34d4a2e37861d0ec411226feb27ef9dc\nRelated-Bug: #1479879\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/f8ae6d950c0a4234f299152c89f8d14e94864c71"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/f8ae6d950c0a4234f299152c89f8d14e94864c71"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}