)]}'
{"id":"openstack%2Fnova-specs~603585","triplet_id":"openstack%2Fnova-specs~master~I5f403af2e8c98e0804d29ec36dbe4465961b2d83","project":"openstack/nova-specs","branch":"master","topic":"bp/alloc-candidates-in-tree","hashtags":[],"change_id":"I5f403af2e8c98e0804d29ec36dbe4465961b2d83","subject":"Spec: allocation candidates in tree","status":"MERGED","created":"2018-09-19 08:02:48.000000000","updated":"2018-11-14 08:22:18.000000000","submitted":"2018-11-14 08:22:18.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":16,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"603585-1542183738513-bba6691b","meta_rev_id":"84338ca185f1bb2c37e4fb1646a770ddd80d9f58","_number":603585,"virtual_id_number":603585,"owner":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"date":"2018-09-20 07:43:04.000000000","_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":2,"date":"2018-11-14 08:22:18.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"}],"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":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"recommended":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"all":[{"value":1,"date":"2018-10-24 02:31:57.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},{"value":1,"date":"2018-11-14 08:17:18.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":1,"date":"2018-09-20 13:47:50.000000000","permitted_voting_range":{"min":1,"max":1},"_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":"2018-10-23 21:56:36.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":2,"date":"2018-11-14 08:16:23.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2018-09-27 13:40:59.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"}],"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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"all":[{"value":0,"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"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":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":1,"date":"2018-11-14 08:16:23.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"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":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_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":"2018-09-20 13:47: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":"2018-09-27 13:40:59.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":"2018-10-23 21:56:36.000000000","updated_by":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"reviewer":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"state":"REVIEWER"},{"updated":"2018-10-24 02:31:57.000000000","updated_by":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"reviewer":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"state":"REVIEWER"},{"updated":"2018-11-14 08:16:23.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"state":"REVIEWER"},{"updated":"2018-11-14 08:17:18.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2018-11-14 08:22:18.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":"9de3cfbe50fbbe4da09b2aa3c0e56d3f6892af2c","author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"date":"2018-09-19 08:02:48.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"dc7d3c6031c15037c3b9bb05522c82a5d2275e8f","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-09-19 09:05:54.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/85/603585/1/check/openstack-tox-docs/dfeb1d5/html/ : SUCCESS in 5m 39s\n- openstack-tox-pep8 http://logs.openstack.org/85/603585/1/check/openstack-tox-pep8/cf72245/ : SUCCESS in 4m 39s","accounts_in_message":[],"_revision_number":1},{"id":"1fbbf1bca63ca07fec42e93ad6ce1ba2445c41c4","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2018-09-19 13:57:57.000000000","message":"Patch Set 1: Code-Review+1\n\n(5 comments)\n\nNice work Tetsuro. Clear, concise, and well-written.","accounts_in_message":[],"_revision_number":1},{"id":"0e244869b9e1d064b3e5a2559cfc4de8a91cb71c","author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"date":"2018-09-20 01:07:31.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"0213f3aa191c25dbd59feb08ac4e9ed31902db51","author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"date":"2018-09-20 01:07:50.000000000","message":"Patch Set 2:\n\n(3 comments)","accounts_in_message":[],"_revision_number":2},{"id":"a0c7fb6d371d1a7740300e9b3b116a0b3c0acf41","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-09-20 06:01:22.000000000","message":"Patch Set 2: 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- openstack-tox-docs http://logs.openstack.org/85/603585/2/check/openstack-tox-docs/f6d10b5/ : RETRY_LIMIT in 5m 27s\n- openstack-tox-pep8 http://logs.openstack.org/85/603585/2/check/openstack-tox-pep8/0ad9750/ : RETRY_LIMIT in 5m 28s","accounts_in_message":[],"_revision_number":2},{"id":"b663e8ad6aba354d068314b9996d1e76233dd141","author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"date":"2018-09-20 07:43:04.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"bdbd6ca6f15045f169df5c391db4e61a132c49a4","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-09-20 08:40:42.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/85/603585/2/check/openstack-tox-docs/1714006/html/ : SUCCESS in 5m 33s\n- openstack-tox-pep8 http://logs.openstack.org/85/603585/2/check/openstack-tox-pep8/3b9ee56/ : SUCCESS in 4m 37s","accounts_in_message":[],"_revision_number":2},{"id":"6ee4f5a632b3f82757d3533b5612ed97e5d9630c","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2018-09-20 13:47:50.000000000","message":"Patch Set 2: Code-Review+1\n\neven better","accounts_in_message":[],"_revision_number":2},{"id":"57dc027d03b7d7ecd4bbddec83421d9b3dad4ff4","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2018-09-27 13:40:59.000000000","message":"Patch Set 2: Code-Review+1\n\nnice clear explanation","accounts_in_message":[],"_revision_number":2},{"id":"b9a9e1d32806c6a0f66a551a61862593e86769b3","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2018-10-23 21:56:36.000000000","message":"Patch Set 2: Code-Review+2\n\n(4 comments)\n\nI have nits inline but those could be dealt with in a follow up. This is just the placement API changes, but it\u0027s consistent with what we discussed at the Stein PTG:\n\nhttps://etherpad.openstack.org/p/nova-ptg-stein ~L136\n\nThe accompanying nova changes don\u0027t really require a spec and could just be related bug fixes.","accounts_in_message":[],"_revision_number":2},{"id":"dacc17b8893e5f87c3feb284c04248809ecf8ef4","author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"date":"2018-10-24 02:31:57.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"42e1d33f8084a74b5ff53f58040f493e6446d4be","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2018-11-14 08:16:23.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\n(1 comment)\n\nI would have expected this to talk about the nova side fix to address the actual problem for which this is a solution, and thus my query about how we\u0027re going to test this would lead me to -1 it. But, I guess since this is just the placement side of the implementation I can\u0027t really hold it up, but I _am_ curious about how we test things like this peri- and post-split.","accounts_in_message":[],"_revision_number":2},{"id":"66d0abfcf33adb4adc2ed475287eaceb8ce5048c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-11-14 08:16:34.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"d69cadc16f7e86431d5c1fc7772ac24414780123","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2018-11-14 08:17:18.000000000","message":"Patch Set 2: Code-Review+1\n\n(2 comments)\n\nwe have talked about this in the past and had pretty solid usecases for using it in denver so +1 from me.","accounts_in_message":[],"_revision_number":2},{"id":"bae805506d886da6bd3d9b9fcc1117922eca4b73","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2018-11-14 08:20:31.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"adb03769094e4489889fbd152c2dccc633e26b49","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-11-14 08:22:18.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"b869800e2a2c808dd4b8e18369589116df5e267e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-11-14 08:22:18.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/85/603585/2/gate/openstack-tox-docs/ca04df4/html/ : SUCCESS in 5m 28s\n- openstack-tox-pep8 http://logs.openstack.org/85/603585/2/gate/openstack-tox-pep8/3c2a726/ : SUCCESS in 4m 15s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"8ba67f69660651e2da38c86f4632e803ce7b4b00","revisions":{"d6cb313f4560674eaa981a7b87b9c4bf83f84ad8":{"kind":"REWORK","_number":1,"created":"2018-09-19 08:02:48.000000000","uploader":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"ref":"refs/changes/85/603585/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/85/603585/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/85/603585/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/85/603585/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/85/603585/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/85/603585/1"}}},"commit":{"parents":[{"commit":"d4be64cfabae1d0172d1f1e37012e946e62d8004","subject":"Add support specify volume type when boot instance","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/d4be64cfabae1d0172d1f1e37012e946e62d8004"}]}],"author":{"name":"Tetsuro Nakamura","email":"nakamura.tetsuro@lab.ntt.co.jp","date":"2018-09-19 07:58:35.000000000","tz":540},"committer":{"name":"Tetsuro Nakamura","email":"nakamura.tetsuro@lab.ntt.co.jp","date":"2018-09-19 07:58:35.000000000","tz":540},"subject":"Spec: allocation candidates in tree","message":"Spec: allocation candidates in tree\n\nThis patch proposes a spec to support for filtering allocation\ncandidates by provider tree.\n\nChange-Id: I5f403af2e8c98e0804d29ec36dbe4465961b2d83\nBlueprint: alloc-candidates-in-tree\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/d6cb313f4560674eaa981a7b87b9c4bf83f84ad8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/d6cb313f4560674eaa981a7b87b9c4bf83f84ad8"}]},"branch":"refs/heads/master"},"8ba67f69660651e2da38c86f4632e803ce7b4b00":{"kind":"REWORK","_number":2,"created":"2018-09-20 01:07:31.000000000","uploader":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"ref":"refs/changes/85/603585/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova-specs","ref":"refs/changes/85/603585/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/85/603585/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/85/603585/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova-specs refs/changes/85/603585/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova-specs refs/changes/85/603585/2"}}},"commit":{"parents":[{"commit":"d4be64cfabae1d0172d1f1e37012e946e62d8004","subject":"Add support specify volume type when boot instance","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/d4be64cfabae1d0172d1f1e37012e946e62d8004"}]}],"author":{"name":"Tetsuro Nakamura","email":"nakamura.tetsuro@lab.ntt.co.jp","date":"2018-09-19 07:58:35.000000000","tz":540},"committer":{"name":"Tetsuro Nakamura","email":"nakamura.tetsuro@lab.ntt.co.jp","date":"2018-09-19 09:43:01.000000000","tz":540},"subject":"Spec: allocation candidates in tree","message":"Spec: allocation candidates in tree\n\nThis patch proposes a spec to support for filtering allocation\ncandidates by provider tree.\n\nChange-Id: I5f403af2e8c98e0804d29ec36dbe4465961b2d83\nBlueprint: alloc-candidates-in-tree\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/8ba67f69660651e2da38c86f4632e803ce7b4b00"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova-specs/commit/8ba67f69660651e2da38c86f4632e803ce7b4b00"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}