)]}'
{"doc/source/man/nova-manage.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"10543fd2574c985f642f30f6dd3950532ea75f9b","unresolved":false,"context_lines":[{"line_number":163,"context_line":"    found, 2 if host mappings were found for the cell (cell not empty), and"},{"line_number":164,"context_line":"    3 if there are instances mapped to the cell (cell not empty)."},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"``nova-manage cell_v2 update_cell --cell_uuid \u003ccell_uuid\u003e [--name \u003ccell_name\u003e] [--transport-url \u003ctransport_url\u003e] [--database_connection \u003cdatabase_connection]``"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    Updates the properties of a cell by the given uuid. If the cell is not"},{"line_number":169,"context_line":"    found by uuid, this command will return an exit code of 1. If the"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a430d35_65e68e16","line":166,"range":{"start_line":166,"start_character":137,"end_line":166,"end_character":156},"updated":"2017-02-07 18:16:51.000000000","message":"Missing a \u003e at the end of database_connection.","commit_id":"59819c40e91c025f79e7645b9b2875cc4b857b49"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"1c8c4d3dd2dbd7d9093c240e44bcec2eb173a826","unresolved":false,"context_lines":[{"line_number":172,"context_line":""},{"line_number":173,"context_line":"    NOTE: Updating the transport_url or database_connection fields on"},{"line_number":174,"context_line":"    a running system will NOT result in all nodes immediately using the"},{"line_number":175,"context_line":"    new values. Use caution when changing these values."},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"Nova Logs"},{"line_number":178,"context_line":"~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a430d35_8eb5247f","line":175,"updated":"2017-02-08 18:26:03.000000000","message":"Based on https://github.com/openstack/nova/blob/master/nova/compute/api.py#L109 and my comment in PS2, it seems this should be clear and say that it takes effect after restarting nova-api.","commit_id":"1102441a4259fe155eb2a61cd16c6e015787472c"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f0b6f01aab99d2327683ddab02cb1284bd147dc8","unresolved":false,"context_lines":[{"line_number":172,"context_line":""},{"line_number":173,"context_line":"    NOTE: Updating the transport_url or database_connection fields on"},{"line_number":174,"context_line":"    a running system will NOT result in all nodes immediately using the"},{"line_number":175,"context_line":"    new values. Use caution when changing these values."},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"Nova Logs"},{"line_number":178,"context_line":"~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a430d35_42eced29","line":175,"in_reply_to":"1a430d35_8eb5247f","updated":"2017-02-08 19:56:12.000000000","message":"Well, the thing is, we know we need to age out that cache, we\u0027re just not doing it now. So if we say restart here, it\u0027ll be wrong (or misleading) once we do that. Is \"use caution\" really not enough here?","commit_id":"1102441a4259fe155eb2a61cd16c6e015787472c"}],"nova/cmd/manage.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b760542c36ce5562f8d351164d3799882492dd23","unresolved":false,"context_lines":[{"line_number":1509,"context_line":""},{"line_number":1510,"context_line":"        If the cell is not found by uuid, this command will return an exit"},{"line_number":1511,"context_line":"        code of 1. If the properties cannot be set, this will return 2."},{"line_number":1512,"context_line":"        Otherwise, the exit code will 0."},{"line_number":1513,"context_line":""},{"line_number":1514,"context_line":"        NOTE: Updating the transport_url or database_connection fields on"},{"line_number":1515,"context_line":"        a running system will not result in all nodes immediately using the"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_7b7630a5","line":1512,"range":{"start_line":1512,"start_character":33,"end_line":1512,"end_character":37},"updated":"2017-02-07 17:08:43.000000000","message":"will be","commit_id":"790f8ac796a6a04496e67e564e87e249bf47fd9c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b760542c36ce5562f8d351164d3799882492dd23","unresolved":false,"context_lines":[{"line_number":1511,"context_line":"        code of 1. If the properties cannot be set, this will return 2."},{"line_number":1512,"context_line":"        Otherwise, the exit code will 0."},{"line_number":1513,"context_line":""},{"line_number":1514,"context_line":"        NOTE: Updating the transport_url or database_connection fields on"},{"line_number":1515,"context_line":"        a running system will not result in all nodes immediately using the"},{"line_number":1516,"context_line":"        new values. Use caution when changing these values."},{"line_number":1517,"context_line":"        \"\"\""},{"line_number":1518,"context_line":"        ctxt \u003d context.get_admin_context()"},{"line_number":1519,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_fb6a20b2","line":1516,"range":{"start_line":1514,"start_character":14,"end_line":1516,"end_character":59},"updated":"2017-02-07 17:08:43.000000000","message":"I\u0027d move this into the help for those options above.","commit_id":"790f8ac796a6a04496e67e564e87e249bf47fd9c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b760542c36ce5562f8d351164d3799882492dd23","unresolved":false,"context_lines":[{"line_number":1532,"context_line":"        try:"},{"line_number":1533,"context_line":"            cell_mapping.save()"},{"line_number":1534,"context_line":"        except Exception as e:"},{"line_number":1535,"context_line":"            print(_(\u0027Unable to save CellMapping: %s\u0027) % e)"},{"line_number":1536,"context_line":"            return 2"},{"line_number":1537,"context_line":""},{"line_number":1538,"context_line":"        return 0"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_1bafece2","line":1535,"range":{"start_line":1535,"start_character":31,"end_line":1535,"end_character":35},"updated":"2017-02-07 17:08:43.000000000","message":"nit: I\u0027d say \u0027update\u0027 rather than save here since that\u0027s the end-user operation.","commit_id":"790f8ac796a6a04496e67e564e87e249bf47fd9c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"10543fd2574c985f642f30f6dd3950532ea75f9b","unresolved":false,"context_lines":[{"line_number":1502,"context_line":"          help\u003d_(\u0027Set the cell name.\u0027))"},{"line_number":1503,"context_line":"    @args(\u0027--transport-url\u0027, metavar\u003d\u0027\u003ctransport\u003e\u0027, dest\u003d\u0027transport_url\u0027,"},{"line_number":1504,"context_line":"          help\u003d_(\u0027Set the cell transport_url. NOTE that running nodes \u0027"},{"line_number":1505,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1506,"context_line":"    @args(\u0027--database_connection\u0027, metavar\u003d\u0027\u003cdatabase\u003e\u0027, dest\u003d\u0027db_connection\u0027,"},{"line_number":1507,"context_line":"          help\u003d_(\u0027Set the cell database_connection. NOTE that running nodes \u0027"},{"line_number":1508,"context_line":"                 \u0027will not see the change until restart!\u0027))"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_e5951e33","line":1505,"range":{"start_line":1505,"start_character":48,"end_line":1505,"end_character":55},"updated":"2017-02-07 18:16:51.000000000","message":"Why is a restart required since this is updating the database directly? i.e. when we create an instance in conductor, we\u0027ll get a host from the scheduler, then we lookup the host mapping for that host which points to a cell that the host is in, which gives us the cell mapping which would have it\u0027s transport and/or db connection updated here. Why would a restart be required for that?","commit_id":"59819c40e91c025f79e7645b9b2875cc4b857b49"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"829690aa32f44ce80ab52bd90464d443e26a155d","unresolved":false,"context_lines":[{"line_number":1502,"context_line":"          help\u003d_(\u0027Set the cell name.\u0027))"},{"line_number":1503,"context_line":"    @args(\u0027--transport-url\u0027, metavar\u003d\u0027\u003ctransport\u003e\u0027, dest\u003d\u0027transport_url\u0027,"},{"line_number":1504,"context_line":"          help\u003d_(\u0027Set the cell transport_url. NOTE that running nodes \u0027"},{"line_number":1505,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1506,"context_line":"    @args(\u0027--database_connection\u0027, metavar\u003d\u0027\u003cdatabase\u003e\u0027, dest\u003d\u0027db_connection\u0027,"},{"line_number":1507,"context_line":"          help\u003d_(\u0027Set the cell database_connection. NOTE that running nodes \u0027"},{"line_number":1508,"context_line":"                 \u0027will not see the change until restart!\u0027))"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_cede4cdc","line":1505,"range":{"start_line":1505,"start_character":48,"end_line":1505,"end_character":55},"in_reply_to":"1a430d35_c169edfa","updated":"2017-02-08 18:24:19.000000000","message":"Oh yeah, gdi:\n\nhttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L109\n\nShouldn\u0027t we be more specific about what needs to be restarted then? Does this mean I need to restart all services in my entire deployment? Really it just means nova-api.","commit_id":"59819c40e91c025f79e7645b9b2875cc4b857b49"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"81f73abe17bcfb6191029e9095b206ec0d6412a9","unresolved":false,"context_lines":[{"line_number":1502,"context_line":"          help\u003d_(\u0027Set the cell name.\u0027))"},{"line_number":1503,"context_line":"    @args(\u0027--transport-url\u0027, metavar\u003d\u0027\u003ctransport\u003e\u0027, dest\u003d\u0027transport_url\u0027,"},{"line_number":1504,"context_line":"          help\u003d_(\u0027Set the cell transport_url. NOTE that running nodes \u0027"},{"line_number":1505,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1506,"context_line":"    @args(\u0027--database_connection\u0027, metavar\u003d\u0027\u003cdatabase\u003e\u0027, dest\u003d\u0027db_connection\u0027,"},{"line_number":1507,"context_line":"          help\u003d_(\u0027Set the cell database_connection. NOTE that running nodes \u0027"},{"line_number":1508,"context_line":"                 \u0027will not see the change until restart!\u0027))"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_c169edfa","line":1505,"range":{"start_line":1505,"start_character":48,"end_line":1505,"end_character":55},"in_reply_to":"1a430d35_e5951e33","updated":"2017-02-07 23:35:20.000000000","message":"This is for the CellMapping, which we list and cache all of them at the first go right now:\n\nhttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L2515-L2519","commit_id":"59819c40e91c025f79e7645b9b2875cc4b857b49"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"10543fd2574c985f642f30f6dd3950532ea75f9b","unresolved":false,"context_lines":[{"line_number":1528,"context_line":"            cell_mapping.name \u003d name"},{"line_number":1529,"context_line":"        if transport_url:"},{"line_number":1530,"context_line":"            cell_mapping.transport_url \u003d transport_url"},{"line_number":1531,"context_line":"        if db_connection:"},{"line_number":1532,"context_line":"            cell_mapping.database_connection \u003d db_connection"},{"line_number":1533,"context_line":""},{"line_number":1534,"context_line":"        try:"},{"line_number":1535,"context_line":"            cell_mapping.save()"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_c5de6210","line":1532,"range":{"start_line":1531,"start_character":8,"end_line":1532,"end_character":60},"updated":"2017-02-07 18:16:51.000000000","message":"Just thinking out loud, should we automatically run discover_hosts on this cell if we change it\u0027s db connection? I guess if that\u0027s something we think should be done, people could do it separately, or we could later add an option to do that to this command. If it\u0027s something we think someone should do though, then we might want to mention that in the docs.","commit_id":"59819c40e91c025f79e7645b9b2875cc4b857b49"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"1c8c4d3dd2dbd7d9093c240e44bcec2eb173a826","unresolved":false,"context_lines":[{"line_number":1502,"context_line":"          help\u003d_(\u0027Set the cell name.\u0027))"},{"line_number":1503,"context_line":"    @args(\u0027--transport-url\u0027, metavar\u003d\u0027\u003ctransport\u003e\u0027, dest\u003d\u0027transport_url\u0027,"},{"line_number":1504,"context_line":"          help\u003d_(\u0027Set the cell transport_url. NOTE that running nodes \u0027"},{"line_number":1505,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1506,"context_line":"    @args(\u0027--database_connection\u0027, metavar\u003d\u0027\u003cdatabase\u003e\u0027, dest\u003d\u0027db_connection\u0027,"},{"line_number":1507,"context_line":"          help\u003d_(\u0027Set the cell database_connection. NOTE that running nodes \u0027"},{"line_number":1508,"context_line":"                 \u0027will not see the change until restart!\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a430d35_2ea51046","line":1505,"range":{"start_line":1505,"start_character":48,"end_line":1505,"end_character":55},"updated":"2017-02-08 18:26:03.000000000","message":"As mentioned, this should just be nova-api right?","commit_id":"1102441a4259fe155eb2a61cd16c6e015787472c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a3d75823dda49a68af2205ed8ee4e7e04ce26f34","unresolved":false,"context_lines":[{"line_number":1502,"context_line":"          help\u003d_(\u0027Set the cell name.\u0027))"},{"line_number":1503,"context_line":"    @args(\u0027--transport-url\u0027, metavar\u003d\u0027\u003ctransport\u003e\u0027, dest\u003d\u0027transport_url\u0027,"},{"line_number":1504,"context_line":"          help\u003d_(\u0027Set the cell transport_url. NOTE that running nodes \u0027"},{"line_number":1505,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1506,"context_line":"    @args(\u0027--database_connection\u0027, metavar\u003d\u0027\u003cdatabase\u003e\u0027, dest\u003d\u0027db_connection\u0027,"},{"line_number":1507,"context_line":"          help\u003d_(\u0027Set the cell database_connection. NOTE that running nodes \u0027"},{"line_number":1508,"context_line":"                 \u0027will not see the change until restart!\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a430d35_dd1efc0a","line":1505,"range":{"start_line":1505,"start_character":48,"end_line":1505,"end_character":55},"in_reply_to":"1a430d35_22fe01ae","updated":"2017-02-08 20:18:41.000000000","message":"\u003e but later will need to age those out and we\u0027ve even discussed a fanout rpc to invalidate those caches.\n\nWe could change this when the later thing happens, although realistically by that time we\u0027ll have forgotten.\n\nI don\u0027t want to hold this change up on this point. If at some point someone comes into the channel and asks about what this restart means, I want you to send me a check for $5 US.","commit_id":"1102441a4259fe155eb2a61cd16c6e015787472c"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f0b6f01aab99d2327683ddab02cb1284bd147dc8","unresolved":false,"context_lines":[{"line_number":1502,"context_line":"          help\u003d_(\u0027Set the cell name.\u0027))"},{"line_number":1503,"context_line":"    @args(\u0027--transport-url\u0027, metavar\u003d\u0027\u003ctransport\u003e\u0027, dest\u003d\u0027transport_url\u0027,"},{"line_number":1504,"context_line":"          help\u003d_(\u0027Set the cell transport_url. NOTE that running nodes \u0027"},{"line_number":1505,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1506,"context_line":"    @args(\u0027--database_connection\u0027, metavar\u003d\u0027\u003cdatabase\u003e\u0027, dest\u003d\u0027db_connection\u0027,"},{"line_number":1507,"context_line":"          help\u003d_(\u0027Set the cell database_connection. NOTE that running nodes \u0027"},{"line_number":1508,"context_line":"                 \u0027will not see the change until restart!\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a430d35_22fe01ae","line":1505,"range":{"start_line":1505,"start_character":48,"end_line":1505,"end_character":55},"in_reply_to":"1a430d35_2ea51046","updated":"2017-02-08 19:56:12.000000000","message":"I really don\u0027t think we should be that specific. Right now, API does cache the cells, but later will need to age those out and we\u0027ve even discussed a fanout rpc to invalidate those caches. Conductor does not cache them, but likely needs to, and eventually we need to be deploying a conductor above the cell level to handle that call anyway. Thinking this over again, I would have changed this to \"will not see the change immediately\".\n\nI put \"restart\" here in the help lines because I think it\u0027s the easiest way to convey the general idea in a single-line help string, but it\u0027s really not what I want to say. Making it more specific does not seem like a win to me.","commit_id":"1102441a4259fe155eb2a61cd16c6e015787472c"},{"author":{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},"change_message_id":"a22adbd0362ae21a2ffe920441b76b1d504c2887","unresolved":false,"context_lines":[{"line_number":1496,"context_line":"        cell_mapping.destroy()"},{"line_number":1497,"context_line":"        return 0"},{"line_number":1498,"context_line":""},{"line_number":1499,"context_line":"    @args(\u0027--cell_uuid\u0027, metavar\u003d\u0027\u003ccell_uuid\u003e\u0027, dest\u003d\u0027cell_uuid\u0027,"},{"line_number":1500,"context_line":"          required\u003dTrue, help\u003d_(\u0027The uuid of the cell to update.\u0027))"},{"line_number":1501,"context_line":"    @args(\u0027--name\u0027, metavar\u003d\u0027\u003cname\u003e\u0027, dest\u003d\u0027name\u0027,"},{"line_number":1502,"context_line":"          help\u003d_(\u0027Set the cell name.\u0027))"},{"line_number":1503,"context_line":"    @args(\u0027--transport-url\u0027, metavar\u003d\u0027\u003ctransport\u003e\u0027, dest\u003d\u0027transport_url\u0027,"},{"line_number":1504,"context_line":"          help\u003d_(\u0027Set the cell transport_url. NOTE that running nodes \u0027"},{"line_number":1505,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1506,"context_line":"    @args(\u0027--database_connection\u0027, metavar\u003d\u0027\u003cdatabase\u003e\u0027, dest\u003d\u0027db_connection\u0027,"},{"line_number":1507,"context_line":"          help\u003d_(\u0027Set the cell database_connection. NOTE that running nodes \u0027"},{"line_number":1508,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1509,"context_line":"    def update_cell(self, cell_uuid, name, transport_url, db_connection):"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a430d35_84afc57b","line":1506,"range":{"start_line":1499,"start_character":1,"end_line":1506,"end_character":78},"updated":"2017-02-08 19:05:53.000000000","message":"FYI\n\nThis was new to me recently, but apparently the \"metavar\" string should match the string used on the man page b/c it is user-facing. For example, \"transport\" vs \"transport_url\", and \"database\" vs. \"database_connection\".\n\n    nova-manage cell_v2 update_cell --cell_uuid \u003ccell_uuid\u003e [--name \u003ccell_name\u003e] [--transport-url \u003ctransport_url\u003e] [--database_connection \u003cdatabase_connection\u003e]\n\nAnyhoo, we decided to keep the man pages consistent, and fix these metavars up in a subsequent pass, so if you don\u0027t end up fixing the \"metavar\" strings when you respin next, I\u0027ll fix them later when this merges.","commit_id":"1102441a4259fe155eb2a61cd16c6e015787472c"},{"author":{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},"change_message_id":"15970c796068f8220299e09ad7959a727c8a4976","unresolved":false,"context_lines":[{"line_number":1496,"context_line":"        cell_mapping.destroy()"},{"line_number":1497,"context_line":"        return 0"},{"line_number":1498,"context_line":""},{"line_number":1499,"context_line":"    @args(\u0027--cell_uuid\u0027, metavar\u003d\u0027\u003ccell_uuid\u003e\u0027, dest\u003d\u0027cell_uuid\u0027,"},{"line_number":1500,"context_line":"          required\u003dTrue, help\u003d_(\u0027The uuid of the cell to update.\u0027))"},{"line_number":1501,"context_line":"    @args(\u0027--name\u0027, metavar\u003d\u0027\u003cname\u003e\u0027, dest\u003d\u0027name\u0027,"},{"line_number":1502,"context_line":"          help\u003d_(\u0027Set the cell name.\u0027))"},{"line_number":1503,"context_line":"    @args(\u0027--transport-url\u0027, metavar\u003d\u0027\u003ctransport\u003e\u0027, dest\u003d\u0027transport_url\u0027,"},{"line_number":1504,"context_line":"          help\u003d_(\u0027Set the cell transport_url. NOTE that running nodes \u0027"},{"line_number":1505,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1506,"context_line":"    @args(\u0027--database_connection\u0027, metavar\u003d\u0027\u003cdatabase\u003e\u0027, dest\u003d\u0027db_connection\u0027,"},{"line_number":1507,"context_line":"          help\u003d_(\u0027Set the cell database_connection. NOTE that running nodes \u0027"},{"line_number":1508,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1509,"context_line":"    def update_cell(self, cell_uuid, name, transport_url, db_connection):"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a430d35_b99f5264","line":1506,"range":{"start_line":1499,"start_character":1,"end_line":1506,"end_character":78},"in_reply_to":"1a430d35_84afc57b","updated":"2017-02-09 19:36:29.000000000","message":"Here\u0027s a follow-up patch fixing the metavar strings for all cell_v2 commands:\n\n- https://review.openstack.org/#/c/431714/","commit_id":"1102441a4259fe155eb2a61cd16c6e015787472c"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f0b6f01aab99d2327683ddab02cb1284bd147dc8","unresolved":false,"context_lines":[{"line_number":1496,"context_line":"        cell_mapping.destroy()"},{"line_number":1497,"context_line":"        return 0"},{"line_number":1498,"context_line":""},{"line_number":1499,"context_line":"    @args(\u0027--cell_uuid\u0027, metavar\u003d\u0027\u003ccell_uuid\u003e\u0027, dest\u003d\u0027cell_uuid\u0027,"},{"line_number":1500,"context_line":"          required\u003dTrue, help\u003d_(\u0027The uuid of the cell to update.\u0027))"},{"line_number":1501,"context_line":"    @args(\u0027--name\u0027, metavar\u003d\u0027\u003cname\u003e\u0027, dest\u003d\u0027name\u0027,"},{"line_number":1502,"context_line":"          help\u003d_(\u0027Set the cell name.\u0027))"},{"line_number":1503,"context_line":"    @args(\u0027--transport-url\u0027, metavar\u003d\u0027\u003ctransport\u003e\u0027, dest\u003d\u0027transport_url\u0027,"},{"line_number":1504,"context_line":"          help\u003d_(\u0027Set the cell transport_url. NOTE that running nodes \u0027"},{"line_number":1505,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1506,"context_line":"    @args(\u0027--database_connection\u0027, metavar\u003d\u0027\u003cdatabase\u003e\u0027, dest\u003d\u0027db_connection\u0027,"},{"line_number":1507,"context_line":"          help\u003d_(\u0027Set the cell database_connection. NOTE that running nodes \u0027"},{"line_number":1508,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1509,"context_line":"    def update_cell(self, cell_uuid, name, transport_url, db_connection):"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a430d35_e21eb90a","line":1506,"range":{"start_line":1499,"start_character":1,"end_line":1506,"end_character":78},"in_reply_to":"1a430d35_84afc57b","updated":"2017-02-08 19:56:12.000000000","message":"ugh, okay I was just trying to make these single-line, but okay.","commit_id":"1102441a4259fe155eb2a61cd16c6e015787472c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"1c8c4d3dd2dbd7d9093c240e44bcec2eb173a826","unresolved":false,"context_lines":[{"line_number":1505,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1506,"context_line":"    @args(\u0027--database_connection\u0027, metavar\u003d\u0027\u003cdatabase\u003e\u0027, dest\u003d\u0027db_connection\u0027,"},{"line_number":1507,"context_line":"          help\u003d_(\u0027Set the cell database_connection. NOTE that running nodes \u0027"},{"line_number":1508,"context_line":"                 \u0027will not see the change until restart!\u0027))"},{"line_number":1509,"context_line":"    def update_cell(self, cell_uuid, name, transport_url, db_connection):"},{"line_number":1510,"context_line":"        \"\"\"Updates the properties of a cell by the given uuid."},{"line_number":1511,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a430d35_6e69d8f8","line":1508,"range":{"start_line":1508,"start_character":48,"end_line":1508,"end_character":55},"updated":"2017-02-08 18:26:03.000000000","message":"same","commit_id":"1102441a4259fe155eb2a61cd16c6e015787472c"}]}