)]}'
{"projects/60_nova/from-rocky/upgrade-nova":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"482d923f26b6b193b99fe454535445ebe13fa899","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    # Copy the placement-related table data from the nova_api database"},{"line_number":84,"context_line":"    # into the placement database."},{"line_number":85,"context_line":"    ${db_script} --migrate ${rc_file}"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    # Copy any policy overrides from nova. We assume yaml here since that is"},{"line_number":88,"context_line":"    # the default in code, but it could also be a json file."}],"source_content_type":"application/octet-stream","patch_set":4,"id":"3f79a3b5_58b5c4ee","line":85,"updated":"2018-12-11 17:06:06.000000000","message":"It\u0027s probably worth adding a comment either here or above on L44 that order matters here so that the DB migration script can \"stamp\" the version of the placement database which relies on the placement.conf have [placement_database]/connection set first.\n\nDocumenting this script is going to be very important since other deployment projects are basing their tooling on the flow here.","commit_id":"b7b6b1294e22290beb476b458716de77df17f07a"},{"author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"change_message_id":"baa32317fd08d133d0fb9893fc825786d9073241","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    # Copy the placement-related table data from the nova_api database"},{"line_number":84,"context_line":"    # into the placement database."},{"line_number":85,"context_line":"    ${db_script} --migrate ${rc_file}"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    # Copy any policy overrides from nova. We assume yaml here since that is"},{"line_number":88,"context_line":"    # the default in code, but it could also be a json file."}],"source_content_type":"application/octet-stream","patch_set":4,"id":"3f79a3b5_b8e14ad1","line":85,"in_reply_to":"3f79a3b5_58b5c4ee","updated":"2018-12-13 12:18:06.000000000","message":"\u003e It\u0027s probably worth adding a comment either here or above on L44\n \u003e that order matters\n\nGood call. Since both \"sync\" and \"stamp\" commands in the script will rely on the placement.conf, I think what to write here is *upgrading* the DB schema relies on the configuration. Done.\n\n \u003e Documenting this script is going to be very important since other\n \u003e deployment projects are basing their tooling on the flow here.\n\nGreat to know. I\u0027ll keep it in my mind.","commit_id":"b7b6b1294e22290beb476b458716de77df17f07a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4f0ed86f985d47b92234836ad014e2fa77d63b96","unresolved":false,"context_lines":[{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    # Copy the placement-related table data from the nova_api database into"},{"line_number":85,"context_line":"    # the placement database and \"stamp\" the database version to bring it"},{"line_number":86,"context_line":"    # under alembic control. Note that that stamping relies on the"},{"line_number":87,"context_line":"    # `placement.conf` have `[placement_database]/connection` set first."},{"line_number":88,"context_line":"    ${db_script} --migrate ${rc_file}"},{"line_number":89,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":6,"id":"3f79a3b5_7a5dad51","line":86,"range":{"start_line":86,"start_character":39,"end_line":86,"end_character":43},"updated":"2018-12-17 14:23:43.000000000","message":"s/that//?","commit_id":"ad2047fc52078e7e7fe4d6571275807341d0a12b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4f0ed86f985d47b92234836ad014e2fa77d63b96","unresolved":false,"context_lines":[{"line_number":84,"context_line":"    # Copy the placement-related table data from the nova_api database into"},{"line_number":85,"context_line":"    # the placement database and \"stamp\" the database version to bring it"},{"line_number":86,"context_line":"    # under alembic control. Note that that stamping relies on the"},{"line_number":87,"context_line":"    # `placement.conf` have `[placement_database]/connection` set first."},{"line_number":88,"context_line":"    ${db_script} --migrate ${rc_file}"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    # Copy any policy overrides from nova. We assume yaml here since that is"}],"source_content_type":"application/octet-stream","patch_set":6,"id":"3f79a3b5_3a673526","line":87,"range":{"start_line":87,"start_character":23,"end_line":87,"end_character":27},"updated":"2018-12-17 14:23:43.000000000","message":"having","commit_id":"ad2047fc52078e7e7fe4d6571275807341d0a12b"}]}