Page MenuHomePhabricator

When a Tester run fails for an Implementation, automatically de-approve said Implementation from its function
Open, HighPublic

Description

When an "approved" (connected) Tester is run for a Function against an approved Implementation, it should always pass. If it fails (i.e. votes false, not returns an error that might be temporary), the system is in an inconsistent state. To fix this, we must automatically de-approve the failing Implementation.

[TBD: If the Tester fails all the Implementations, should it instead be disabled? How do we know what to do when?]

Event Timeline

I am worried that due to a hiccup in the infra, we might end up descoping all implementations. Let's descope this one for now to get a better feel for how stable the tests run.

In order to allow for that, T319478 is being extended to sort by correctness and speed.