)]}'
{"id":"openstack%2Fdevstack~226480","triplet_id":"openstack%2Fdevstack~master~If1b77eef0d4f0ebdcdf761ecb5e2011484f73871","project":"openstack/devstack","branch":"master","topic":"bug/1498599","hashtags":[],"change_id":"If1b77eef0d4f0ebdcdf761ecb5e2011484f73871","subject":"Fix return value of get_or_add_user_project_role","status":"MERGED","created":"2015-09-22 18:15:20.000000000","updated":"2015-09-28 21:06:00.000000000","submitted":"2015-09-28 21:05:58.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d189f4573c2ef90c89eb41ea677f3268d3e5332e","_number":226480,"virtual_id_number":226480,"owner":{"_account_id":15640,"name":"Roxana Gherle","email":"roxana.gherle@hpe.com","username":"roxana-gherle"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"date":"2015-09-22 20:00:23.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":2,"date":"2015-09-28 21:05:58.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"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":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"all":[{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2015-09-22 20:25:11.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":2,"date":"2015-09-28 19:48:04.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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":1,"date":"2015-09-28 19:48:04.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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-09-22 20:00:23.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2015-09-22 20:25:11.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"},{"updated":"2015-09-28 19:48:04.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":"2015-09-28 21:05:58.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"d3f694547bafa9261998254b05584fec887d7859","author":{"_account_id":15640,"name":"Roxana Gherle","email":"roxana.gherle@hpe.com","username":"roxana-gherle"},"date":"2015-09-22 18:15:20.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b32e88d5ff3dcfb699f13b89757ff7d76a87a024","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-22 19:36:09.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/80/226480/1/check/gate-tempest-dsvm-full/98e51dc/ : SUCCESS in 54m 47s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/80/226480/1/check/gate-tempest-dsvm-postgres-full/725dcf4/ : SUCCESS in 50m 10s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/80/226480/1/check/gate-tempest-dsvm-neutron-full/e97a1a5/ : SUCCESS in 1h 15m 26s\n- gate-grenade-dsvm http://logs.openstack.org/80/226480/1/check/gate-grenade-dsvm/fc72ede/ : SUCCESS in 45m 31s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/80/226480/1/check/gate-tempest-dsvm-large-ops/7e2c42b/ : SUCCESS in 51m 47s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/80/226480/1/check/gate-tempest-dsvm-neutron-large-ops/5c443c3/ : SUCCESS in 39m 05s\n- gate-devstack-bashate http://logs.openstack.org/80/226480/1/check/gate-devstack-bashate/846a451/ : SUCCESS in 18s\n- gate-devstack-unit-tests http://logs.openstack.org/80/226480/1/check/gate-devstack-unit-tests/0adc870/ : SUCCESS in 19s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/80/226480/1/check/gate-grenade-dsvm-partial-ncpu/e4e84ba/ : SUCCESS in 39m 46s\n- gate-tempest-dsvm-ironic-agent_ssh http://logs.openstack.org/80/226480/1/check/gate-tempest-dsvm-ironic-agent_ssh/767dc62/ : SUCCESS in 41m 58s\n- gate-devstack-dsvm-updown http://logs.openstack.org/80/226480/1/check/gate-devstack-dsvm-updown/3285e3f/ : SUCCESS in 31m 36s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/80/226480/1/check/gate-tempest-dsvm-f21/9157148/ : SUCCESS in 43m 05s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/80/226480/1/check/gate-tempest-dsvm-centos7/c0d9460/ : SUCCESS in 32m 04s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/80/226480/1/check/gate-devstack-publish-docs/bb0c461//doc/build/html/ : SUCCESS in 1m 49s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/80/226480/1/check/gate-swift-dsvm-functional-nv/13205ce/ : SUCCESS in 34m 27s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/80/226480/1/check/gate-grenade-dsvm-neutron/f42a70b/ : SUCCESS in 1h 09m 46s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/80/226480/1/check/gate-tempest-dsvm-multinode-full/e2b9f5e/ : FAILURE in 55m 34s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/80/226480/1/check/gate-tempest-dsvm-neutron-multinode-full/37411d8/ : SUCCESS in 1h 00m 50s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"5a7db7ad91b81e53effd3000680cfba608f2b5ea","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-09-22 20:00:23.000000000","message":"Patch Set 1:\n\nTesting completed on Citrix Xenserver. For rechecking only on the Citrix Xenserver CI,add a review comment with \"xenserver: recheck\". Contact info: openstack@citrix.com.\nFor debugging suggestions, see https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures\n\n- check-citrix-xenserver http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/80/226480/1/12071/index.html : SUCCESS in 01 h 42 m 13 s","accounts_in_message":[],"_revision_number":1},{"id":"de7aa135eb401fbda48210b872a90d794ee4ba69","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-09-22 20:25:11.000000000","message":"Patch Set 1: Code-Review+2\n\nThanks!  I don\u0027t think it actually gets used, but it is the way it\u0027s written so it\u0027s nice to have it working","accounts_in_message":[],"_revision_number":1},{"id":"6445c764b87f32afcc8e216ee1a83d9c0ea74b3b","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2015-09-28 19:48:04.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"b7785922668e0bf80c02bba129368176dc426b9f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-28 19:48:10.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"9d7cc692e7212d62346a7039bca8e2316a7d26c4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-28 21:05:58.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/80/226480/1/gate/gate-tempest-dsvm-full/655356c/ : SUCCESS in 43m 58s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/80/226480/1/gate/gate-tempest-dsvm-postgres-full/cd05102/ : SUCCESS in 40m 37s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/80/226480/1/gate/gate-tempest-dsvm-neutron-full/2660770/ : SUCCESS in 1h 17m 14s\n- gate-grenade-dsvm http://logs.openstack.org/80/226480/1/gate/gate-grenade-dsvm/40199a2/ : SUCCESS in 46m 41s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/80/226480/1/gate/gate-tempest-dsvm-large-ops/7a2858f/ : SUCCESS in 39m 34s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/80/226480/1/gate/gate-tempest-dsvm-neutron-large-ops/b64d850/ : SUCCESS in 34m 06s\n- gate-devstack-bashate http://logs.openstack.org/80/226480/1/gate/gate-devstack-bashate/1d65148/ : SUCCESS in 28s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/80/226480/1/gate/gate-grenade-dsvm-partial-ncpu/642304a/ : SUCCESS in 41m 54s","accounts_in_message":[],"_revision_number":1},{"id":"70df8f771becbc249eb3c5986d3de18d3c5c2bc4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-28 21:06:00.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"50821bed081e94dfd4b75cf02121a42a56cdbaac","revisions":{"50821bed081e94dfd4b75cf02121a42a56cdbaac":{"kind":"REWORK","_number":1,"created":"2015-09-22 18:15:20.000000000","uploader":{"_account_id":15640,"name":"Roxana Gherle","email":"roxana.gherle@hpe.com","username":"roxana-gherle"},"ref":"refs/changes/80/226480/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/80/226480/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/80/226480/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/80/226480/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/80/226480/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/80/226480/1"}}},"commit":{"parents":[{"commit":"c4c08686f81384c90386228a45583667d877b6ef","subject":"Merge \"Ironic: Fix tag ID detection of tap devices\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c4c08686f81384c90386228a45583667d877b6ef"}]}],"author":{"name":"Roxana Gherle","email":"roxana.gherle@hp.com","date":"2015-09-22 17:52:46.000000000","tz":-420},"committer":{"name":"Roxana Gherle","email":"roxana.gherle@hp.com","date":"2015-09-22 18:11:36.000000000","tz":-420},"subject":"Fix return value of get_or_add_user_project_role","message":"Fix return value of get_or_add_user_project_role\n\nget_or_add_user_project_role function was always returning an\nempty user_role_id because the role assignment command does not\nreturn any output. Added a command to get the user_role_id after\nthe assignment happens.\n\nCloses-Bug: #1498599\nChange-Id: If1b77eef0d4f0ebdcdf761ecb5e2011484f73871\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/50821bed081e94dfd4b75cf02121a42a56cdbaac"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/50821bed081e94dfd4b75cf02121a42a56cdbaac"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}