<HTML><BODY><div>Sergey, thanks for review. </div><div> </div><div>The reason is that each temporary docker image is about 5 GB and we don’t have space to keep it localy.</div><div>I will update commit message.</div><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Среда, 20 мая 2020, 11:31 +03:00 от Sergey Bronnikov <sergeyb@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><style type="text/css"></style><div><div id="style_15899634611519935494_BODY">Hi, Oleg!<br><br>generally LGTM, but see my question regarding commit message.<br><br>On 03:36 Wed 20 May , Oleg Piskunov wrote:<div class="mail-quote-collapse">><br>><br>> - add cleanup after perf docker<br>> image preparation (perf_bootstrap job).<br>> - cleanup perf docker images on execution hosts<br>> after perf testing (cleanup stage).<br>> <br>> Closes #5003</div><br>There is a best practice to explain 'what' and 'why' in a commit message<br>body. It is unclear why we should cleanup these images and cannot reuse<br>existed images.<br><br><snipped></div></div></div></div></blockquote><div> <div> </div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Oleg Piskunov</div></div></div><div> </div></div></BODY></HTML>