)]}'
{"nova/cmd/manage.py":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"1f9a36267f81529e3537f72658bf1e490742996a","unresolved":false,"context_lines":[{"line_number":1190,"context_line":"          dest\u003d\u0027transport_url\u0027,"},{"line_number":1191,"context_line":"          help\u003d\u0027The transport url for the cell message queue\u0027)"},{"line_number":1192,"context_line":"    def simple_cell_setup(self, transport_url):"},{"line_number":1193,"context_line":"        cell_uuid \u003d self._map_cell_and_hosts(transport_url)"},{"line_number":1194,"context_line":"        if cell_uuid is None:"},{"line_number":1195,"context_line":"            # There are no compute hosts which means no cell_mapping was"},{"line_number":1196,"context_line":"            # created. This should also mean that there are no instances."}],"source_content_type":"text/x-python","patch_set":1,"id":"9ad45d7e_4ec6714a","line":1193,"updated":"2016-08-12 15:00:39.000000000","message":"This is me perhaps being overly cautious but if this command is able to be run multiple times without failure I worry that it will be abused as a way to add new hosts into a cell. There\u0027s a dedicated, and cleaner, method for doing that being proposed at https://review.openstack.org/#/c/328491/.\n\nIf the call to _map_cell_and_hosts was moved to after the map_cell0 try/except it would alleviate that concern.","commit_id":"2ae93d1ed462c1944f7a699f9a9fde62de8a1762"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5858f2532afcf090efebda4da6cdd8c809dd08e0","unresolved":false,"context_lines":[{"line_number":1198,"context_line":"        try:"},{"line_number":1199,"context_line":"            self.map_cell0()"},{"line_number":1200,"context_line":"        except db_exc.DBDuplicateEntry:"},{"line_number":1201,"context_line":"            print(\u0027Already setup, nothing to do.\u0027)"},{"line_number":1202,"context_line":"            return 0"},{"line_number":1203,"context_line":"        self.map_instances(cell_uuid)"},{"line_number":1204,"context_line":"        return 0"}],"source_content_type":"text/x-python","patch_set":1,"id":"9ad45d7e_6236eee8","line":1201,"updated":"2016-08-12 13:54:50.000000000","message":"Nit: seems a bit terse, but I guess it says what it needs to.","commit_id":"2ae93d1ed462c1944f7a699f9a9fde62de8a1762"}]}