)]}'
{"tripleo_common/image/image_uploader.py":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"ce70d17ce9f7edd6bdd2f802b0e7c0f3c12ad43f","unresolved":false,"context_lines":[{"line_number":2214,"context_line":"            layer_digest[\u0027digest\u0027] \u003d layer"},{"line_number":2215,"context_line":"            # podman currently doesn\u0027t do compressed layers so just use"},{"line_number":2216,"context_line":"            # the diff-size"},{"line_number":2217,"context_line":"            layer_digest[\u0027size\u0027] \u003d layers[layer][\u0027diff-size\u0027]"},{"line_number":2218,"context_line":"            manifest[\u0027layers\u0027].append(layer_digest)"},{"line_number":2219,"context_line":"        return manifest"},{"line_number":2220,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1fa4df85_567c4cb4","line":2217,"updated":"2020-03-02 15:48:50.000000000","message":"in _copy_layer_local_to_registry we layer_entry.get(\u0027diff-size\u0027). So I assume it can be None. If so, this code will raise an uncaught exception","commit_id":"2950738526d7b93553102ceb77717d31c445caa2"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"46057d14fbb8900b96b2dcd798de269116ba1646","unresolved":false,"context_lines":[{"line_number":2214,"context_line":"            layer_digest[\u0027digest\u0027] \u003d layer"},{"line_number":2215,"context_line":"            # podman currently doesn\u0027t do compressed layers so just use"},{"line_number":2216,"context_line":"            # the diff-size"},{"line_number":2217,"context_line":"            layer_digest[\u0027size\u0027] \u003d layers[layer][\u0027diff-size\u0027]"},{"line_number":2218,"context_line":"            manifest[\u0027layers\u0027].append(layer_digest)"},{"line_number":2219,"context_line":"        return manifest"},{"line_number":2220,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1fa4df85_6aaf1ce3","line":2217,"in_reply_to":"1fa4df85_567c4cb4","updated":"2020-03-02 15:57:08.000000000","message":"It\u0027s unlikely it\u0027ll be None since that\u0027s the standard storage size.  I just tend to use .get(). We could add an additional check but podman doesn\u0027t store things compressed so i think this should always be set","commit_id":"2950738526d7b93553102ceb77717d31c445caa2"}]}