)]}'
{"id":"openstack%2Fdevstack~398012","triplet_id":"openstack%2Fdevstack~master~I7b32804d47bec743c0b13e434e6a7958728896ea","project":"openstack/devstack","branch":"master","topic":"bug/1629133","hashtags":[],"change_id":"I7b32804d47bec743c0b13e434e6a7958728896ea","subject":"Derive IP ranges from new ADDRS_SAFE_TO_USE vars","status":"MERGED","created":"2016-11-16 01:27:29.000000000","updated":"2016-11-18 18:01:22.000000000","submitted":"2016-11-18 18:01:22.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":8,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"c8b5a41f02f6da4b8da63625d3c0ee42d77ca7a7","_number":398012,"virtual_id_number":398012,"owner":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"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":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":2,"date":"2016-11-18 18:01:22.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},{"value":0,"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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-11-18 17:08:20.000000000","_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":1,"date":"2016-11-16 12:42:27.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":1,"date":"2016-11-16 22:23:15.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2016-11-16 10:29:16.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},{"value":0,"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"value":1,"date":"2016-11-16 15:44:30.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":2,"date":"2016-11-16 23:30:42.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"all":[{"value":1,"date":"2016-11-18 17:08:20.000000000","_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},{"value":0,"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},{"value":0,"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-11-16 03:50:21.000000000","updated_by":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2016-11-16 10:29:16.000000000","updated_by":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"reviewer":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"state":"REVIEWER"},{"updated":"2016-11-16 12:42:27.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2016-11-16 15:44:30.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-11-16 22:23:15.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"REVIEWER"},{"updated":"2016-11-16 23:30:42.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"},{"updated":"2016-11-18 17:08:20.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-11-18 18:01:22.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"93de7764021ce9a89d5f2db22eec2ca66c6e7c28","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2016-11-16 01:27:29.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"0c931f011c3a0a801cb572925e4f3066d725a1c2","author":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"date":"2016-11-16 03:50:21.000000000","message":"Patch Set 1:\n\nBuild succeeded (check pipeline).\n\n- tempest-dsvm-ovsdpdk-nfv-networking-xenial http://intel-openstack-ci-logs.ovh/12/398012/1/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/80e5e02 : SUCCESS in 54m 57s\n- tempest-dsvm-ovsdpdk-nfv-networking-centos http://intel-openstack-ci-logs.ovh/12/398012/1/check/tempest-dsvm-ovsdpdk-nfv-networking-centos/e0fe494 : SUCCESS in 1h 10m 23s (non-voting)\n- tempest-dsvm-full-nfv-xenial http://intel-openstack-ci-logs.ovh/12/398012/1/check/tempest-dsvm-full-nfv-xenial/1fdb4d8 : SUCCESS in 2h 22m 35s","accounts_in_message":[],"_revision_number":1},{"id":"015b5c601e2d5879f02bb74a0bbfc5ad6c2097e3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-16 04:12:24.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/12/398012/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0b21398/ : SUCCESS in 1h 02m 51s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/12/398012/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/d8c3e89/ : SUCCESS in 33m 42s\n- gate-grenade-dsvm-ubuntu-xenial http://logs.openstack.org/12/398012/1/check/gate-grenade-dsvm-ubuntu-xenial/9987180/ : SUCCESS in 27m 42s\n- gate-devstack-docs-ubuntu-xenial http://logs.openstack.org/12/398012/1/check/gate-devstack-docs-ubuntu-xenial/5edd63b/ : FAILURE in 1m 35s\n- gate-devstack-bashate http://logs.openstack.org/12/398012/1/check/gate-devstack-bashate/74cfc7e/ : SUCCESS in 1m 24s\n- gate-devstack-unit-tests http://logs.openstack.org/12/398012/1/check/gate-devstack-unit-tests/43209f7/ : SUCCESS in 1m 37s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv http://logs.openstack.org/12/398012/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/f7e060f/ : SUCCESS in 26m 06s (non-voting)\n- gate-devstack-dsvm-updown http://logs.openstack.org/12/398012/1/check/gate-devstack-dsvm-updown/a2e3413/ : FAILURE in 56m 49s\n- gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv http://logs.openstack.org/12/398012/1/check/gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv/d0356c5/ : SUCCESS in 49m 41s (non-voting)\n- gate-swift-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/12/398012/1/check/gate-swift-dsvm-functional-ubuntu-xenial-nv/b092964/ : SUCCESS in 19m 17s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial http://logs.openstack.org/12/398012/1/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial/b195d12/ : SUCCESS in 1h 16m 26s (non-voting)\n- gate-grenade-dsvm-multinode http://logs.openstack.org/12/398012/1/check/gate-grenade-dsvm-multinode/eb7f8cd/ : SUCCESS in 29m 48s\n- gate-grenade-dsvm-neutron-multinode-ubuntu-xenial http://logs.openstack.org/12/398012/1/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/89f46d1/ : SUCCESS in 38m 14s\n- gate-tempest-dsvm-platform-centos7-nv http://logs.openstack.org/12/398012/1/check/gate-tempest-dsvm-platform-centos7-nv/2e3ef8a/ : FAILURE in 32m 05s (non-voting)\n- gate-tempest-dsvm-platform-fedora24-nv http://logs.openstack.org/12/398012/1/check/gate-tempest-dsvm-platform-fedora24-nv/729bdc2/ : FAILURE in 58m 39s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"e8d76be612d9a2a3e5491e7cd426b05ee69568da","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2016-11-16 05:27:04.000000000","message":"Patch Set 2: Published edit on patch set 1","accounts_in_message":[],"_revision_number":2},{"id":"9c8e6bbd5944aa8127b5adbe9a12f2ce70a7b13e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-16 06:51:42.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/12/398012/2/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/4308d38/ : SUCCESS in 1h 02m 01s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/12/398012/2/check/gate-grenade-dsvm-neutron-ubuntu-xenial/0e22fbf/ : SUCCESS in 58m 44s\n- gate-grenade-dsvm-ubuntu-xenial http://logs.openstack.org/12/398012/2/check/gate-grenade-dsvm-ubuntu-xenial/d8345e0/ : SUCCESS in 41m 48s\n- gate-devstack-docs-ubuntu-xenial http://docs-draft.openstack.org/12/398012/2/check/gate-devstack-docs-ubuntu-xenial/6607aac//doc/build/html/ : SUCCESS in 1m 29s\n- gate-devstack-bashate http://logs.openstack.org/12/398012/2/check/gate-devstack-bashate/b55ad44/ : SUCCESS in 1m 20s\n- gate-devstack-unit-tests http://logs.openstack.org/12/398012/2/check/gate-devstack-unit-tests/fa4f6ec/ : SUCCESS in 2m 42s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv http://logs.openstack.org/12/398012/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/9caa284/ : SUCCESS in 24m 33s (non-voting)\n- gate-devstack-dsvm-updown http://logs.openstack.org/12/398012/2/check/gate-devstack-dsvm-updown/b009612/ : SUCCESS in 30m 11s\n- gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv http://logs.openstack.org/12/398012/2/check/gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv/b86767a/ : SUCCESS in 48m 39s (non-voting)\n- gate-swift-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/12/398012/2/check/gate-swift-dsvm-functional-ubuntu-xenial-nv/cc3e477/ : SUCCESS in 19m 39s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial http://logs.openstack.org/12/398012/2/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial/a018b64/ : SUCCESS in 1h 10m 22s (non-voting)\n- gate-grenade-dsvm-multinode http://logs.openstack.org/12/398012/2/check/gate-grenade-dsvm-multinode/b34a404/ : SUCCESS in 28m 16s\n- gate-grenade-dsvm-neutron-multinode-ubuntu-xenial http://logs.openstack.org/12/398012/2/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/f930fec/ : SUCCESS in 37m 06s\n- gate-tempest-dsvm-platform-centos7-nv http://logs.openstack.org/12/398012/2/check/gate-tempest-dsvm-platform-centos7-nv/5f1102c/ : FAILURE in 30m 59s (non-voting)\n- gate-tempest-dsvm-platform-fedora24-nv http://logs.openstack.org/12/398012/2/check/gate-tempest-dsvm-platform-fedora24-nv/34bed5d/ : SUCCESS in 1h 23m 48s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"dbee3ee76df5f8d424ee110823698fb9e6ea774f","author":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"date":"2016-11-16 07:50:58.000000000","message":"Patch Set 2:\n\nBuild succeeded (check pipeline).\n\n- tempest-dsvm-ovsdpdk-nfv-networking-xenial http://intel-openstack-ci-logs.ovh/12/398012/2/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/fbbf1ab : SUCCESS in 57m 08s\n- tempest-dsvm-ovsdpdk-nfv-networking-centos http://intel-openstack-ci-logs.ovh/12/398012/2/check/tempest-dsvm-ovsdpdk-nfv-networking-centos/3c93f8b : SUCCESS in 1h 12m 02s (non-voting)\n- tempest-dsvm-full-nfv-xenial http://intel-openstack-ci-logs.ovh/12/398012/2/check/tempest-dsvm-full-nfv-xenial/b309588 : SUCCESS in 2h 23m 35s","accounts_in_message":[],"_revision_number":2},{"id":"f2039426cc348e93bb8243a9cb4306fa6b24cf16","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2016-11-16 08:17:50.000000000","message":"Patch Set 2:\n\nBuild succeeded.\n\n- dsvm-tempest-neutron-network http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/12/398012/2/check/dsvm-tempest-neutron-network/1570036 : SUCCESS in 2h 10m 22s","accounts_in_message":[],"_revision_number":2},{"id":"053d7e43fe1b6e87ca9148cf59c8627619a6e12c","author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"date":"2016-11-16 08:39:51.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"d8c34d75ea0ab68da0d8610d0b1ed8ed9468e29b","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2016-11-16 08:44:42.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"220ce3eef6ce212e13189da4439dc125607273b2","author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"date":"2016-11-16 09:22:17.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"e8e3cf95eece6cf952867a8770436f72924fd95e","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2016-11-16 09:27:00.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"89b69781ace83b65c02a9b057db9a8cfd13df674","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2016-11-16 09:37:37.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"2d3b56ca9440a9f9ae2ea8a93d3ca78cace8c20d","author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"date":"2016-11-16 10:29:16.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"aaa647b8a0abf555c4aa2b4c3f965ee9fc7f7d78","author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"date":"2016-11-16 12:42:27.000000000","message":"Patch Set 2: Code-Review+1\n\nI think this looks wonderful - thanks!","accounts_in_message":[],"_revision_number":2},{"id":"34812902970752f4bf49d4a8b5167e297c9b721f","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2016-11-16 15:44:30.000000000","message":"Patch Set 2: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"21ea6b00f1188989ad0e4e3a20fa9b19bda21153","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2016-11-16 22:23:15.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"833cd9dea7849bbf125c27e09de41e49c00ecb94","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2016-11-16 23:30:42.000000000","message":"Patch Set 2: Code-Review+2\n\n(1 comment)\n\nAfter a bit of digging, this seems sensible to me.  I say that because it\u0027s probably a better solution to bug #1628267 than the one we merged.\n\nI\u0027d appreciate it if the documentation got some help as mentioned inline.  I think the idea when subnetpools were added was that they were essentially transparent, but it seems that\u0027s not quite the case.  Basically, AIUI, it needs to be made clear that FIXED_RANGE is a deprecated nova-net thing, and that for neutron the safe-to-use addrs will be put into the subnetpool and networks allocated from there (by default).\n\nI also wonder if this could provide us reverting Ic89ceca76afda67da5545111972c3348011f294f ... to actually make it transparent again.  I think it is quite reasonable to say \"we use subnet pools\" to simplify all this rule and just work out the issues so everyone can use it \n\nI hope to see that as clarifying follow-ons, but I think this is probably headed in the right direction.","accounts_in_message":[],"_revision_number":2},{"id":"1de366cb448f4bdcc5fb765e8751d9151c0d3a09","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2016-11-17 00:06:38.000000000","message":"Patch Set 2:\n\n(1 comment)\n\nYeah, I think having the discussion about eliminating non-subnetpool automatic private networks in devstack is reasonable. If someone wants a private network not using the default subnetpool they could always add one.\n\nReally the thing that\u0027s screwed up is the automatic routing we are doing for private v4 networks in devstack. This isn\u0027t how a normal neutron deployment works and I\u0027d like to eliminate it here: https://review.openstack.org/#/c/397987/\n\nOnce that is gone, we can care a whole lot less about what is used in a private network because it will no longer conflict with the host.","accounts_in_message":[],"_revision_number":2},{"id":"73d6ea5ba399aa74b496d9c9b8e686bea0196783","author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"date":"2016-11-18 17:08:20.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"c6c79245204c80b3497a3b136ac0f65e257cb392","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-18 17:08:31.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"ebfd073c2ac65ff12ea1eef006d26203e537c80a","date":"2016-11-18 18:01:22.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2},{"id":"658fa06c2325d9330736e65e749c83ffdbffab2c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-18 18:01:22.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/12/398012/2/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/5428783/ : SUCCESS in 51m 43s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/12/398012/2/gate/gate-grenade-dsvm-neutron-ubuntu-xenial/56fc8f4/ : SUCCESS in 32m 00s\n- gate-grenade-dsvm-ubuntu-xenial http://logs.openstack.org/12/398012/2/gate/gate-grenade-dsvm-ubuntu-xenial/60d3450/ : SUCCESS in 41m 51s\n- gate-devstack-docs-ubuntu-xenial http://docs-draft.openstack.org/12/398012/2/gate/gate-devstack-docs-ubuntu-xenial/839c478//doc/build/html/ : SUCCESS in 1m 43s\n- gate-devstack-bashate http://logs.openstack.org/12/398012/2/gate/gate-devstack-bashate/8083b81/ : SUCCESS in 1m 29s\n- gate-grenade-dsvm-neutron-multinode-ubuntu-xenial http://logs.openstack.org/12/398012/2/gate/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/0527194/ : SUCCESS in 36m 19s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"4bfbc291eefd92d8b7885f36275b7ff541e067ab","revisions":{"ffa8cfe3e3253121c924cfdc8919ce0f3c3e7a37":{"kind":"REWORK","_number":1,"created":"2016-11-16 01:27:29.000000000","uploader":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"ref":"refs/changes/12/398012/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/12/398012/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/398012/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/398012/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/398012/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/12/398012/1"}}},"commit":{"parents":[{"commit":"7976f31fb956dcb1cbeea92906f5609f0b43acc4","subject":"Merge \"Unset admin_project config options for Keystone\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/7976f31fb956dcb1cbeea92906f5609f0b43acc4"}]}],"author":{"name":"Kevin Benton","email":"kevin@benton.pub","date":"2016-11-16 01:26:05.000000000","tz":-480},"committer":{"name":"Kevin Benton","email":"kevin@benton.pub","date":"2016-11-16 01:26:57.000000000","tz":-480},"subject":"Derive IP ranges from new ADDRS_SAFE_TO_USE vars","message":"Derive IP ranges from new ADDRS_SAFE_TO_USE vars\n\nThe switch to using subnetpools caused quite a bit of confusion\nbecause it didn\u0027t respect the value of FIXED_RANGE. This caused\nconflicts in the gate with it\u0027s default IPv4 value of 10.0.0.0/8.\n\nThis patch does a few things to address the issue:\n* It introduces the IPV4_ADDRS_SAFE_TO_USE and IPV6_ADDRS_SAFE_TO_USE\n  values and adjusts all of the FIXED_RANGE and SUBNETPOOL_PREFIX values\n  to dervive from them by default.\n  * This addresses the concern that was raised about implying that\n    SUBNETPOOL_PREFIX and FIXED_RANGE are equivalent when setting\n    SUBNETPOOL_PREFIX\u003dFIXED_RANGE by default. Now we have a new value\n    for the operator specify a chunk of addresses that are safe to\n    use for private networks without implementation implications.\n  * Backwards compatibility is maintained by alloing users to override\n    override all of these values.\n* The default for IPV4_ADDRS_SAFE_TO_USE uses /22 instead of /24\n  * Because we want to be able to use subnetpools for auto allocated\n    topologies and we want to be able to have a large chunk of\n    instances on each network, we needed a little more breathing room\n    in the default v4 network size.\n* SUBNET_POOL_SIZE_V4 default is changed from 24 to 26\n  * In conjuction with this change and the one above, the default\n    subnetpool will support up to 16 64-address allocations.\n  * This should be enough to cover any regular gate scenarios.\n  * If someone wants a bigger/smaller subnet, they can ask for that\n    in the API request, change this value themselves, or use a different\n    network entirely.\n* FIXED_RANGE_V6 defaults to a max prefix of /64 from IPV6_ADDRS_SAFE_TO_USE\n  * This avoids the private subnet in the non-subnetpool case from being\n    larger than /64 to avoid issues identified in rfc 7421.\n  * Users can still explicitly set this value to whatever they want.\n    This \u0027max\u0027 behavior is only for the default.\n  * This allows IPV6_ADDRS_SAFE_TO_USE to default to a /56, which leaves\n    tons of room for v6 subnetpools.\n\nCloses-Bug: #1629133\nChange-Id: I7b32804d47bec743c0b13e434e6a7958728896ea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ffa8cfe3e3253121c924cfdc8919ce0f3c3e7a37"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ffa8cfe3e3253121c924cfdc8919ce0f3c3e7a37"}]},"branch":"refs/heads/master"},"4bfbc291eefd92d8b7885f36275b7ff541e067ab":{"kind":"REWORK","_number":2,"created":"2016-11-16 05:27:04.000000000","uploader":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"ref":"refs/changes/12/398012/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/12/398012/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/398012/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/398012/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/398012/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/12/398012/2"}}},"commit":{"parents":[{"commit":"7976f31fb956dcb1cbeea92906f5609f0b43acc4","subject":"Merge \"Unset admin_project config options for Keystone\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/7976f31fb956dcb1cbeea92906f5609f0b43acc4"}]}],"author":{"name":"Kevin Benton","email":"kevin@benton.pub","date":"2016-11-16 01:26:05.000000000","tz":-480},"committer":{"name":"Kevin Benton","email":"kevin@benton.pub","date":"2016-11-16 05:26:03.000000000","tz":0},"subject":"Derive IP ranges from new ADDRS_SAFE_TO_USE vars","message":"Derive IP ranges from new ADDRS_SAFE_TO_USE vars\n\nThe switch to using subnetpools caused quite a bit of confusion\nbecause it didn\u0027t respect the value of FIXED_RANGE. This caused\nconflicts in the gate with it\u0027s default IPv4 value of 10.0.0.0/8.\n\nThis patch does a few things to address the issue:\n* It introduces the IPV4_ADDRS_SAFE_TO_USE and IPV6_ADDRS_SAFE_TO_USE\n  values and adjusts all of the FIXED_RANGE and SUBNETPOOL_PREFIX values\n  to dervive from them by default.\n  * This addresses the concern that was raised about implying that\n    SUBNETPOOL_PREFIX and FIXED_RANGE are equivalent when setting\n    SUBNETPOOL_PREFIX\u003dFIXED_RANGE by default. Now we have a new value\n    for the operator specify a chunk of addresses that are safe to\n    use for private networks without implementation implications.\n  * Backwards compatibility is maintained by alloing users to override\n    override all of these values.\n* The default for IPV4_ADDRS_SAFE_TO_USE uses /22 instead of /24\n  * Because we want to be able to use subnetpools for auto allocated\n    topologies and we want to be able to have a large chunk of\n    instances on each network, we needed a little more breathing room\n    in the default v4 network size.\n* SUBNET_POOL_SIZE_V4 default is changed from 24 to 26\n  * In conjuction with this change and the one above, the default\n    subnetpool will support up to 16 64-address allocations.\n  * This should be enough to cover any regular gate scenarios.\n  * If someone wants a bigger/smaller subnet, they can ask for that\n    in the API request, change this value themselves, or use a different\n    network entirely.\n* FIXED_RANGE_V6 defaults to a max prefix of /64 from IPV6_ADDRS_SAFE_TO_USE\n  * This avoids the private subnet in the non-subnetpool case from being\n    larger than /64 to avoid issues identified in rfc 7421.\n  * Users can still explicitly set this value to whatever they want.\n    This \u0027max\u0027 behavior is only for the default.\n  * This allows IPV6_ADDRS_SAFE_TO_USE to default to a /56, which leaves\n    tons of room for v6 subnetpools.\n\nCloses-Bug: #1629133\nChange-Id: I7b32804d47bec743c0b13e434e6a7958728896ea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4bfbc291eefd92d8b7885f36275b7ff541e067ab"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4bfbc291eefd92d8b7885f36275b7ff541e067ab"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}