)]}'
{"glanceclient/v1/shell.py":[{"author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"change_message_id":"ee5455e1aeb206180532bf918d19714eb7696754","unresolved":false,"context_lines":[{"line_number":227,"context_line":"        filesize \u003d utils.get_file_size(fields[\u0027data\u0027])"},{"line_number":228,"context_line":"        if filesize is not None:"},{"line_number":229,"context_line":"            # NOTE(kragniz): do not show a progress bar if the size of the"},{"line_number":230,"context_line":"            # input is unknown (most likely a piped input)"},{"line_number":231,"context_line":"            fields[\u0027data\u0027] \u003d progressbar.VerboseFileWrapper("},{"line_number":232,"context_line":"                fields[\u0027data\u0027], filesize"},{"line_number":233,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a961159_5f7e426b","line":230,"updated":"2015-01-06 13:40:23.000000000","message":"An alternative would be to display a spinning progress counter if the size is unknown. That would let a user know that data chunks are being uploaded (the command is running and has not frozen).\n\nThe current patch is fine, so if you don\u0027t think the above is worth looking into just let me know.","commit_id":"90543244423692cade8673cf78fa601c9af04f1c"},{"author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"change_message_id":"a13dc490f1ae1be505e6e4aff36387535e5163e9","unresolved":false,"context_lines":[{"line_number":227,"context_line":"        filesize \u003d utils.get_file_size(fields[\u0027data\u0027])"},{"line_number":228,"context_line":"        if filesize is not None:"},{"line_number":229,"context_line":"            # NOTE(kragniz): do not show a progress bar if the size of the"},{"line_number":230,"context_line":"            # input is unknown (most likely a piped input)"},{"line_number":231,"context_line":"            fields[\u0027data\u0027] \u003d progressbar.VerboseFileWrapper("},{"line_number":232,"context_line":"                fields[\u0027data\u0027], filesize"},{"line_number":233,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a961159_7fc4de91","line":230,"in_reply_to":"3a961159_5f7e426b","updated":"2015-01-06 13:43:23.000000000","message":"That sounds like a neat idea, but maybe something for a follow up patch?","commit_id":"90543244423692cade8673cf78fa601c9af04f1c"},{"author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"change_message_id":"3424b080b4ef21f43760f3624d5d927502ebaeb7","unresolved":false,"context_lines":[{"line_number":227,"context_line":"        filesize \u003d utils.get_file_size(fields[\u0027data\u0027])"},{"line_number":228,"context_line":"        if filesize is not None:"},{"line_number":229,"context_line":"            # NOTE(kragniz): do not show a progress bar if the size of the"},{"line_number":230,"context_line":"            # input is unknown (most likely a piped input)"},{"line_number":231,"context_line":"            fields[\u0027data\u0027] \u003d progressbar.VerboseFileWrapper("},{"line_number":232,"context_line":"                fields[\u0027data\u0027], filesize"},{"line_number":233,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a961159_ff044e7f","line":230,"in_reply_to":"3a961159_7fc4de91","updated":"2015-01-06 13:57:36.000000000","message":"Sure (the difference between a bugfix and an enhancement)","commit_id":"90543244423692cade8673cf78fa601c9af04f1c"}]}
