)]}'
{"id":"openstack%2Fplacement~666460","triplet_id":"openstack%2Fplacement~master~I141cde0395f45696d78ad73674a276eb6178aa70","project":"openstack/placement","branch":"master","topic":"granular-aggregate-flow-down","hashtags":[],"change_id":"I141cde0395f45696d78ad73674a276eb6178aa70","subject":"Add a test for granular member_of not flowing down","status":"MERGED","created":"2019-06-19 22:37:17.000000000","updated":"2019-07-10 07:23:35.000000000","submitted":"2019-07-10 07:23:35.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":8,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"666460-1562743415634-d0a65de9","meta_rev_id":"692248ab725f1e8f9543bfe100ef9f92ad90e4e0","_number":666460,"virtual_id_number":666460,"owner":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"date":"2019-07-08 16:40:18.000000000","_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":2,"date":"2019-07-10 07:23:35.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":0,"date":"2019-07-09 22:46:53.000000000","_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"}],"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":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"all":[{"value":2,"date":"2019-07-03 11:56:21.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2019-06-26 08:27:43.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":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"}],"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":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"all":[{"value":1,"date":"2019-07-10 07:11:41.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"}],"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"]},{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-06-26 08:27:43.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-07-08 16:40:18.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-07-10 07:11:41.000000000","updated_by":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"reviewer":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"state":"REVIEWER"},{"updated":"2019-07-10 07:23:35.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":"19994798b527c8d9ac4c41eab8ff071468ee17c3","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-06-19 22:37:17.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"4b2e43568b21a958e2407c3b03589c92973f4e19","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-06-19 22:38:32.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"a9253b0ea381baee813f25cfa8f9af50ce30f9e5","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-20 01:55:19.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/60/666460/1/check/openstack-tox-docs/ca0a895/html/ : SUCCESS in 4m 00s\n- openstack-tox-cover http://logs.openstack.org/60/666460/1/check/openstack-tox-cover/dfd7054/cover/ : SUCCESS in 5m 10s\n- openstack-tox-lower-constraints http://logs.openstack.org/60/666460/1/check/openstack-tox-lower-constraints/3026369/ : SUCCESS in 5m 08s\n- openstack-tox-pep8 http://logs.openstack.org/60/666460/1/check/openstack-tox-pep8/abb028d/ : SUCCESS in 3m 58s\n- openstack-tox-py27 http://logs.openstack.org/60/666460/1/check/openstack-tox-py27/0994559/ : SUCCESS in 4m 11s\n- openstack-tox-py36 http://logs.openstack.org/60/666460/1/check/openstack-tox-py36/2f734b2/ : SUCCESS in 4m 17s\n- openstack-tox-py37 http://logs.openstack.org/60/666460/1/check/openstack-tox-py37/cf0046a/ : SUCCESS in 7m 08s\n- openstack-tox-functional http://logs.openstack.org/60/666460/1/check/openstack-tox-functional/b24ccfc/ : SUCCESS in 6m 45s\n- openstack-tox-functional-py36 http://logs.openstack.org/60/666460/1/check/openstack-tox-functional-py36/205f630/ : SUCCESS in 5m 07s\n- placement-nova-tox-functional-py36 http://logs.openstack.org/60/666460/1/check/placement-nova-tox-functional-py36/64ab5e4/ : SUCCESS in 11m 50s","accounts_in_message":[],"_revision_number":1},{"id":"79399319faa9ef0c900c0184abfdcab362259d7b","author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"date":"2019-06-25 09:39:01.000000000","message":"Patch Set 1: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":1},{"id":"e22f2cd3907c84c79fb7a3094ab45392ddb0ee5e","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-06-25 14:34:39.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"7f8e39e37cdb7a6bd28163b95547b973a5979191","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-06-25 14:35:26.000000000","message":"Patch Set 1:\n\n(2 comments)","accounts_in_message":[],"_revision_number":1},{"id":"ad40b9e1c366ca796df771f578c7ec3a90a5f436","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-26 02:24:47.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/60/666460/2/check/openstack-tox-docs/eab9f1a/html/ : SUCCESS in 4m 47s\n- openstack-tox-cover http://logs.openstack.org/60/666460/2/check/openstack-tox-cover/fdb6547/cover/ : SUCCESS in 5m 19s\n- openstack-tox-lower-constraints http://logs.openstack.org/60/666460/2/check/openstack-tox-lower-constraints/97bbcbb/ : SUCCESS in 4m 38s\n- openstack-tox-pep8 http://logs.openstack.org/60/666460/2/check/openstack-tox-pep8/2a9dfcd/ : SUCCESS in 3m 51s\n- openstack-tox-py27 http://logs.openstack.org/60/666460/2/check/openstack-tox-py27/a7d1d78/ : SUCCESS in 4m 16s\n- openstack-tox-py36 http://logs.openstack.org/60/666460/2/check/openstack-tox-py36/23ae254/ : SUCCESS in 4m 49s\n- openstack-tox-py37 http://logs.openstack.org/60/666460/2/check/openstack-tox-py37/a3b0f34/ : SUCCESS in 5m 07s\n- openstack-tox-functional http://logs.openstack.org/60/666460/2/check/openstack-tox-functional/9b893b5/ : SUCCESS in 4m 30s\n- openstack-tox-functional-py36 http://logs.openstack.org/60/666460/2/check/openstack-tox-functional-py36/0426e19/ : SUCCESS in 4m 36s\n- placement-nova-tox-functional-py36 http://logs.openstack.org/60/666460/2/check/placement-nova-tox-functional-py36/1650c65/ : SUCCESS in 10m 46s","accounts_in_message":[],"_revision_number":2},{"id":"18fb682fb0268e2df11a80b7069e63f7a7c65f23","author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"date":"2019-06-26 08:27:43.000000000","message":"Patch Set 2: Code-Review+2\n\nThank you for updating. This behavior is controversial, but I think it is nice to have this patch to clear the current behavior. Since it is not urgent, it might be also nice to see some feedbacks from other several reviewers before landing.","accounts_in_message":[],"_revision_number":2},{"id":"d379e20964477b8edb53de7c02f60cfb663c4fd1","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-07-03 11:56:21.000000000","message":"Patch Set 2: Code-Review+2\n\nyeah, a useful test to have\n\ntowards the end of the cycle when we write the tao of placement, this is a good thing to remember\n\ntetsuro, I suspect we\u0027re not going to see much more in the way of additional reviews, but I don\u0027t want to +W over the top of your hope for more reviews. If you\u0027re happy please feel free to +W it.","accounts_in_message":[],"_revision_number":2},{"id":"f9158fc0b5f906c704bea08e5a8d98cb50df022f","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2019-07-08 16:40:18.000000000","message":"Patch Set 2:\n\n(1 comment)\n\n\u003e yeah, a useful test to have\n \u003e \n \u003e towards the end of the cycle when we write the tao of placement,\n \u003e this is a good thing to remember\n\nIf we not want to forget then we might want to add a release notes about it.","accounts_in_message":[],"_revision_number":2},{"id":"00f4bf5ad4fe7012409416c708172b12e77ab5e6","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-07-09 22:46:53.000000000","message":"Patch Set 2:\n\n(1 comment)\n\nIs there a reason to hold this or can we get it off the board?","accounts_in_message":[],"_revision_number":2},{"id":"2111421e53619cf19ced5d8603562838a44889aa","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-07-10 07:11:41.000000000","message":"Patch Set 2: Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"e6f77dc8021ffbbf0f76d56b5cdee0365868da4e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-10 07:11:53.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"122be17ff24623894e81832af006f98c9b1fc70e","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-07-10 07:13:38.000000000","message":"Patch Set 2:\n\n\u003e (1 comment)\n \u003e \n \u003e \u003e yeah, a useful test to have\n \u003e \u003e\n \u003e \u003e towards the end of the cycle when we write the tao of placement,\n \u003e \u003e this is a good thing to remember\n \u003e \n \u003e If we not want to forget then we might want to add a release notes\n \u003e about it.\n\nI think for this kind of functionality a release note is more confusing because it is behaviour that has been that way for some time. The thing that needs to be clarified is how this fits together and is useful in the context of the whole system, which release notes are not good at doing.","accounts_in_message":[],"_revision_number":2},{"id":"aee2b63da8fc26a252787c29b39ec9fe76c6cfa3","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-10 07:23:35.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/60/666460/2/gate/openstack-tox-docs/492792a/html/ : SUCCESS in 4m 29s\n- openstack-tox-lower-constraints http://logs.openstack.org/60/666460/2/gate/openstack-tox-lower-constraints/af22361/ : SUCCESS in 5m 13s\n- openstack-tox-pep8 http://logs.openstack.org/60/666460/2/gate/openstack-tox-pep8/2fd38be/ : SUCCESS in 4m 26s\n- openstack-tox-py27 http://logs.openstack.org/60/666460/2/gate/openstack-tox-py27/2e8d2a5/ : SUCCESS in 4m 33s\n- openstack-tox-py36 http://logs.openstack.org/60/666460/2/gate/openstack-tox-py36/e15089c/ : SUCCESS in 4m 46s\n- openstack-tox-py37 http://logs.openstack.org/60/666460/2/gate/openstack-tox-py37/46e958b/ : SUCCESS in 5m 10s\n- openstack-tox-functional http://logs.openstack.org/60/666460/2/gate/openstack-tox-functional/a5276f3/ : SUCCESS in 4m 49s\n- openstack-tox-functional-py36 http://logs.openstack.org/60/666460/2/gate/openstack-tox-functional-py36/18ea804/ : SUCCESS in 5m 13s\n- placement-nova-tox-functional-py36 http://logs.openstack.org/60/666460/2/gate/placement-nova-tox-functional-py36/2933854/ : SUCCESS in 10m 36s","accounts_in_message":[],"_revision_number":2},{"id":"160f75fe7e9f92446281c76b950ad437b6ab85f8","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-10 07:23:35.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"38d3d5d1c6215527f7bb0414e8cea2a50e19c53a","revisions":{"bff3e5e777c72571e0dfd3b4186dc463b9857423":{"kind":"REWORK","_number":1,"created":"2019-06-19 22:37:17.000000000","uploader":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"ref":"refs/changes/60/666460/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/60/666460/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/60/666460/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/60/666460/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/60/666460/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/60/666460/1"}}},"commit":{"parents":[{"commit":"9de03e1b17ddc202532757bb221ee81f02f89bbe","subject":"Uniquify allocation mappings","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/9de03e1b17ddc202532757bb221ee81f02f89bbe"}]}],"author":{"name":"Eric Fried","email":"openstack@fried.cc","date":"2019-06-19 22:35:43.000000000","tz":-300},"committer":{"name":"Eric Fried","email":"openstack@fried.cc","date":"2019-06-19 22:35:43.000000000","tz":-300},"subject":"Add a test for granular member_of flowing down","message":"Add a test for granular member_of flowing down\n\nThis is a simple addition to demonstrate that \"aggregates flow down\"\nworks for granular as well as unsuffixed member_of. It was prompted by\n[1].\n\n[1] https://review.opendev.org/#/c/662191/5/doc/source/specs/train/approved/2005575-nested-magic-1.rst@302\n\nChange-Id: I141cde0395f45696d78ad73674a276eb6178aa70\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/bff3e5e777c72571e0dfd3b4186dc463b9857423"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/bff3e5e777c72571e0dfd3b4186dc463b9857423"}]},"branch":"refs/heads/master"},"38d3d5d1c6215527f7bb0414e8cea2a50e19c53a":{"kind":"REWORK","_number":2,"created":"2019-06-25 14:34:39.000000000","uploader":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"ref":"refs/changes/60/666460/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/60/666460/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/60/666460/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/60/666460/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/60/666460/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/60/666460/2"}}},"commit":{"parents":[{"commit":"9de03e1b17ddc202532757bb221ee81f02f89bbe","subject":"Uniquify allocation mappings","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/9de03e1b17ddc202532757bb221ee81f02f89bbe"}]}],"author":{"name":"Eric Fried","email":"openstack@fried.cc","date":"2019-06-19 22:35:43.000000000","tz":-300},"committer":{"name":"Eric Fried","email":"openstack@fried.cc","date":"2019-06-25 14:34:04.000000000","tz":-300},"subject":"Add a test for granular member_of not flowing down","message":"Add a test for granular member_of not flowing down\n\nThis is a simple addition to demonstrate that \"aggregates flow down\"\ndoes not apply to granular member_of. It was prompted by [1].\n\n[1] https://review.opendev.org/#/c/662191/5/doc/source/specs/train/approved/2005575-nested-magic-1.rst@302\n\nChange-Id: I141cde0395f45696d78ad73674a276eb6178aa70\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/38d3d5d1c6215527f7bb0414e8cea2a50e19c53a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/38d3d5d1c6215527f7bb0414e8cea2a50e19c53a"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}