)]}'
{"id":"openstack%2Fplacement~665638","triplet_id":"openstack%2Fplacement~master~If00f01534b7d0ec84ca8abaeef5b90a16cbcffc3","project":"openstack/placement","branch":"master","topic":"cd/mappings-fix","hashtags":[],"change_id":"If00f01534b7d0ec84ca8abaeef5b90a16cbcffc3","subject":"Uniquify allocation mappings","status":"MERGED","created":"2019-06-17 10:19:14.000000000","updated":"2019-06-17 16:26:41.000000000","submitted":"2019-06-17 16:26:41.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"665638-1560788801541-63dd4f34","meta_rev_id":"ad55709f1599186c3d7951ca9a2ebde431625538","_number":665638,"virtual_id_number":665638,"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":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"date":"2019-06-17 14:19:08.000000000","_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":2,"date":"2019-06-17 16:26:41.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-17 14:14:28.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-17 14:15:55.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"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":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"all":[{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":1,"date":"2019-06-17 14:15:55.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"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":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"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":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-06-17 14:14:28.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-17 14:15:55.000000000","updated_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"reviewer":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"state":"REVIEWER"},{"updated":"2019-06-17 16:26:41.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":"f722488a5b40c55633e87ccad2f575ffa5956106","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-06-17 10:19:14.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"16d85ba85de7411f711f4a692a989a17d83f5118","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-17 11:50:41.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-py3 http://logs.openstack.org/38/665638/1/check/grenade-py3/b873740/ : SUCCESS in 1h 07m 49s\n- tempest-full-py3 http://logs.openstack.org/38/665638/1/check/tempest-full-py3/63ec704/ : SUCCESS in 1h 29m 24s\n- openstack-tox-docs http://logs.openstack.org/38/665638/1/check/openstack-tox-docs/fb9c667/html/ : SUCCESS in 4m 28s\n- openstack-tox-cover http://logs.openstack.org/38/665638/1/check/openstack-tox-cover/52dd346/cover/ : SUCCESS in 5m 41s\n- openstack-tox-lower-constraints http://logs.openstack.org/38/665638/1/check/openstack-tox-lower-constraints/6c26c45/ : SUCCESS in 6m 43s\n- openstack-tox-pep8 http://logs.openstack.org/38/665638/1/check/openstack-tox-pep8/e8bc974/ : SUCCESS in 4m 04s\n- openstack-tox-py27 http://logs.openstack.org/38/665638/1/check/openstack-tox-py27/d6b96e1/ : SUCCESS in 4m 51s\n- openstack-tox-py36 http://logs.openstack.org/38/665638/1/check/openstack-tox-py36/9dc8d5b/ : SUCCESS in 4m 24s\n- openstack-tox-py37 http://logs.openstack.org/38/665638/1/check/openstack-tox-py37/c81b9ac/ : SUCCESS in 5m 16s\n- openstack-tox-functional http://logs.openstack.org/38/665638/1/check/openstack-tox-functional/e6dd9d2/ : SUCCESS in 5m 06s\n- openstack-tox-functional-py36 http://logs.openstack.org/38/665638/1/check/openstack-tox-functional-py36/0c39859/ : SUCCESS in 4m 52s\n- placement-nova-tox-functional-py36 http://logs.openstack.org/38/665638/1/check/placement-nova-tox-functional-py36/6314388/ : SUCCESS in 9m 48s\n- placement-perfload http://logs.openstack.org/38/665638/1/check/placement-perfload/adf3f9b/ : SUCCESS in 7m 23s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"9d9e4f535fefa23ffcd290a449b8142118d61ce4","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-06-17 14:14:28.000000000","message":"Patch Set 1: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"538e1178576ca3e3c68deb55d903ccd4f9d6126c","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2019-06-17 14:15:55.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1\n\n(1 comment)\n\nTest case fails without the fix. LGTM!","accounts_in_message":[],"_revision_number":1},{"id":"a82cec8c516c6c9a62f5da32ce4f8e597e67bc3c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-17 14:16:15.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"d805f8516c6126a579c6b5708bc320f2837dd342","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-06-17 14:19:08.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"c33941022f50fa3901b51a0cf514366cb1f7231b","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-17 16:26:41.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"2a5ef12f1a4f67c8f6a782e6fe0a7ac776f52cb8","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-17 16:26:41.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade-py3 http://logs.openstack.org/38/665638/1/gate/grenade-py3/aa3b8cf/ : SUCCESS in 1h 00m 35s\n- tempest-full-py3 http://logs.openstack.org/38/665638/1/gate/tempest-full-py3/cf0fd86/ : SUCCESS in 1h 42m 35s\n- openstack-tox-docs http://logs.openstack.org/38/665638/1/gate/openstack-tox-docs/bdcfe86/html/ : SUCCESS in 4m 30s\n- openstack-tox-lower-constraints http://logs.openstack.org/38/665638/1/gate/openstack-tox-lower-constraints/19883e3/ : SUCCESS in 7m 06s\n- openstack-tox-pep8 http://logs.openstack.org/38/665638/1/gate/openstack-tox-pep8/0bab28c/ : SUCCESS in 4m 37s\n- openstack-tox-py27 http://logs.openstack.org/38/665638/1/gate/openstack-tox-py27/150f561/ : SUCCESS in 5m 12s\n- openstack-tox-py36 http://logs.openstack.org/38/665638/1/gate/openstack-tox-py36/da0de3d/ : SUCCESS in 4m 24s\n- openstack-tox-py37 http://logs.openstack.org/38/665638/1/gate/openstack-tox-py37/35585f9/ : SUCCESS in 11m 40s\n- openstack-tox-functional http://logs.openstack.org/38/665638/1/gate/openstack-tox-functional/f836079/ : SUCCESS in 4m 57s\n- openstack-tox-functional-py36 http://logs.openstack.org/38/665638/1/gate/openstack-tox-functional-py36/90ba27f/ : SUCCESS in 5m 15s\n- placement-nova-tox-functional-py36 http://logs.openstack.org/38/665638/1/gate/placement-nova-tox-functional-py36/f678a8a/ : SUCCESS in 10m 38s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"9de03e1b17ddc202532757bb221ee81f02f89bbe","revisions":{"9de03e1b17ddc202532757bb221ee81f02f89bbe":{"kind":"REWORK","_number":1,"created":"2019-06-17 10:19:14.000000000","uploader":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"ref":"refs/changes/38/665638/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/38/665638/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/38/665638/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/38/665638/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/38/665638/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/38/665638/1"}}},"commit":{"parents":[{"commit":"c2df483bec79cbec7575064e298939abf9407c31","subject":"Merge \"Remove a redundant test\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/c2df483bec79cbec7575064e298939abf9407c31"}]}],"author":{"name":"Chris Dent","email":"cdent@anticdent.org","date":"2019-06-17 10:16:34.000000000","tz":60},"committer":{"name":"Chris Dent","email":"cdent@anticdent.org","date":"2019-06-17 10:16:34.000000000","tz":60},"subject":"Uniquify allocation mappings","message":"Uniquify allocation mappings\n\nWhen creating the allocation mappings output, use a set() not a\nlist() so that any single resource provider only shows up once.\nThe initial implementation (at\nIe78ed7e050416d4ccb62697ba608131038bb4303) allowed a provider to\nshow up multiple times if it contributed more than one class\nof inventory.\n\nThis slipped through because of a missing test. A test that covers\nthe issue is added.k\n\nChange-Id: If00f01534b7d0ec84ca8abaeef5b90a16cbcffc3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/9de03e1b17ddc202532757bb221ee81f02f89bbe"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/9de03e1b17ddc202532757bb221ee81f02f89bbe"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}