)]}'
{"ironic/drivers/modules/image_cache.py":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"f01ed62579920286c4de280d2282a984355fa387","unresolved":true,"context_lines":[{"line_number":256,"context_line":"            else:"},{"line_number":257,"context_line":"                survived.append((file_name, last_used, stat))"},{"line_number":258,"context_line":"        if count:"},{"line_number":259,"context_line":"            LOG.debug(\u0027Removed %(count)d expired file(s) from %(dir)s\u0027,"},{"line_number":260,"context_line":"                      {\u0027count\u0027: count, \u0027dir\u0027: self.master_dir})"},{"line_number":261,"context_line":"        return survived, amount"},{"line_number":262,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"e8bf947f_d2a75f79","line":259,"updated":"2022-08-09 16:05:37.000000000","message":"If you\u0027re logging at debug, seems like a missed opportunity to just log the names of deleted files, too.","commit_id":"e48c62187ca15e130c35b421e5e311eea9a334db"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"acba836906c7b60caee8b7bec0c1f7f6b4cebc6b","unresolved":true,"context_lines":[{"line_number":256,"context_line":"            else:"},{"line_number":257,"context_line":"                survived.append((file_name, last_used, stat))"},{"line_number":258,"context_line":"        if count:"},{"line_number":259,"context_line":"            LOG.debug(\u0027Removed %(count)d expired file(s) from %(dir)s\u0027,"},{"line_number":260,"context_line":"                      {\u0027count\u0027: count, \u0027dir\u0027: self.master_dir})"},{"line_number":261,"context_line":"        return survived, amount"},{"line_number":262,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"67ae80d0_cd506e78","line":259,"in_reply_to":"800c3eeb_e09cbcef","updated":"2022-08-09 16:46:08.000000000","message":"Yeah, I could easily go either way. Usually OpenStack in debug is so verbose I just prefer being more verbose over less, but if someone is running on prod, in debug, I could see this getting ugly for sure.","commit_id":"e48c62187ca15e130c35b421e5e311eea9a334db"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"05fd3c92cbbdceca609884d6dbf6d777e8c911ff","unresolved":true,"context_lines":[{"line_number":256,"context_line":"            else:"},{"line_number":257,"context_line":"                survived.append((file_name, last_used, stat))"},{"line_number":258,"context_line":"        if count:"},{"line_number":259,"context_line":"            LOG.debug(\u0027Removed %(count)d expired file(s) from %(dir)s\u0027,"},{"line_number":260,"context_line":"                      {\u0027count\u0027: count, \u0027dir\u0027: self.master_dir})"},{"line_number":261,"context_line":"        return survived, amount"},{"line_number":262,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"800c3eeb_e09cbcef","line":259,"in_reply_to":"e8bf947f_d2a75f79","updated":"2022-08-09 16:14:04.000000000","message":"I\u0027m afraid about large bursts of deployments, which could result in hundreds of files logged here afterwards.","commit_id":"e48c62187ca15e130c35b421e5e311eea9a334db"}]}