On Mon, Apr 9, 2012 at 5:42 PM, Dirk Pranke
<dpranke@chromium.org> wrote:
3) Don't use test_expectations.txt to suppress failures across a
single cycle of the bot, just so you can gather updated baselines
without the tree going red. While it might seem that you're doing tree
maintainers a favor, in my experience this just makes things confusing
and it's better to let the tree go red until you can actually
rebaseline things. It's too easy to add a suppression "for now" and
then forget about it.
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?