)]}'
{"id":"openstack%2Fplacement~664594","triplet_id":"openstack%2Fplacement~master~I136bd7cd89f1bd54f0d1691268545850af234f18","project":"openstack/placement","branch":"master","topic":"story/2005859","hashtags":[],"change_id":"I136bd7cd89f1bd54f0d1691268545850af234f18","subject":"Correctly limit provider summaries when nested","status":"MERGED","created":"2019-06-11 13:41:02.000000000","updated":"2019-06-12 09:45:26.000000000","submitted":"2019-06-12 09:45:25.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"664594-1560332726040-4a3a791a","meta_rev_id":"e523b334000bde9daf97b5feda9eaac1cfa23e36","_number":664594,"virtual_id_number":664594,"owner":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":2,"date":"2019-06-12 09:45:25.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"all":[{"value":2,"date":"2019-06-11 17:10:50.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":2,"date":"2019-06-12 08:02:03.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"all":[{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":1,"date":"2019-06-12 08:02:03.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-06-11 17:10:50.000000000","updated_by":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"reviewer":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"state":"REVIEWER"},{"updated":"2019-06-12 08:02:03.000000000","updated_by":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"reviewer":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"state":"REVIEWER"},{"updated":"2019-06-12 09:45:25.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"90aa9564205f869ca6758c6dc06c789c2845c9bf","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-06-11 13:41:02.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c58ec1cf54224a72230a589131ff6bcfff59978e","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-06-11 17:10:50.000000000","message":"Patch Set 1: Code-Review+2\n\n(4 comments)\n\nLooks right. Future-proofing suggestion inline.","accounts_in_message":[],"_revision_number":1},{"id":"383487d226036bcb5075514ce1f90001fe879a82","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-11 20:11:28.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- grenade-py3 http://logs.openstack.org/94/664594/1/check/grenade-py3/25a9295/ : FAILURE in 52m 19s\n- tempest-full-py3 http://logs.openstack.org/94/664594/1/check/tempest-full-py3/cbd6930/ : SUCCESS in 1h 35m 14s\n- openstack-tox-docs http://logs.openstack.org/94/664594/1/check/openstack-tox-docs/1898a4c/html/ : SUCCESS in 4m 33s\n- openstack-tox-cover http://logs.openstack.org/94/664594/1/check/openstack-tox-cover/108b34c/cover/ : SUCCESS in 4m 41s\n- openstack-tox-lower-constraints http://logs.openstack.org/94/664594/1/check/openstack-tox-lower-constraints/ab41628/ : SUCCESS in 4m 32s\n- openstack-tox-pep8 http://logs.openstack.org/94/664594/1/check/openstack-tox-pep8/ba5fc65/ : SUCCESS in 4m 21s\n- openstack-tox-py27 http://logs.openstack.org/94/664594/1/check/openstack-tox-py27/e7cd889/ : SUCCESS in 4m 12s\n- openstack-tox-py36 http://logs.openstack.org/94/664594/1/check/openstack-tox-py36/05f3266/ : SUCCESS in 3m 56s\n- openstack-tox-py37 http://logs.openstack.org/94/664594/1/check/openstack-tox-py37/ad63828/ : SUCCESS in 4m 31s\n- build-openstack-releasenotes http://logs.openstack.org/94/664594/1/check/build-openstack-releasenotes/fa72da5/html/ : SUCCESS in 4m 10s\n- openstack-tox-functional http://logs.openstack.org/94/664594/1/check/openstack-tox-functional/0510aa7/ : SUCCESS in 5m 17s\n- openstack-tox-functional-py36 http://logs.openstack.org/94/664594/1/check/openstack-tox-functional-py36/c7ab2e5/ : SUCCESS in 4m 55s\n- placement-nova-tox-functional-py36 http://logs.openstack.org/94/664594/1/check/placement-nova-tox-functional-py36/c4d2444/ : SUCCESS in 11m 15s\n- placement-perfload http://logs.openstack.org/94/664594/1/check/placement-perfload/97a2774/ : SUCCESS in 7m 19s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"b2ac2e3fb3dc65ade2c1f4825a551e22da583988","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-06-11 20:18:11.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"93bf073b28011e014171db4ca660e16a9b6505f6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-12 04:02:35.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-py3 http://logs.openstack.org/94/664594/1/check/grenade-py3/d91d6d0/ : SUCCESS in 49m 37s\n- tempest-full-py3 http://logs.openstack.org/94/664594/1/check/tempest-full-py3/cbcaf4f/ : SUCCESS in 1h 28m 50s\n- openstack-tox-docs http://logs.openstack.org/94/664594/1/check/openstack-tox-docs/5daa49e/html/ : SUCCESS in 5m 00s\n- openstack-tox-cover http://logs.openstack.org/94/664594/1/check/openstack-tox-cover/c95ac72/cover/ : SUCCESS in 7m 17s\n- openstack-tox-lower-constraints http://logs.openstack.org/94/664594/1/check/openstack-tox-lower-constraints/39127e2/ : SUCCESS in 7m 38s\n- openstack-tox-pep8 http://logs.openstack.org/94/664594/1/check/openstack-tox-pep8/02a831f/ : SUCCESS in 4m 37s\n- openstack-tox-py27 http://logs.openstack.org/94/664594/1/check/openstack-tox-py27/2bcecfd/ : SUCCESS in 4m 44s\n- openstack-tox-py36 http://logs.openstack.org/94/664594/1/check/openstack-tox-py36/1937097/ : SUCCESS in 4m 04s\n- openstack-tox-py37 http://logs.openstack.org/94/664594/1/check/openstack-tox-py37/d43dec5/ : SUCCESS in 6m 27s\n- build-openstack-releasenotes http://logs.openstack.org/94/664594/1/check/build-openstack-releasenotes/6136157/html/ : SUCCESS in 4m 33s\n- openstack-tox-functional http://logs.openstack.org/94/664594/1/check/openstack-tox-functional/8f96b94/ : SUCCESS in 4m 56s\n- openstack-tox-functional-py36 http://logs.openstack.org/94/664594/1/check/openstack-tox-functional-py36/0635d2c/ : SUCCESS in 4m 38s\n- placement-nova-tox-functional-py36 http://logs.openstack.org/94/664594/1/check/placement-nova-tox-functional-py36/2a4a899/ : SUCCESS in 12m 50s\n- placement-perfload http://logs.openstack.org/94/664594/1/check/placement-perfload/669e8ef/ : SUCCESS in 6m 29s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"af40a99ead9bed8cc635e9ac7279c6b3c81cbe8e","author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"date":"2019-06-12 08:02:03.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1\n\nGood find. LGTM. Thank you.","accounts_in_message":[],"_revision_number":1},{"id":"b75d16b251f8102f446d728fcc5c87865369247f","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-12 08:02:17.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"186a81068f3e35b498c95eb5494bcb405ca85298","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-12 09:45:25.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade-py3 http://logs.openstack.org/94/664594/1/gate/grenade-py3/a0b8000/ : SUCCESS in 58m 28s\n- tempest-full-py3 http://logs.openstack.org/94/664594/1/gate/tempest-full-py3/40a616f/ : SUCCESS in 1h 42m 22s\n- openstack-tox-docs http://logs.openstack.org/94/664594/1/gate/openstack-tox-docs/7a581b2/html/ : SUCCESS in 4m 47s\n- openstack-tox-lower-constraints http://logs.openstack.org/94/664594/1/gate/openstack-tox-lower-constraints/1d491c9/ : SUCCESS in 4m 47s\n- openstack-tox-pep8 http://logs.openstack.org/94/664594/1/gate/openstack-tox-pep8/3715bf1/ : SUCCESS in 4m 17s\n- openstack-tox-py27 http://logs.openstack.org/94/664594/1/gate/openstack-tox-py27/d501268/ : SUCCESS in 4m 40s\n- openstack-tox-py36 http://logs.openstack.org/94/664594/1/gate/openstack-tox-py36/065cf3c/ : SUCCESS in 5m 02s\n- openstack-tox-py37 http://logs.openstack.org/94/664594/1/gate/openstack-tox-py37/4b157ff/ : SUCCESS in 4m 19s\n- build-openstack-releasenotes http://logs.openstack.org/94/664594/1/gate/build-openstack-releasenotes/7c8bc6c/html/ : SUCCESS in 4m 42s\n- openstack-tox-functional http://logs.openstack.org/94/664594/1/gate/openstack-tox-functional/9de9e20/ : SUCCESS in 4m 56s\n- openstack-tox-functional-py36 http://logs.openstack.org/94/664594/1/gate/openstack-tox-functional-py36/65a162f/ : SUCCESS in 4m 47s\n- placement-nova-tox-functional-py36 http://logs.openstack.org/94/664594/1/gate/placement-nova-tox-functional-py36/041372e/ : SUCCESS in 11m 28s","accounts_in_message":[],"_revision_number":1},{"id":"26fc996b08a49441be12ec5248e80b49fa69dd9a","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-12 09:45:26.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"e1783b0087e36caebf7f22c8c1cd927bd33a0834","revisions":{"e1783b0087e36caebf7f22c8c1cd927bd33a0834":{"kind":"REWORK","_number":1,"created":"2019-06-11 13:41:02.000000000","uploader":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"ref":"refs/changes/94/664594/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/94/664594/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/94/664594/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/94/664594/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/94/664594/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/94/664594/1"}}},"commit":{"parents":[{"commit":"c919b1f568544a07bb5e8ddd1efd26dc802e7d8d","subject":"Merge \"Modernize CORS config and setup\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/c919b1f568544a07bb5e8ddd1efd26dc802e7d8d"}]}],"author":{"name":"Chris Dent","email":"cdent@anticdent.org","date":"2019-06-11 13:29:55.000000000","tz":60},"committer":{"name":"Chris Dent","email":"cdent@anticdent.org","date":"2019-06-11 13:40:25.000000000","tz":60},"subject":"Correctly limit provider summaries when nested","message":"Correctly limit provider summaries when nested\n\nWhen making a GET /allocation_candidates request with the \u0027limit\u0027\nparameter, include all the providers in the same tree as the providers\nmentioned in the allocation requests. That is, in a nested situation,\nif we limit the allocation requests to 10, we would usually expect\nconsiderably more than 10 providers in provider summaries, to include\nany non-contributing providers in the same tree.\n\nAccomplish this by filtering provider summaries on root_provider_uuid\ninstead of individual provider uuid.\n\nA gabbi test is added which exercises this in a nested situation.\n\nAn existing test is updated to reflect the new filtering style. It\ndoes not exercise the nested scenario.\n\nA reno is added indicating the fix.\n\nChange-Id: I136bd7cd89f1bd54f0d1691268545850af234f18\nStory: 2005859\nTask: 33654\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/e1783b0087e36caebf7f22c8c1cd927bd33a0834"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/e1783b0087e36caebf7f22c8c1cd927bd33a0834"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}