)]}'
{"id":"openstack%2Fdevstack~40243","triplet_id":"openstack%2Fdevstack~master~Ia941c426a1563bcc4cb5eae64ea30c0bf7677220","project":"openstack/devstack","branch":"master","topic":"bug/1207402","hashtags":[],"change_id":"Ia941c426a1563bcc4cb5eae64ea30c0bf7677220","subject":"Run Neutron migrations after creating schema","status":"MERGED","created":"2013-08-05 15:56:57.000000000","updated":"2013-08-06 17:14:10.000000000","submitted":"2013-08-06 17:14:10.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"da8507d9cfde6e6cfeff573401b2c8335eee9afc","_number":40243,"virtual_id_number":40243,"owner":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-08-06 17:14:10.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":0,"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-08-06 09:41:35.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":1,"date":"2013-08-06 13:38:59.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":2,"date":"2013-08-06 16:17:46.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-08-06 13:15:59.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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":0,"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":1,"date":"2013-08-06 16:17:46.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-08-06 13:15:59.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":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":0,"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-08-06 13:15:59.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":"2013-08-06 13:38:59.000000000","updated_by":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"reviewer":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"state":"REVIEWER"},{"updated":"2013-08-06 16:17:46.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2013-08-06 17:14:10.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"0f64661e413828b9fb08fe4b2bdaf2847453cc8c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-05 16:51:42.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/43/40243/1/check/gate-tempest-devstack-vm-full/d226456 : SUCCESS in 54m 29s\n- http://logs.openstack.org/43/40243/1/check/gate-tempest-devstack-vm-cells-full/dcdb4dd : FAILURE in 11m 14s (non-voting)\n- http://logs.openstack.org/43/40243/1/check/gate-tempest-devstack-vm-testr-full/dd1e4c4 : FAILURE in 25m 47s (non-voting)\n- http://logs.openstack.org/43/40243/1/check/gate-tempest-devstack-vm-postgres-full/e19bbfe : SUCCESS in 44m 35s\n- http://logs.openstack.org/43/40243/1/check/gate-tempest-devstack-vm-neutron/30c7e0d : SUCCESS in 18m 49s (non-voting)\n- http://logs.openstack.org/43/40243/1/check/gate-grenade-devstack-vm/cd1b54b : SUCCESS in 23m 04s\n","accounts_in_message":[],"_revision_number":1},{"id":"5f10fa1060cb314651300295e5cf2f25b096a670","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-08-05 18:48:40.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nThe database initialization really needs to be in init_neutron() so the configure function can be called without wiping the database.  The call to ini_neutron is already in place in stack.sh.","accounts_in_message":[],"_revision_number":1},{"id":"9c6ac1500d525b64b7263de0baece8f2c9e822b3","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-08-05 19:09:48.000000000","message":"Patch Set 1:\n\nAgreed.\n\nI did that in configure because neutron apparently does everything there - and the code for creating the db was there.\n\nI will move the code in init_neutron.\n\nI hope it is ok to have a check for creating the db and performing the migration only if q-svc is enabled, as the database should not be created on hosts where only agents are deployed.","accounts_in_message":[],"_revision_number":1},{"id":"875abfc06a25a4acb54bde42986788294e028a20","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-08-05 19:39:06.000000000","message":"Patch Set 1: Work In Progress","accounts_in_message":[],"_revision_number":1},{"id":"8d0ac554cbeb0e8feb756211589d1f976ce75e36","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-08-05 20:22:33.000000000","message":"Patch Set 1:\n\nIt\u0027s OK to have a check in init_neutron(), but in general I\u0027d prefer to have it in stack.sh.  I think having the relationships between all of the services in one place is helpful, particularly where that relationship crosses project boundaries.","accounts_in_message":[],"_revision_number":1},{"id":"5fb520d1ba4157d535cfdf42648099b032592eac","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-08-06 09:40:16.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"ac80897b06f6020f8e9fb723b272519f4644930e","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-08-06 09:41:35.000000000","message":"Patch Set 2:\n\ndb creation and synchronisation has been moved to init_neutron.\nAs suggested, the check regarding whether init_neutron should be executed or not has been added into stack.sh.","accounts_in_message":[],"_revision_number":2},{"id":"2c5c9f7023d996e26f48a38eebf864b68320f2db","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-06 10:34:17.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/43/40243/2/check/gate-tempest-devstack-vm-full/66c91e6 : SUCCESS in 53m 55s\n- http://logs.openstack.org/43/40243/2/check/gate-tempest-devstack-vm-cells-full/6b9e565 : FAILURE in 11m 00s (non-voting)\n- http://logs.openstack.org/43/40243/2/check/gate-tempest-devstack-vm-testr-full/c12b140 : FAILURE in 25m 42s (non-voting)\n- http://logs.openstack.org/43/40243/2/check/gate-tempest-devstack-vm-postgres-full/deff73e : SUCCESS in 47m 05s\n- http://logs.openstack.org/43/40243/2/check/gate-tempest-devstack-vm-neutron/d547cad : SUCCESS in 21m 43s\n- http://logs.openstack.org/43/40243/2/check/gate-grenade-devstack-vm/8f7a7a0 : SUCCESS in 21m 48s\n","accounts_in_message":[],"_revision_number":2},{"id":"d0aa1a01bd3fcccb9026fd182b2033736106ed9d","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-08-06 13:15:59.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"e8f3dd1f0bfde2d9a4483d15bcf6242a8fdc00dc","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-08-06 13:38:59.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"18df00b0804449e0e6b5f1ead2b34981c486b728","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-08-06 16:17:46.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"39806db5627551457986e15ff254cbd0b6dabf6b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-06 16:17:47.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"6617afca8d0907684b2ea9e3db13bd10fd412449","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-06 17:14:10.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2},{"id":"8f4217951cedc2bda4fa56ea65a1a538413b0420","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-06 17:14:10.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/43/40243/2/gate/gate-tempest-devstack-vm-full/29a0d69 : SUCCESS in 43m 43s\n- http://logs.openstack.org/43/40243/2/gate/gate-tempest-devstack-vm-postgres-full/7d8a56a : SUCCESS in 56m 20s\n- http://logs.openstack.org/43/40243/2/gate/gate-tempest-devstack-vm-neutron/62e6d00 : SUCCESS in 17m 32s\n- http://logs.openstack.org/43/40243/2/gate/gate-grenade-devstack-vm/b20babd : SUCCESS in 21m 51s\n","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"dd64988ff4c0a174691cc7d45e3eaef8acdc4fc8","revisions":{"5b9fdd20fffa8b1b7b8106d561f8628ae7aebe7f":{"kind":"REWORK","_number":1,"created":"2013-08-05 15:56:57.000000000","uploader":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"ref":"refs/changes/43/40243/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/43/40243/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/43/40243/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/43/40243/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/43/40243/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/43/40243/1"}}},"commit":{"parents":[{"commit":"e024bee3edc44bc872862fac5b022a3922ed421c","subject":"Merge \"Update to PLUMgrid plugin configuration\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e024bee3edc44bc872862fac5b022a3922ed421c"}]}],"author":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-08-05 15:56:17.000000000","tz":-420},"committer":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-08-05 15:58:54.000000000","tz":-420},"subject":"Run Neutron migrations after creating schema","message":"Run Neutron migrations after creating schema\n\nPartial-Bug 1207402\n\nThis patch executes Neutron DB migrations (based on alembic)\nbefore starting the Neutron services, similar to what happens\nfor most of other openstack projects managed by devstack.\nThis will ensure devstack always creates correctly version-stamped\nNeutron databases.\n\nChange-Id: Ia941c426a1563bcc4cb5eae64ea30c0bf7677220\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5b9fdd20fffa8b1b7b8106d561f8628ae7aebe7f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5b9fdd20fffa8b1b7b8106d561f8628ae7aebe7f"}]},"branch":"refs/heads/master"},"dd64988ff4c0a174691cc7d45e3eaef8acdc4fc8":{"kind":"REWORK","_number":2,"created":"2013-08-06 09:40:16.000000000","uploader":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"ref":"refs/changes/43/40243/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/43/40243/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/43/40243/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/43/40243/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/43/40243/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/43/40243/2"}}},"commit":{"parents":[{"commit":"e024bee3edc44bc872862fac5b022a3922ed421c","subject":"Merge \"Update to PLUMgrid plugin configuration\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e024bee3edc44bc872862fac5b022a3922ed421c"}]}],"author":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-08-05 15:56:17.000000000","tz":-420},"committer":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-08-06 09:42:08.000000000","tz":-420},"subject":"Run Neutron migrations after creating schema","message":"Run Neutron migrations after creating schema\n\nPartial-Bug 1207402\n\nThis patch executes Neutron DB migrations (based on alembic)\nbefore starting the Neutron services, similar to what happens\nfor most of other openstack projects managed by devstack.\nThis will ensure devstack always creates correctly version-stamped\nNeutron databases.\n\nChange-Id: Ia941c426a1563bcc4cb5eae64ea30c0bf7677220\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/dd64988ff4c0a174691cc7d45e3eaef8acdc4fc8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/dd64988ff4c0a174691cc7d45e3eaef8acdc4fc8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}