<div class="gmail_quote">On Mon, Apr 9, 2012 at 5:42 PM, Dirk Pranke <span dir="ltr"><<a href="mailto:dpranke@chromium.org">dpranke@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

3) Don't use test_expectations.txt to suppress failures across a<br>
single cycle of the bot, just so you can gather updated baselines<br>
without the tree going red. While it might seem that you're doing tree<br>
maintainers a favor, in my experience this just makes things confusing<br>
and it's better to let the tree go red until you can actually<br>
rebaseline things. It's too easy to add a suppression "for now" and<br>
then forget about it.<br></blockquote><div><br></div><div>IIUC, this is not possible if you are landing the change with the commit queue, as it will see the failing tests and CQ-? Also, I've been told explicitly by reviewers to mark the test fails with a bug filed against it, rather than commit something that breaks the tree. Would you see this as acceptable behaviour given the bug filed against it and in a comment in test_expectations?</div>

</div>