)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"472ebffe669a669b0ec4d9c28013ae2b6a74ea3c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"a database name of nova_api_cell0 by default, intead of nova_cell0."},{"line_number":12,"context_line":"This was confusing because cell0 has \"main\" schema inside, not \"api\""},{"line_number":13,"context_line":"schema."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Depends-On: I4e7f6c5eaa068c98e5c4ef3feaee50d8e4f5d484"},{"line_number":16,"context_line":"Change-Id: I86797785f76c2c927a4db8fef72b8f8d986af6b9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a3c09a3_d4358646","line":14,"updated":"2017-01-15 18:59:55.000000000","message":"Closes-Bug: #1656673","commit_id":"4ee236ef2ead8945af386c8331ef7832e5185d9f"}],"nova/cmd/manage.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5c2a4861aa13b67fb5964b549f1f8a71503d50d7","unresolved":false,"context_lines":[{"line_number":1119,"context_line":"          metavar\u003d\u0027\u003cdatabase_connection\u003e\u0027,"},{"line_number":1120,"context_line":"          help\u003d\u0027The database connection url for cell0. \u0027"},{"line_number":1121,"context_line":"               \u0027This is optional. If not provided, a standard database \u0027"},{"line_number":1122,"context_line":"               \u0027connection will be used based on the API database connection \u0027"},{"line_number":1123,"context_line":"               \u0027from the Nova configuration.\u0027"},{"line_number":1124,"context_line":"         )"},{"line_number":1125,"context_line":"    def map_cell0(self, database_connection\u003dNone):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a3c09a3_da576f00","line":1122,"range":{"start_line":1122,"start_character":53,"end_line":1122,"end_character":65},"updated":"2017-01-15 21:42:50.000000000","message":"This needs updating.","commit_id":"833cc2e82b2bf8f6a128bcf485ab3023b8b8fff8"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"4c650f0d5ee35a2742f9244b67976f39280cc0ca","unresolved":false,"context_lines":[{"line_number":1119,"context_line":"          metavar\u003d\u0027\u003cdatabase_connection\u003e\u0027,"},{"line_number":1120,"context_line":"          help\u003d\u0027The database connection url for cell0. \u0027"},{"line_number":1121,"context_line":"               \u0027This is optional. If not provided, a standard database \u0027"},{"line_number":1122,"context_line":"               \u0027connection will be used based on the main database connection \u0027"},{"line_number":1123,"context_line":"               \u0027from the Nova configuration.\u0027"},{"line_number":1124,"context_line":"         )"},{"line_number":1125,"context_line":"    def map_cell0(self, database_connection\u003dNone):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a3c09a3_829d555b","line":1122,"updated":"2017-01-16 16:51:17.000000000","message":"I had thought the reason this defaulted to the API database connection was because the cell0 database is expected to be co-located with the API database, being that both are API-cell-level constructs. As a default, they would share the same connection and user/password. These are the two databases where there is only one each, and they are in the API cell only.\n\nI\u0027m trying to think if there\u0027s any caveat to making it default to the main database connection. It would co-locate the cell0 database with whatever main database they have, and if it\u0027s different than the API database host/user/password, then they would start out separated instead of together.","commit_id":"b28202fb52284dfbc9a9180ac017ddd1977efbf6"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"58857e07d53773c3e7e50b1bf9243e8249a69d8e","unresolved":false,"context_lines":[{"line_number":1119,"context_line":"          metavar\u003d\u0027\u003cdatabase_connection\u003e\u0027,"},{"line_number":1120,"context_line":"          help\u003d\u0027The database connection url for cell0. \u0027"},{"line_number":1121,"context_line":"               \u0027This is optional. If not provided, a standard database \u0027"},{"line_number":1122,"context_line":"               \u0027connection will be used based on the main database connection \u0027"},{"line_number":1123,"context_line":"               \u0027from the Nova configuration.\u0027"},{"line_number":1124,"context_line":"         )"},{"line_number":1125,"context_line":"    def map_cell0(self, database_connection\u003dNone):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a3c09a3_33d79524","line":1122,"in_reply_to":"7a3c09a3_78865c14","updated":"2017-01-16 18:00:48.000000000","message":"Your point in IRC that operators may plan differently for cell0 because it grows indefinitely with the instances table unlike the API database, gives reason to potentially start them out separate.","commit_id":"b28202fb52284dfbc9a9180ac017ddd1977efbf6"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"68ab8b8f764e2fef73f593d1b6967e02fc50c4e9","unresolved":false,"context_lines":[{"line_number":1119,"context_line":"          metavar\u003d\u0027\u003cdatabase_connection\u003e\u0027,"},{"line_number":1120,"context_line":"          help\u003d\u0027The database connection url for cell0. \u0027"},{"line_number":1121,"context_line":"               \u0027This is optional. If not provided, a standard database \u0027"},{"line_number":1122,"context_line":"               \u0027connection will be used based on the main database connection \u0027"},{"line_number":1123,"context_line":"               \u0027from the Nova configuration.\u0027"},{"line_number":1124,"context_line":"         )"},{"line_number":1125,"context_line":"    def map_cell0(self, database_connection\u003dNone):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a3c09a3_78865c14","line":1122,"in_reply_to":"7a3c09a3_829d555b","updated":"2017-01-16 17:25:15.000000000","message":"The api nodes shouldn\u0027t have a default main database going forward, everything should be read from a cell database based on the mappings. So, configuring those nodes with their main db as the cell0 one would be a good migration path towards that ideal I think.\n\nI understand the thinking that \"the cell0 will always be co-located with the api db, so it makes sense\", but I think that won\u0027t always be the case, and that it\u0027s super confusing. We\u0027re taking something with \"api schema\", mangling the name, putting \"main schema\" inside it, and naming it something super close to the api thing.","commit_id":"b28202fb52284dfbc9a9180ac017ddd1977efbf6"}]}