)]}'
{"nova/exception.py":[{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"deeb99ffbf70d265f35cc3faa29faea64a26395f","unresolved":false,"context_lines":[{"line_number":547,"context_line":""},{"line_number":548,"context_line":""},{"line_number":549,"context_line":"class NetworkAmbiguous(Invalid):"},{"line_number":550,"context_line":"    message \u003d _(\"More that one possible network found. Specify \""},{"line_number":551,"context_line":"                \"network ID(s) to select whcih one(s) to connect to,\")"},{"line_number":552,"context_line":""},{"line_number":553,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2FtVc%3D","line":550,"updated":"2013-06-24 20:21:34.000000000","message":"that-\u003ethan","commit_id":"fc0763e7551338dc2cb25228029f3648a26a47d9"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"deeb99ffbf70d265f35cc3faa29faea64a26395f","unresolved":false,"context_lines":[{"line_number":548,"context_line":""},{"line_number":549,"context_line":"class NetworkAmbiguous(Invalid):"},{"line_number":550,"context_line":"    message \u003d _(\"More that one possible network found. Specify \""},{"line_number":551,"context_line":"                \"network ID(s) to select whcih one(s) to connect to,\")"},{"line_number":552,"context_line":""},{"line_number":553,"context_line":""},{"line_number":554,"context_line":"class DatastoreNotFound(NotFound):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2FtnY%3D","line":551,"updated":"2013-06-24 20:21:34.000000000","message":"*which\n\nAnd it looks like a comma instead of a period at the end of the line.","commit_id":"fc0763e7551338dc2cb25228029f3648a26a47d9"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"c7ce433ba3b5bbcac4381152a319a389a2d960fa","unresolved":false,"context_lines":[{"line_number":548,"context_line":""},{"line_number":549,"context_line":"class NetworkAmbiguous(Invalid):"},{"line_number":550,"context_line":"    message \u003d _(\"More than one possible network found. Specify \""},{"line_number":551,"context_line":"                \"network ID(s) to select which one(s) to connect to,\")"},{"line_number":552,"context_line":""},{"line_number":553,"context_line":""},{"line_number":554,"context_line":"class DatastoreNotFound(NotFound):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAP3%2F%2FqDY%3D","line":551,"updated":"2013-07-15 06:16:33.000000000","message":"Why the trailing comma here?","commit_id":"6940ae5df7f4f6ed5972f0205741ef38354187c7"}],"nova/network/api.py":[{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"7ddd355d14087e15720a960de6f164551db8d10f","unresolved":false,"context_lines":[{"line_number":404,"context_line":"        \"\"\"validate the networks passed at the time of creating"},{"line_number":405,"context_line":"        the server"},{"line_number":406,"context_line":"        \"\"\""},{"line_number":407,"context_line":"        if requested_networks is None or len(requested_networks) \u003d\u003d 0:"},{"line_number":408,"context_line":"            return"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"        return self.network_rpcapi.validate_networks(context,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2FwlE%3D","line":407,"updated":"2013-06-21 22:03:10.000000000","message":"This is the same as \"if not requested_networks:\".","commit_id":"912ea46f99eb8cb19db762b0d598d8db7df8fac9"}],"nova/network/quantumv2/api.py":[{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"b14afe6d32b027d1ef679ede761d566809779245","unresolved":false,"context_lines":[{"line_number":449,"context_line":"        if not requested_networks:"},{"line_number":450,"context_line":"            nets \u003d self._get_available_networks(context, context.project_id)"},{"line_number":451,"context_line":"            if len(nets) \u003e 1:"},{"line_number":452,"context_line":"                # Atttaching to more that one netwrok by default doesn\u0027t"},{"line_number":453,"context_line":"                # make sense, as the order will be arbitary and the guest OS"},{"line_number":454,"context_line":"                # won\u0027t know which to configure"},{"line_number":455,"context_line":"                msg \u003d (_(\"Multiple possible networks found, use an Network \""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2FxkQ%3D","line":452,"updated":"2013-06-21 15:08:12.000000000","message":"s/netwrok/network/","commit_id":"dd1cf81d56b958bbce9400e294cbe30b3964c80a"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"7ddd355d14087e15720a960de6f164551db8d10f","unresolved":false,"context_lines":[{"line_number":445,"context_line":"        LOG.debug(_(\u0027validate_networks() for %s\u0027),"},{"line_number":446,"context_line":"                  requested_networks)"},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"        net_ids \u003d []"},{"line_number":449,"context_line":"        if not requested_networks:"},{"line_number":450,"context_line":"            nets \u003d self._get_available_networks(context, context.project_id)"},{"line_number":451,"context_line":"            if len(nets) \u003e 1:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Fwl8%3D","line":448,"updated":"2013-06-21 22:03:10.000000000","message":"Since this isn\u0027t used in the requested_networks check, I think it would make more sense to leave it below like it was before.","commit_id":"912ea46f99eb8cb19db762b0d598d8db7df8fac9"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"7ddd355d14087e15720a960de6f164551db8d10f","unresolved":false,"context_lines":[{"line_number":449,"context_line":"        if not requested_networks:"},{"line_number":450,"context_line":"            nets \u003d self._get_available_networks(context, context.project_id)"},{"line_number":451,"context_line":"            if len(nets) \u003e 1:"},{"line_number":452,"context_line":"                # Atttaching to more that one network by default doesn\u0027t"},{"line_number":453,"context_line":"                # make sense, as the order will be arbitary and the guest OS"},{"line_number":454,"context_line":"                # won\u0027t know which to configure"},{"line_number":455,"context_line":"                msg \u003d (_(\"Multiple possible networks found, use an Network \""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Fwks%3D","line":452,"updated":"2013-06-21 22:03:10.000000000","message":"Extra t in Attaching and that -\u003e than","commit_id":"912ea46f99eb8cb19db762b0d598d8db7df8fac9"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"7ddd355d14087e15720a960de6f164551db8d10f","unresolved":false,"context_lines":[{"line_number":450,"context_line":"            nets \u003d self._get_available_networks(context, context.project_id)"},{"line_number":451,"context_line":"            if len(nets) \u003e 1:"},{"line_number":452,"context_line":"                # Atttaching to more that one network by default doesn\u0027t"},{"line_number":453,"context_line":"                # make sense, as the order will be arbitary and the guest OS"},{"line_number":454,"context_line":"                # won\u0027t know which to configure"},{"line_number":455,"context_line":"                msg \u003d (_(\"Multiple possible networks found, use an Network \""},{"line_number":456,"context_line":"                         \"ID to be more specific.\"))"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Fwko%3D","line":453,"updated":"2013-06-21 22:03:10.000000000","message":"arbitary -\u003e arbitrary","commit_id":"912ea46f99eb8cb19db762b0d598d8db7df8fac9"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"7ddd355d14087e15720a960de6f164551db8d10f","unresolved":false,"context_lines":[{"line_number":452,"context_line":"                # Atttaching to more that one network by default doesn\u0027t"},{"line_number":453,"context_line":"                # make sense, as the order will be arbitary and the guest OS"},{"line_number":454,"context_line":"                # won\u0027t know which to configure"},{"line_number":455,"context_line":"                msg \u003d (_(\"Multiple possible networks found, use an Network \""},{"line_number":456,"context_line":"                         \"ID to be more specific.\"))"},{"line_number":457,"context_line":"                raise exception.NetworkAmbiguous(msg)"},{"line_number":458,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Fwkk%3D","line":455,"updated":"2013-06-21 22:03:10.000000000","message":"an -\u003e a\n\nAlso, I think the outer () are unnecessary.","commit_id":"912ea46f99eb8cb19db762b0d598d8db7df8fac9"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"deeb99ffbf70d265f35cc3faa29faea64a26395f","unresolved":false,"context_lines":[{"line_number":451,"context_line":"                # Attaching to more than one network by default doesn\u0027t"},{"line_number":452,"context_line":"                # make sense, as the order will be arbitrary and the guest OS"},{"line_number":453,"context_line":"                # won\u0027t know which to configure"},{"line_number":454,"context_line":"                msg \u003d _(\"Multiple possible networks found, use an Network \""},{"line_number":455,"context_line":"                         \"ID to be more specific.\")"},{"line_number":456,"context_line":"                raise exception.NetworkAmbiguous(msg)"},{"line_number":457,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2FtWY%3D","line":454,"updated":"2013-06-24 20:21:34.000000000","message":"I still think \"an\" should be \"a\" here.","commit_id":"fc0763e7551338dc2cb25228029f3648a26a47d9"}]}