)]}'
{"id":"openstack%2Fpuppet-openstack_extras~292649","triplet_id":"openstack%2Fpuppet-openstack_extras~stable%2Fliberty~I625adb5d8ef1898bd9a44a76a09b3f985146c96c","project":"openstack/puppet-openstack_extras","branch":"stable/liberty","topic":"add_identity_api_version_liberty","hashtags":[],"change_id":"I625adb5d8ef1898bd9a44a76a09b3f985146c96c","subject":"authfile: fix OS_IDENTITY_API_VERSION","status":"ABANDONED","created":"2016-03-14 23:54:44.000000000","updated":"2016-03-17 00:33:30.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d0febd32e5f572007a0b840daf48c4e7893346ae","_number":292649,"virtual_id_number":292649,"owner":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"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"},{"date":"2016-03-17 00:03:36.000000000","_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},{"value":1,"date":"2016-03-15 00:26:37.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2016-03-15 09:21:09.000000000","_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"}],"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":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"all":[{"value":-1,"date":"2016-03-15 09:18:13.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":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"}],"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"},{"date":"2016-03-14 23:54:44.000000000","_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":0},"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"}],"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":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-03-14 23:54:44.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-03-15 00:26:37.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-03-15 09:18:13.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":"2016-03-17 00:03:36.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"}],"messages":[{"id":"034c9889286840b1ecba298533fe2c34fece0b9c","author":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"date":"2016-03-14 23:54:44.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e5b84c582c7ba5935f1ab1c5660cac255cc7ca43","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-03-15 00:26:37.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-puppet-openstack_extras-puppet-lint http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-lint/e65c57e/ : SUCCESS in 2m 56s\n- gate-puppet-openstack_extras-puppet-syntax-3-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-syntax-3-dsvm-centos7/ed20856/ : SUCCESS in 3m 09s\n- gate-puppet-openstack_extras-puppet-syntax-4-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-syntax-4-dsvm-centos7/65216d9/ : SUCCESS in 3m 08s\n- gate-puppet-openstack_extras-puppet-unit-3.3-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-unit-3.3-dsvm-centos7/9050335/ : SUCCESS in 4m 37s\n- gate-puppet-openstack_extras-puppet-unit-3.4-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-unit-3.4-dsvm-centos7/b86ffaf/ : SUCCESS in 4m 31s\n- gate-puppet-openstack_extras-puppet-unit-3.6-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-unit-3.6-dsvm-centos7/d0c469c/ : SUCCESS in 6m 33s\n- gate-puppet-openstack_extras-puppet-unit-3.7-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-unit-3.7-dsvm-centos7/57a14cd/ : SUCCESS in 6m 51s\n- gate-puppet-openstack_extras-puppet-unit-3.8-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-unit-3.8-dsvm-centos7/e1892fa/ : SUCCESS in 4m 43s\n- gate-puppet-openstack_extras-puppet-unit-4.0-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-unit-4.0-dsvm-centos7/726b46c/ : SUCCESS in 4m 32s\n- gate-puppet-openstack_extras-puppet-unit-latest-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-unit-latest-dsvm-centos7/b47857d/ : SUCCESS in 6m 38s (non-voting)\n- gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/fdbd067/ : SUCCESS in 20m 33s\n- gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/2fe7259/ : SUCCESS in 25m 20s\n- gate-puppet-openstack-integration-scenario002-tempest-dsvm-trusty http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-trusty/70f7e40/ : SUCCESS in 28m 43s\n- gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/a73f9af/ : SUCCESS in 23m 27s\n- gate-puppet-openstack_extras-puppet-beaker-rspec-dsvm-trusty http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-beaker-rspec-dsvm-trusty/16e2d60/ : SUCCESS in 7m 33s\n- gate-puppet-openstack_extras-puppet-beaker-rspec-dsvm-centos7 http://logs.openstack.org/49/292649/1/check/gate-puppet-openstack_extras-puppet-beaker-rspec-dsvm-centos7/6a19dc3/ : SUCCESS in 6m 35s","accounts_in_message":[],"_revision_number":1},{"id":"ef8a0297e6b8616b19f367b5980db2b9941f33b4","author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"date":"2016-03-15 09:18:13.000000000","message":"Patch Set 1: Code-Review-1\n\nprev commit is not merged, why you need a new patch for fixing that? Please update first commit.","accounts_in_message":[],"_revision_number":1},{"id":"8b4ac557de1daf896f120470c752d9215ac05668","author":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"date":"2016-03-15 09:21:09.000000000","message":"Patch Set 1:\n\nThe changes were cherry-picked off of master.","accounts_in_message":[],"_revision_number":1},{"id":"33778f1eeba490b2454831f7514268ae10016ad9","author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"date":"2016-03-15 09:25:01.000000000","message":"Patch Set 1:\n\nAs i understand, we are talking about this one: https://review.openstack.org/#/c/292648/1\n\nSo, even it was cherry-picked from master (but you didn\u0027t mentioned in commit message about that), i\u0027m not sure that it is good idea to spawn commits instead of fixing such minor problem in place.","accounts_in_message":[],"_revision_number":1},{"id":"095b8475847858ff47e5d49d0b3b6ef5918308a6","author":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"date":"2016-03-15 09:29:59.000000000","message":"Patch Set 1:\n\nthey\u0027re both cherry-picks. I could have made the changes myself but that would mean deviations from master. If the thought is for me to squash commits, I\u0027ll do that but I was preserving git history.","accounts_in_message":[],"_revision_number":1},{"id":"c2edf3af3630ecb160530326e4aacec52dd22015","author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"date":"2016-03-15 09:34:03.000000000","message":"Patch Set 1:\n\nI agree with you about keeping git history, but i think will be better to have cherry-pick without error. Ok, let\u0027s wait then for Emilien comment. If will be better to have two commits - i will drop my -1 from both changes.","accounts_in_message":[],"_revision_number":1},{"id":"782102e0025c28ab237db7548aea6593247a916f","author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"date":"2016-03-17 00:03:36.000000000","message":"Patch Set 1:\n\nIt is better to cherry-pick both commits. However you should use the \"cherry-pick\" button in the gerrit UI and the commit message should contain \"(cherry picked from commit \u003chash\u003e)\" (see e.g. https://review.openstack.org/#/c/293579/) so that it\u0027s obvious where it came from and who the original author was.","accounts_in_message":[],"_revision_number":1},{"id":"3cd2c849b209221606abdbe9c929c2cd083648e4","author":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"date":"2016-03-17 00:33:23.000000000","message":"Patch Set 1:\n\nI will abandon the change and do the cherry-pick through the interface.","accounts_in_message":[],"_revision_number":1},{"id":"7477d51232dc742b9f9381dff163b7dcca4b04dc","author":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"date":"2016-03-17 00:33:30.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"ddb8209f08a27128997d9639af43696478726d21","revisions":{"ddb8209f08a27128997d9639af43696478726d21":{"kind":"REWORK","_number":1,"created":"2016-03-14 23:54:44.000000000","uploader":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"ref":"refs/changes/49/292649/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-openstack_extras","ref":"refs/changes/49/292649/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-openstack_extras refs/changes/49/292649/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-openstack_extras refs/changes/49/292649/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-openstack_extras refs/changes/49/292649/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-openstack_extras refs/changes/49/292649/1"}}},"commit":{"parents":[{"commit":"82ca0bed77206c0a1e3af1671fe136b12c86a3c4","subject":"auth file: default urls to keystone v3","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack_extras/commit/82ca0bed77206c0a1e3af1671fe136b12c86a3c4"}]}],"author":{"name":"Emilien Macchi","email":"emilien@redhat.com","date":"2016-02-16 22:25:07.000000000","tz":-300},"committer":{"name":"Matthew Black","email":"matthew_black@cable.comcast.com","date":"2016-03-14 23:45:55.000000000","tz":-240},"subject":"authfile: fix OS_IDENTITY_API_VERSION","message":"authfile: fix OS_IDENTITY_API_VERSION\n\nIn a previous commit (I did, mea culpa), I set IDENTITY_API_VERSION\nwhile we should use OS_IDENTITY_API_VERSION.\n\nChange-Id: I625adb5d8ef1898bd9a44a76a09b3f985146c96c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack_extras/commit/ddb8209f08a27128997d9639af43696478726d21"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack_extras/commit/ddb8209f08a27128997d9639af43696478726d21"}]},"branch":"refs/heads/stable/liberty"}},"requirements":[],"submit_records":[],"submit_requirements":[]}