)]}'
{"swift/common/storage_policy.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"f46b04f2836082fed564a91e64945c343d2bf4b8","unresolved":false,"context_lines":[{"line_number":236,"context_line":"        if ec_type not in VALID_EC_ALGORITHMS:"},{"line_number":237,"context_line":"            raise PolicyError(\u0027Wrong ec_type %s for policy %s, should be one\u0027"},{"line_number":238,"context_line":"                              \u0027 of \"%s\"\u0027 % (ec_type, self.name,"},{"line_number":239,"context_line":"                              \u0027, \u0027.join(VALID_EC_ALGORITHMS)))"},{"line_number":240,"context_line":"        self._ec_type \u003d ec_type"},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"        # Define _ec_ndata as the number of EC data fragments"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_c5a0c0f6","line":239,"updated":"2015-01-05 20:07:08.000000000","message":"idk, if py_eclib grows a newly supported type I\u0027m not sure we should require you patch swift to use it...","commit_id":"134a323c6edeacee023896311c49c90e25660dd2"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"f46b04f2836082fed564a91e64945c343d2bf4b8","unresolved":false,"context_lines":[{"line_number":286,"context_line":"                              \"Unsupported ec_type: %s for policy %s. \""},{"line_number":287,"context_line":"                              \"Supported EC algorithms are %s\""},{"line_number":288,"context_line":"                              % (self._ec_type, self.name,"},{"line_number":289,"context_line":"                                 \u0027, \u0027.join(VALID_EC_ALGORITHMS)))"},{"line_number":290,"context_line":""},{"line_number":291,"context_line":"        # quorum_size is the minimum number of data + parity elements required"},{"line_number":292,"context_line":"        # to be able to guarantee the desired fault tolerance, which is the"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_85895871","line":289,"updated":"2015-01-05 20:07:08.000000000","message":"so we already catch and handle an invalid type pretty well, seems reasonable that the operator could refer to the documentation of the version of py_eclib they\u0027re running to get the list of types supported by that version of py_eclib.\n\nIf py_eclib could be patched to expose *it\u0027s* list of valid types - then it would seem reasonable to add that context to the error message - but I\u0027m not very keen on trying to keep our code in sync with them...","commit_id":"134a323c6edeacee023896311c49c90e25660dd2"}]}