)]}'
{"nova/api/openstack/compute/contrib/disk_config.py":[{"author":{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},"change_message_id":"eef1057d247605da95ce149f45eb2865b8addad9","unresolved":false,"context_lines":[{"line_number":103,"context_line":""},{"line_number":104,"context_line":"class ServerDiskConfigController(wsgi.Controller):"},{"line_number":105,"context_line":"    def _add_disk_config(self, context, servers):"},{"line_number":106,"context_line":"        if authorize(context):"},{"line_number":107,"context_line":"            # Filter out any servers that already have the key set"},{"line_number":108,"context_line":"            # (most likely from a remote zone)"},{"line_number":109,"context_line":"            servers \u003d [s for s in servers if API_DISK_CONFIG not in s]"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAEH%2F%2F%2B7M%3D","line":106,"updated":"2012-01-24 18:24:44.000000000","message":"I\u0027d slightly favor moving this `authorize` call out to `show` and `detail`.\n\nThat would lend a comforting symmetry to the file: all of the expose CRUD methods would follow the exact same pattern:\n\n@wsgi.extends\ndef exposed_method()\n  context \u003d get_context()\n  if authorize(context):\n    do something...","commit_id":"254fc2f3e2f1b00ecaecb99cf92427947cd7c373"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"06d870d57116a53f4d44896678b947076db6d58d","unresolved":false,"context_lines":[{"line_number":103,"context_line":""},{"line_number":104,"context_line":"class ServerDiskConfigController(wsgi.Controller):"},{"line_number":105,"context_line":"    def _add_disk_config(self, context, servers):"},{"line_number":106,"context_line":"        if authorize(context):"},{"line_number":107,"context_line":"            # Filter out any servers that already have the key set"},{"line_number":108,"context_line":"            # (most likely from a remote zone)"},{"line_number":109,"context_line":"            servers \u003d [s for s in servers if API_DISK_CONFIG not in s]"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAEH%2F%2F%2B50%3D","line":106,"in_reply_to":"AAAAEH%2F%2F%2B7M%3D","updated":"2012-01-24 18:31:30.000000000","message":"Done. I\u0027m planning to develop a method that doesn\u0027t require all this extra cruft soon.","commit_id":"254fc2f3e2f1b00ecaecb99cf92427947cd7c373"}]}