)]}'
{"id":"openstack%2Fdevstack~56824","triplet_id":"openstack%2Fdevstack~master~I047de155f0d9d2a1c009533c2f97f505cc80c6de","project":"openstack/devstack","branch":"master","topic":"token_format_deprecated","hashtags":[],"change_id":"I047de155f0d9d2a1c009533c2f97f505cc80c6de","subject":"Keystone don\u0027t use deprecated token_format option","status":"MERGED","created":"2013-11-17 19:06:02.000000000","updated":"2013-11-25 17:08:10.000000000","submitted":"2013-11-25 17:08:09.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d26f996ed745d8c3a68507aeb0ee7d64eb8a821e","_number":56824,"virtual_id_number":56824,"owner":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-11-25 17:08:09.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-11-18 17:03:24.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":2,"date":"2013-11-18 17:13:09.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-11-25 14:53:54.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"date":"2013-11-18 16:59:06.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-11-25 14:53:54.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-11-18 16:59:06.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":"2013-11-25 14:53:54.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2013-11-25 17:08:09.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"1f89aaea9a6e962da4f1a80e1a5396f46c514876","author":{"_account_id":9009,"name":"Red Hat CI","email":"redhatci@redhat.com","username":"redhatci","tags":["SERVICE_USER"]},"date":"2013-11-17 19:47:32.000000000","message":"Patch Set 1:\n\nredhatci results for change 56824\n\nLogs at http://people.redhat.com/~iwienand/56824/\n\n- Patch set 1 | OSLAB RHEL65 : FAIL\n- Patch set 1 | OSLAB Fedora 19 : PASS\n- Patch set 1 | OSLAB RHEL64 : FAIL\n- Patch set 1 | OSLAB CentOS 6.4 : FAIL\n\nend","accounts_in_message":[],"_revision_number":1},{"id":"a4c015b4fe3b8e46e722c49a23b1b34362e7468f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-17 19:53:40.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/24/56824/1/check/gate-devstack-bash8/6b1a9ac : SUCCESS in 11s\n- check-tempest-devstack-vm-full http://logs.openstack.org/24/56824/1/check/check-tempest-devstack-vm-full/3a02e21 : SUCCESS in 36m 31s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/24/56824/1/check/check-tempest-devstack-vm-postgres-full/6a36b27 : SUCCESS in 47m 30s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/24/56824/1/check/check-tempest-devstack-vm-neutron/2b289a8 : SUCCESS in 29m 19s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/24/56824/1/check/gate-tempest-devstack-vm-large-ops/13a2342 : SUCCESS in 13m 54s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/24/56824/1/check/gate-tempest-devstack-vm-neutron-large-ops/e1c05c5 : SUCCESS in 16m 02s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/24/56824/1/check/check-grenade-devstack-vm/4e860dd : SUCCESS in 35m 36s\n- check-swift-devstack-vm-functional http://logs.openstack.org/24/56824/1/check/check-swift-devstack-vm-functional/59210ff : SUCCESS in 12m 09s\n","accounts_in_message":[],"_revision_number":1},{"id":"70cb14d23adf4f0931aac6aa532006d8fb78e303","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-11-18 16:59:06.000000000","message":"Patch Set 1:\n\nIs support for UUID tokens in general deprecated?  If so maybe we should remove all of that and not just the option setting?","accounts_in_message":[],"_revision_number":1},{"id":"90eca72a98c87728b71df92762c3657358b3b358","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2013-11-18 17:03:24.000000000","message":"Patch Set 1:\n\nDean - UUID tokens are not deprecated, and I don\u0027t think there\u0027s any plan to deprecate them. If there was a suggestion to remove UUID token format support I\u0027d push back on that since there\u0027s some question as to if PKI tokens are the best option in all cases.\n\nI tried this change with KEYSTONE_TOKEN_FORMAT\u003dUUID and it worked for me (I got back a UUID token with keystone token-get).","accounts_in_message":[],"_revision_number":1},{"id":"d51cfd47ac0fd87b94ad1e730a195a3c7b48856b","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2013-11-18 17:05:36.000000000","message":"Patch Set 1:\n\nIf the question is more about if devstack needs to support UUID token config... that I\u0027m not sure of.","accounts_in_message":[],"_revision_number":1},{"id":"1e24a419f032fd4c07d88ecd18fe88999640d0bc","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-11-18 17:13:09.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\nThanks Brant, that\u0027s what I needed, DevStack does need to continue to support UUID tokens as long as Keystone has them.","accounts_in_message":[],"_revision_number":1},{"id":"2ef00c807335d0bcb92eb1a3199d13bc427948a8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-25 14:53:54.000000000","message":"Patch Set 1: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"2bddbcfa2e33be17abefadf74b962d2b0508b30d","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-11-25 14:53:54.000000000","message":"Patch Set 1: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":1},{"id":"e78b5701a287b080e56f7729284a5557d47548e5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-25 17:08:09.000000000","message":"Patch Set 1: Verified\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/24/56824/1/gate/gate-devstack-bash8/ef414c6 : SUCCESS in 9s\n- gate-tempest-devstack-vm-full http://logs.openstack.org/24/56824/1/gate/gate-tempest-devstack-vm-full/c301060 : SUCCESS in 39m 41s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/24/56824/1/gate/gate-tempest-devstack-vm-postgres-full/6ed8d84 : SUCCESS in 31m 00s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/24/56824/1/gate/gate-tempest-devstack-vm-neutron/fcbce62 : SUCCESS in 38m 20s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/24/56824/1/gate/gate-tempest-devstack-vm-large-ops/810ec54 : SUCCESS in 14m 25s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/24/56824/1/gate/gate-tempest-devstack-vm-neutron-large-ops/f74d9d5 : SUCCESS in 16m 26s\n- gate-grenade-devstack-vm http://logs.openstack.org/24/56824/1/gate/gate-grenade-devstack-vm/2eb1a58 : SUCCESS in 32m 05s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/24/56824/1/gate/gate-swift-devstack-vm-functional/d153412 : SUCCESS in 22m 38s\n","accounts_in_message":[],"_revision_number":1},{"id":"a60ce3faa5df4929a4de8b45c4ae85105ce6f3ca","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-25 17:08:10.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"f653419b15d244fa7e01a101de93174d282185ed","revisions":{"f653419b15d244fa7e01a101de93174d282185ed":{"kind":"REWORK","_number":1,"created":"2013-11-17 19:06:02.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/24/56824/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/24/56824/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/56824/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/56824/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/56824/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/24/56824/1"}}},"commit":{"parents":[{"commit":"c85ac787ba43115a3f47416470ef16e21ad20caf","subject":"Merge \"Don\u0027t install novnc from package by default\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c85ac787ba43115a3f47416470ef16e21ad20caf"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2013-11-17 19:03:52.000000000","tz":-360},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2013-11-17 19:03:52.000000000","tz":-360},"subject":"Keystone don\u0027t use deprecated token_format option","message":"Keystone don\u0027t use deprecated token_format option\n\ndevstack was setting the token_format option in the keystone\nconfiguration file. This option is deprecated so should not be\nused.\n\nChange-Id: I047de155f0d9d2a1c009533c2f97f505cc80c6de\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f653419b15d244fa7e01a101de93174d282185ed"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f653419b15d244fa7e01a101de93174d282185ed"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
