)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"ba0774a1a1ecd9bdd61afee31fb46f7271c5e97c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"While upgrading from Queens to Rocky, if the db sync is done before"},{"line_number":10,"context_line":"the api_db sync it will result in an InternalError because of a"},{"line_number":11,"context_line":"schema change in the nova_api.cell_mappings table. So the nova_api"},{"line_number":12,"context_line":"has to be synced before nova_cell db."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ic790ef7c3531c2b672621310524797548246b2ef"},{"line_number":15,"context_line":"Closes-Bug: #1761775"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf659307_a9fedfab","line":12,"updated":"2018-04-06 15:35:37.000000000","message":"I would change this to \"nova api db should be sync\u0027d before the main db, since the former has information about the latter in it\" and then \"meh, we found this in rocky because we changed CellMapping\". And also something about \"this was mostly hidden because we run after devstack, which was doing it in the right way\".\n\n..but wordsmithed up all nice like.","commit_id":"9688880d952dc3f492f4269e328d37c164c227a4"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"ae834e0ea73593add06628139bd3c3d79f8872a2","unresolved":false,"context_lines":[{"line_number":10,"context_line":"the api_db sync it will result in an InternalError because of a"},{"line_number":11,"context_line":"schema change in the nova_api.cell_mappings table. So the nova_api"},{"line_number":12,"context_line":"has to be synced before nova_cell db."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ic790ef7c3531c2b672621310524797548246b2ef"},{"line_number":15,"context_line":"Closes-Bug: #1761775"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf659307_89475bbd","line":13,"updated":"2018-04-06 15:44:06.000000000","message":"When we added the api_db sync to grenade back in Mitaka, the api database was optional and the command was just appended after the existing main database sync invocation. In reality, the api db should always be sync\u0027d first, as it has information in it about how to contact the main database, and thus must be initialized first in order to be useful.\n\nIn Rocky, we added several \"for all cells\" commands to nova-manage which need this db to be initialized first, as well as one data model change that depends on the sync having added a new column (in api_db sync) before it\u0027s used (in db sync).\n\nDevstack has been doing this in the proper order for quite some time, and the fact that grenade was doing it backwards was hidden by the fact that devstack had run at least once before we ever got to this part of grenade, which meant that the database was already initialized. The data model change was the first time where the actual upgrade order became meaningful.","commit_id":"9688880d952dc3f492f4269e328d37c164c227a4"}]}