)]}'
{"id":"openstack%2Fdevstack~204625","triplet_id":"openstack%2Fdevstack~master~Ie8f018806bd3e79dfad22f1d9e45781bc0074cf6","project":"openstack/devstack","branch":"master","topic":"neutron_services_refactor","hashtags":[],"change_id":"Ie8f018806bd3e79dfad22f1d9e45781bc0074cf6","subject":"Move create_neutron_initial_network into l3 service file","status":"ABANDONED","created":"2015-07-22 14:00:36.000000000","updated":"2015-10-29 05:14:48.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f467959a02300fb07cbec1b2c1a93e6bba0e394b","_number":204625,"virtual_id_number":204625,"owner":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2015-07-22 14:14:49.000000000","_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"date":"2015-07-22 15:12:05.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":-1,"date":"2015-07-28 09:08:16.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-1,"date":"2015-07-22 14:11:25.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"}],"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":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-07-22 14:11:25.000000000","updated_by":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"reviewer":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"state":"REVIEWER"},{"updated":"2015-07-22 15:12:05.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2015-07-28 09:08:16.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"6a0d3f158730d1b393460244fce3a9088b62c526","author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"date":"2015-07-22 14:00:36.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"1a50d97468e1d490e1e3ece2209ab4317aea94b9","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-07-22 14:11:25.000000000","message":"Patch Set 1: Code-Review-1\n\nWhy do you move \"network\" creation into L3?\n\nYour patches make neutron-legacy slim but make L3 fat only .","accounts_in_message":[],"_revision_number":1},{"id":"6199b8e44580b997906b8145ce3e08b4fe633b4f","author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"date":"2015-07-22 14:14:49.000000000","message":"Patch Set 1:\n\n@Hirofumi\n\nThis work is inspired by https://review.openstack.org/#/c/168438/ - where we try to slim down neutron-legacy in prep for a full refactor","accounts_in_message":[],"_revision_number":1},{"id":"6619388bbde351727f55f1122e2ed2b73261802f","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-07-22 14:25:33.000000000","message":"Patch Set 1:\n\nI understand. It\u0027s good try.\n\nHowever, we should avoid moving them into the place that is not reasonable even if this patch is just preparation.","accounts_in_message":[],"_revision_number":1},{"id":"97073d0a45f4a48134765ce359df19ef1bcff488","author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"date":"2015-07-22 14:54:44.000000000","message":"Patch Set 1:\n\nWhere would be reasonable to move these pieces?","accounts_in_message":[],"_revision_number":1},{"id":"774783c56f97bb3f878659ff82903201e76ddd82","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-07-22 15:12:05.000000000","message":"Patch Set 1:\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/25/204625/1/5267/index.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":1},{"id":"a0748a02da8a5f424489162ec0c6e1b8a0470c7e","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-07-22 23:59:37.000000000","message":"Patch Set 1:\n\nI would choose remaining neutron-legacy or creating new script file that creates network resources if I will do. I think at least l3 service file is not reasonable because create_neutron_initial_network can be executed without q-l3.","accounts_in_message":[],"_revision_number":1},{"id":"6870b8f592d8959d71ec7605c976b1b8a0479b49","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-23 01:41:06.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/25/204625/1/check/gate-tempest-dsvm-full/4045440/ : SUCCESS in 41m 16s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/25/204625/1/check/gate-tempest-dsvm-postgres-full/a5bd065/ : SUCCESS in 51m 31s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/25/204625/1/check/gate-tempest-dsvm-neutron-full/4b27076/ : SUCCESS in 1h 08m 06s\n- gate-grenade-dsvm http://logs.openstack.org/25/204625/1/check/gate-grenade-dsvm/a05e5f8/ : SUCCESS in 45m 45s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/25/204625/1/check/gate-tempest-dsvm-large-ops/20826cf/ : SUCCESS in 43m 27s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/25/204625/1/check/gate-tempest-dsvm-neutron-large-ops/f0b1c04/ : SUCCESS in 43m 25s\n- gate-devstack-bashate http://logs.openstack.org/25/204625/1/check/gate-devstack-bashate/d80e854/ : SUCCESS in 30s\n- gate-devstack-unit-tests http://logs.openstack.org/25/204625/1/check/gate-devstack-unit-tests/dac66c0/ : SUCCESS in 24s\n- gate-devstack-dsvm-cells http://logs.openstack.org/25/204625/1/check/gate-devstack-dsvm-cells/cfa37b1/ : SUCCESS in 30m 46s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/25/204625/1/check/gate-grenade-dsvm-partial-ncpu/c207ca3/ : SUCCESS in 42m 06s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/25/204625/1/check/gate-tempest-dsvm-ironic-pxe_ssh/e1e5982/ : SUCCESS in 37m 40s\n- gate-devstack-dsvm-updown http://logs.openstack.org/25/204625/1/check/gate-devstack-dsvm-updown/7b0d761/ : SUCCESS in 28m 35s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/25/204625/1/check/gate-tempest-dsvm-f21/80c0416/ : FAILURE in 52m 02s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/25/204625/1/check/gate-tempest-dsvm-centos7/4986952/ : SUCCESS in 31m 29s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/25/204625/1/check/gate-devstack-publish-docs/dc3c934//doc/build/html/ : SUCCESS in 59s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/25/204625/1/check/gate-swift-dsvm-functional-nv/6d8d7d3/ : SUCCESS in 27m 57s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/25/204625/1/check/gate-grenade-dsvm-neutron/bfadc26/ : SUCCESS in 55m 32s\n- gate-tempest-dsvm-multinode-smoke http://logs.openstack.org/25/204625/1/check/gate-tempest-dsvm-multinode-smoke/1c7ee31/ : SUCCESS in 32m 51s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-smoke http://logs.openstack.org/25/204625/1/check/gate-tempest-dsvm-neutron-multinode-smoke/71ae7ef/ : SUCCESS in 38m 37s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"a8a66db09f62ef4e3f7d1c098e3406901a13dc24","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-28 09:08:16.000000000","message":"Patch Set 1: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"59bea330484c2c84042fd1b15dd74b4813d7cba2","author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"date":"2015-10-29 05:14:48.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"b76c2015955dca81466316e4148a983c90990620","revisions":{"b76c2015955dca81466316e4148a983c90990620":{"kind":"REWORK","_number":1,"created":"2015-07-22 14:00:36.000000000","uploader":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"ref":"refs/changes/25/204625/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/25/204625/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/25/204625/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/25/204625/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/25/204625/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/25/204625/1"}}},"commit":{"parents":[{"commit":"173c7ba11982dba34f85f9b5a1385af135f1b12c","subject":"Neutron: move subnet creations into the l3 service file","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/173c7ba11982dba34f85f9b5a1385af135f1b12c"}]}],"author":{"name":"Sean M. Collins","email":"sean@coreitpro.com","date":"2015-07-22 13:34:34.000000000","tz":-300},"committer":{"name":"Sean M. Collins","email":"sean@coreitpro.com","date":"2015-07-22 13:34:34.000000000","tz":-300},"subject":"Move create_neutron_initial_network into l3 service file","message":"Move create_neutron_initial_network into l3 service file\n\nChange-Id: Ie8f018806bd3e79dfad22f1d9e45781bc0074cf6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b76c2015955dca81466316e4148a983c90990620"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b76c2015955dca81466316e4148a983c90990620"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}