)]}'
{"id":"openstack%2Fdevstack~381965","triplet_id":"openstack%2Fdevstack~master~I48c86f94098f1501f0e7f90a265dda7e81440eb0","project":"openstack/devstack","branch":"master","topic":"bug/1629133","hashtags":[],"change_id":"I48c86f94098f1501f0e7f90a265dda7e81440eb0","subject":"Reduce the scope of the subnet pool prefix in neutron","status":"MERGED","created":"2016-10-04 16:19:35.000000000","updated":"2016-10-14 20:04:27.000000000","submitted":"2016-10-14 20:04:27.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"6280b2b93c50d9f37e2d349baf6b801a40e32713","_number":381965,"virtual_id_number":381965,"owner":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":2,"date":"2016-10-14 20:04:27.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"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":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"all":[{"value":2,"date":"2016-10-14 18:40:57.000000000","_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},{"value":1,"date":"2016-10-14 17:14:55.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":1,"date":"2016-10-14 17:08:39.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"value":2,"date":"2016-10-14 18:43:38.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":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2016-10-14 15:46:17.000000000","_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"value":1,"date":"2016-10-14 18:43:38.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":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"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":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-10-14 17:08:39.000000000","updated_by":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"reviewer":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"state":"REVIEWER"},{"updated":"2016-10-14 17:14:55.000000000","updated_by":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"reviewer":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"state":"REVIEWER"},{"updated":"2016-10-14 18:40:57.000000000","updated_by":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"reviewer":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"state":"REVIEWER"},{"updated":"2016-10-14 18:43:38.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":"2016-10-14 20:04:27.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"5a4bf55a5d8e566f138d85eca45a3956759bbbab","author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"date":"2016-10-04 16:19:35.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c6ed698282c33f8a8f55b0fb80b5f55e9197cc4c","author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"date":"2016-10-04 16:31:11.000000000","message":"Patch Set 1: Workflow-1\n\nOther solutions are being looked at.","accounts_in_message":[],"_revision_number":1},{"id":"20e159936fd48dbedf7ad79644558a3b89735941","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-10-04 17:53:39.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/65/381965/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/b0be685/ : SUCCESS in 1h 09m 41s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/65/381965/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/d4842da/ : SUCCESS in 51m 29s\n- gate-tempest-dsvm-full-ubuntu-xenial http://logs.openstack.org/65/381965/1/check/gate-tempest-dsvm-full-ubuntu-xenial/2d2e3da/ : SUCCESS in 59m 16s\n- gate-grenade-dsvm-ubuntu-xenial http://logs.openstack.org/65/381965/1/check/gate-grenade-dsvm-ubuntu-xenial/93a32d9/ : SUCCESS in 44m 41s\n- gate-devstack-docs-ubuntu-xenial http://docs-draft.openstack.org/65/381965/1/check/gate-devstack-docs-ubuntu-xenial/eb43d03//doc/build/html/ : SUCCESS in 2m 57s\n- gate-devstack-bashate http://logs.openstack.org/65/381965/1/check/gate-devstack-bashate/57fc6ea/ : SUCCESS in 2m 30s\n- gate-devstack-unit-tests http://logs.openstack.org/65/381965/1/check/gate-devstack-unit-tests/f41a808/ : SUCCESS in 2m 47s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-nv http://logs.openstack.org/65/381965/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-nv/04f6d10/ : SUCCESS in 24m 42s (non-voting)\n- gate-devstack-dsvm-updown http://logs.openstack.org/65/381965/1/check/gate-devstack-dsvm-updown/caf518a/ : SUCCESS in 31m 09s\n- gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv http://logs.openstack.org/65/381965/1/check/gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv/c06ebee/ : SUCCESS in 1h 12m 15s (non-voting)\n- gate-swift-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/65/381965/1/check/gate-swift-dsvm-functional-ubuntu-xenial-nv/d359ca3/ : SUCCESS in 24m 45s (non-voting)\n- gate-tempest-dsvm-multinode-full-ubuntu-xenial http://logs.openstack.org/65/381965/1/check/gate-tempest-dsvm-multinode-full-ubuntu-xenial/7d39028/ : FAILURE in 1h 14m 44s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial http://logs.openstack.org/65/381965/1/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial/4494107/ : SUCCESS in 52m 24s (non-voting)\n- gate-grenade-dsvm-multinode http://logs.openstack.org/65/381965/1/check/gate-grenade-dsvm-multinode/8800ff5/ : SUCCESS in 29m 13s\n- gate-grenade-dsvm-neutron-multinode http://logs.openstack.org/65/381965/1/check/gate-grenade-dsvm-neutron-multinode/968c425/ : SUCCESS in 35m 00s\n- gate-tempest-dsvm-platform-centos7-nv http://logs.openstack.org/65/381965/1/check/gate-tempest-dsvm-platform-centos7-nv/4ca7ba0/ : SUCCESS in 1h 13m 36s (non-voting)\n- gate-tempest-dsvm-platform-fedora24-nv http://logs.openstack.org/65/381965/1/check/gate-tempest-dsvm-platform-fedora24-nv/274b60e/ : SUCCESS in 1h 28m 42s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"785575a467dd4f37ae65f2814802e509a9de2402","author":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"date":"2016-10-04 19:07:20.000000000","message":"Patch Set 1:\n\nBuild succeeded (check pipeline).\n\n- tempest-dsvm-ovsdpdk-nfv-networking-xenial http://intel-openstack-ci-logs.ovh/65/381965/1/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/084268b : SUCCESS in 58m 40s\n- tempest-dsvm-ovsdpdk-nfv-networking-centos http://intel-openstack-ci-logs.ovh/65/381965/1/check/tempest-dsvm-ovsdpdk-nfv-networking-centos/b813be1 : FAILURE in 1h 05m 52s (non-voting)\n- tempest-dsvm-full-nfv-xenial http://intel-openstack-ci-logs.ovh/65/381965/1/check/tempest-dsvm-full-nfv-xenial/8d6bf0a : SUCCESS in 1h 56m 47s","accounts_in_message":[],"_revision_number":1},{"id":"67143559d90d67fef988bd808bae40c2aca3ea1d","author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"date":"2016-10-14 15:46:17.000000000","message":"Patch Set 1: -Workflow\n\nSince last comment in bug, I\u0027m re-proposing the patch here.\n\nhttps://bugs.launchpad.net/devstack/+bug/1629133/comments/18","accounts_in_message":[],"_revision_number":1},{"id":"a02f2d5ad9ea6753dfab9c5821570b1c038bdc89","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-10-14 17:08:39.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"e9357c858e8bbd3bae6ade32af36d6f1906bc6f9","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2016-10-14 17:14:55.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"63147191378b1a8d0cf499d7b8e7f3a588a54ec9","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-10-14 17:15:24.000000000","message":"Patch Set 1:\n\nPersonally, if I had to propose a fix, on top of this, I would also make the logic introduced by [1] conditional and disabled by default (i.e. reverting to the original and intended behavior when I initially introduced subnetpools into devstack). At the same time I would get rid of toggle [2], which IMO makes no sense. Adding some docs as Clark suggested would also be a good idea. Finally I would make this dependent on whatever project-config change is necessary to enable the newly introduced logic [1] to make Trove happy.\n\n[1] https://review.openstack.org/#/c/356026/\n[2] https://review.openstack.org/#/c/378063/","accounts_in_message":[],"_revision_number":1},{"id":"6099c69eeb3cd66a200fb6fa58e053262a0f0ee2","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-10-14 17:16:08.000000000","message":"Patch Set 1:\n\nis that asking too much? :)","accounts_in_message":[],"_revision_number":1},{"id":"a1d5fe2127ffae7b90df613cd6ff5af404db4818","author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"date":"2016-10-14 18:40:57.000000000","message":"Patch Set 1: Code-Review+2\n\nI\u0027m OK with this, and doing follow up patches that @armax suggests","accounts_in_message":[],"_revision_number":1},{"id":"442313c3ccef6a7e6d97fae805c8c9c27f1005a8","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2016-10-14 18:43:38.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"9686e428a5cd3ff45d0e9594c51e25dbd423a222","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-10-14 18:43:48.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"8a9f9b0d76219b0deeaa4c407a86e806fb4afab5","date":"2016-10-14 20:04:27.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":1},{"id":"43f9f2ed8b2e60ce85e4a151e821061e0fbfa8d1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-10-14 20:04:27.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/65/381965/1/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/13d62fa/ : SUCCESS in 43m 32s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/65/381965/1/gate/gate-grenade-dsvm-neutron-ubuntu-xenial/db4b8b2/ : SUCCESS in 30m 32s\n- gate-tempest-dsvm-full-ubuntu-xenial http://logs.openstack.org/65/381965/1/gate/gate-tempest-dsvm-full-ubuntu-xenial/c38d62e/ : SUCCESS in 35m 37s\n- gate-grenade-dsvm-ubuntu-xenial http://logs.openstack.org/65/381965/1/gate/gate-grenade-dsvm-ubuntu-xenial/5fa5aa2/ : SUCCESS in 25m 26s\n- gate-devstack-docs-ubuntu-xenial http://docs-draft.openstack.org/65/381965/1/gate/gate-devstack-docs-ubuntu-xenial/2c29333//doc/build/html/ : SUCCESS in 4m 29s\n- gate-devstack-bashate http://logs.openstack.org/65/381965/1/gate/gate-devstack-bashate/27cf102/ : SUCCESS in 2m 23s\n- gate-grenade-dsvm-multinode http://logs.openstack.org/65/381965/1/gate/gate-grenade-dsvm-multinode/619ceef/ : SUCCESS in 28m 34s\n- gate-grenade-dsvm-neutron-multinode http://logs.openstack.org/65/381965/1/gate/gate-grenade-dsvm-neutron-multinode/1cf23a8/ : SUCCESS in 34m 17s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"ca89d071b3c249fba55a824f7f4fc247b7c22948","revisions":{"ca89d071b3c249fba55a824f7f4fc247b7c22948":{"kind":"REWORK","_number":1,"created":"2016-10-04 16:19:35.000000000","uploader":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"ref":"refs/changes/65/381965/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/65/381965/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/65/381965/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/65/381965/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/65/381965/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/65/381965/1"}}},"commit":{"parents":[{"commit":"563d3777823fa4ee113ac930c4066c3f8d5f8373","subject":"Merge \"Fix typo in documentation\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/563d3777823fa4ee113ac930c4066c3f8d5f8373"}]}],"author":{"name":"Miguel Angel Ajo","email":"majopela@redhat.com","date":"2016-10-04 16:17:44.000000000","tz":120},"committer":{"name":"Miguel Angel Ajo","email":"majopela@redhat.com","date":"2016-10-04 16:19:33.000000000","tz":120},"subject":"Reduce the scope of the subnet pool prefix in neutron","message":"Reduce the scope of the subnet pool prefix in neutron\n\nSome of the clouds used for CI use the 10.2xx.0.0/16 range\nfor VMs, and collide with the wider 10.0.0.0/8.\n\nThis setting allows for creation of 256 subnets out of the pool.\n\nChange-Id: I48c86f94098f1501f0e7f90a265dda7e81440eb0\nCloses-Bug: 1629133\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ca89d071b3c249fba55a824f7f4fc247b7c22948"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ca89d071b3c249fba55a824f7f4fc247b7c22948"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}