)]}'
{"nova/network/api.py":[{"author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"change_message_id":"8845effb5d92bfab93158817faab1b40bb0fb3c8","unresolved":false,"context_lines":[{"line_number":67,"context_line":"            nw_info \u003d api._get_instance_nw_info(context, instance)"},{"line_number":68,"context_line":"        # update cache"},{"line_number":69,"context_line":"        cache \u003d {\u0027network_info\u0027: nw_info.json()}"},{"line_number":70,"context_line":"        update_api \u003d conductor_api if conductor_api else api.db"},{"line_number":71,"context_line":"        instance_arg \u003d instance if conductor_api else instance[\u0027uuid\u0027]"},{"line_number":72,"context_line":"        update_api.instance_info_cache_update(context, instance_arg, cache)"},{"line_number":73,"context_line":"    except Exception:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FS%2Fg%3D","line":70,"updated":"2013-01-29 21:10:03.000000000","message":"I personally would prefer this to be separate blocks for conductor and the DB API rather than using the ternary operators here.\n\nSo:\n\nif conductor_api:\n    conductor_api.instance_info_cache_update(context, instance, cache)\nelse:\n    api.db.instance_info_cache_update(context, instance[\u0027uuid\u0027], cache)\n\nJust my take. Not a deal breaker.","commit_id":"9a14f9efff568601c54b63b9c808c800a5d4dab9"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"237034cd56331464e183736e775545472bc4e0f3","unresolved":false,"context_lines":[{"line_number":67,"context_line":"            nw_info \u003d api._get_instance_nw_info(context, instance)"},{"line_number":68,"context_line":"        # update cache"},{"line_number":69,"context_line":"        cache \u003d {\u0027network_info\u0027: nw_info.json()}"},{"line_number":70,"context_line":"        update_api \u003d conductor_api if conductor_api else api.db"},{"line_number":71,"context_line":"        instance_arg \u003d instance if conductor_api else instance[\u0027uuid\u0027]"},{"line_number":72,"context_line":"        update_api.instance_info_cache_update(context, instance_arg, cache)"},{"line_number":73,"context_line":"    except Exception:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FQmI%3D","line":70,"in_reply_to":"AAAAM3%2F%2FS%2Fg%3D","updated":"2013-01-31 02:57:24.000000000","message":"Yeah, that\u0027s better.  Will change it","commit_id":"9a14f9efff568601c54b63b9c808c800a5d4dab9"}]}