ACK: [SRU][B][X][PATCH v2 0/1] Force-pass the passage of time test in

[Tim Gardner]

On 2/20/23 5:58 AM, Andrei Gherzan wrote:
> Buglink: https://bugs.launchpad.net/bugs/1814234
> 
> SRU Justification:
> 
> [Impact]
> 
> * This test can fail selftest on various test hosts in our testing
>    infrastructure forcing us to keep hinting releases. This leads to
>    maintenance burden for a test that is unreliable as it depends on the
>    context of the running host.
> 
> [Fix]
> 
> * Force-pass the test included by upstream in
>    0b63accf87225b5eb7e52814c374cf02d733d4bb. We have chosen to keep this
>    test but to force its result unconditionally successful as opposed to
>    reverting the aforementioned commit because:
>    1. it keeps running the test with results and data that we can use in
>       the future for any "smarter" way to deal with it - eg. statistical
>       approach
>    2. it minimizes the maintenance burden of this patch kept in our
>       kernel trees
> 
> [Test Plan]
> 
> * It was tested in clean VMs.
> 
> [Where problems could occur]
> 
> * No regression expected and the scope is in this specific selftest.
> 
> v2:
> * fix Buglink reference
> 
> Andrei Gherzan (1):
>    UBUNTU: SAUCE: selftest: rtctest: Force passing unreliable subtest
> 
>   tools/testing/selftests/timers/rtctest.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
Acked-by: Tim Gardner <tim.gardner at canonical.com>
-- 
-----------
Tim Gardner
Canonical, Inc