<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 12, 2018 at 9:39 AM, Tim Northover <span dir="ltr"><<a href="mailto:tnorthover@apple.com" target="_blank">tnorthover@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>> Marking them unsupported on Windows doesn't sound ideal either since<br>
> the tests work fine on Windows, it's just removing the temp files that<br>
> doesn’t.<br>
<br>
</span>I think a test leaving 8GB of stray data isn’t working no matter where it exists, and there’s no guarantee lit would be any more capable of deleting these files. Something in Windows (or maybe just that bot)</blockquote><div><br></div><div>It's flakily failing on several Windows bots both on the LLVM and Chromium waterfalls, so it's not bot-specific.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">is fundamentally broken here and shouldn’t be coddled.<br></blockquote><div><br></div><div>On Windows, a file can't be deleted if something still holds a reference to it. This is probably somehow related to that.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I’m OK with reverting while we decide a plan, but not OK with leaving it as the status-quo. There’s a good chance this is wasting petabytes of data globally, which is not reasonable for such a trivial test.<br></blockquote><div><br></div><div>If the test creates 8GB of data, that seems kind of inefficient even if it's deleted right after. Igor, is it possible to make these tests use 2-4 orders of magnitude less data? If not, maybe the tests cost more than they help and we should just delete them?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Perhaps we could add a —dont-care option to not so that failure doesn’t nuke the whole test?<br></blockquote><div><br></div><div>Cleaning up after tests seems like something lit should do; it feels inefficient if we need to do this manually for every test.</div></div></div></div>