)]}'
{"shade/__init__.py":[{"author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"change_message_id":"c208ca7dadc954c066da64c145130ccfb02c670e","unresolved":false,"context_lines":[{"line_number":52,"context_line":"OBJECT_SHA256_KEY \u003d \u0027x-object-meta-x-shade-sha256\u0027"},{"line_number":53,"context_line":"IMAGE_MD5_KEY \u003d \u0027owner_specified.shade.md5\u0027"},{"line_number":54,"context_line":"IMAGE_SHA256_KEY \u003d \u0027owner_specified.shade.sha256\u0027"},{"line_number":55,"context_line":"DEFAULT_OBJECT_SEGMENT_SIZE \u003d 1073741824  # 1GB"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"OBJECT_CONTAINER_ACLS \u003d {"}],"source_content_type":"text/x-python","patch_set":7,"id":"5afe65bd_1ac78846","line":55,"updated":"2015-06-03 17:05:26.000000000","message":"I\u0027m wondering if we need to allow users to configure this? Its very feasible someone configures their cloud to support \u003c 1GB segments for who knows why reason, for example.","commit_id":"99009f1bcd7622a02f3e9d342389c8ada7d20085"},{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"2f2ac01a6b5aad1c7447f5d297f0e100837cfdd8","unresolved":false,"context_lines":[{"line_number":52,"context_line":"OBJECT_SHA256_KEY \u003d \u0027x-object-meta-x-shade-sha256\u0027"},{"line_number":53,"context_line":"IMAGE_MD5_KEY \u003d \u0027owner_specified.shade.md5\u0027"},{"line_number":54,"context_line":"IMAGE_SHA256_KEY \u003d \u0027owner_specified.shade.sha256\u0027"},{"line_number":55,"context_line":"DEFAULT_OBJECT_SEGMENT_SIZE \u003d 1073741824  # 1GB"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"OBJECT_CONTAINER_ACLS \u003d {"}],"source_content_type":"text/x-python","patch_set":7,"id":"5afe65bd_bd32ae0e","line":55,"in_reply_to":"5afe65bd_1ac78846","updated":"2015-06-03 17:33:42.000000000","message":"agreed, I just didn\u0027t want to think that through while also thinking how to get segmented uploads working at all.","commit_id":"99009f1bcd7622a02f3e9d342389c8ada7d20085"},{"author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"change_message_id":"c208ca7dadc954c066da64c145130ccfb02c670e","unresolved":false,"context_lines":[{"line_number":2017,"context_line":"                container\u003dcontainer, objects\u003d[upload],"},{"line_number":2018,"context_line":"                options\u003ddict(headers\u003dheaders,"},{"line_number":2019,"context_line":"                             segment_size\u003dsegment_size))):"},{"line_number":2020,"context_line":"                if not r[\u0027success\u0027]:"},{"line_number":2021,"context_line":"                    raise OpenStackCloudException("},{"line_number":2022,"context_line":"                        \u0027Failed at action ({action}) [{error}]:\u0027.format(**r))"},{"line_number":2023,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"5afe65bd_fafdcc3a","line":2020,"updated":"2015-06-03 17:05:26.000000000","message":"The behavior here is a bit wierd and I think definitely warrants some explanation - We will except after attempting to upload all segments if any segment fails.\n\nI wonder if the swift behavior here is actually better too? There is a use case where one segment fails, you only want to re-upload that segment and we aren\u0027t really providing the user with a way to do this.","commit_id":"99009f1bcd7622a02f3e9d342389c8ada7d20085"},{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"2f2ac01a6b5aad1c7447f5d297f0e100837cfdd8","unresolved":false,"context_lines":[{"line_number":2017,"context_line":"                container\u003dcontainer, objects\u003d[upload],"},{"line_number":2018,"context_line":"                options\u003ddict(headers\u003dheaders,"},{"line_number":2019,"context_line":"                             segment_size\u003dsegment_size))):"},{"line_number":2020,"context_line":"                if not r[\u0027success\u0027]:"},{"line_number":2021,"context_line":"                    raise OpenStackCloudException("},{"line_number":2022,"context_line":"                        \u0027Failed at action ({action}) [{error}]:\u0027.format(**r))"},{"line_number":2023,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"5afe65bd_30df056b","line":2020,"in_reply_to":"5afe65bd_fafdcc3a","updated":"2015-06-03 17:33:42.000000000","message":"Not sure if it\u0027s critical to have this handled in the first iteration. The way I see it, it won\u0027t really increase the error rate, it will just increase the cost of an error. I\u0027ll have to ponder on this for a while.","commit_id":"99009f1bcd7622a02f3e9d342389c8ada7d20085"}]}