diff options
| author | Oli Lan <olilan@google.com> | 2019-08-14 11:45:37 +0100 |
|---|---|---|
| committer | Oli Lan <olilan@google.com> | 2019-08-16 17:52:50 +0100 |
| commit | 0672808c27f0c0d4b8e871a170b5adfe794fd768 (patch) | |
| tree | aff8d2a7ffece5a67161db5213bb1e2980bc8fff /startop/scripts/lib/cmd_utils.py | |
| parent | 3628bb3a64d3e150c53fabe85e01ab03b2fd9681 (diff) | |
Include NewRollbacks in result of getAvailableRollbacks.
This CL adds NewRollbacks to the return value of getAvailableRollbacks,
as long as the PackageRollbackInfo is complete (i.e. there is one for
each child session in the install).
Including these should avoid the potential race condition between the
install finishing and getAvailableRollbacks being called which affected
some of the tests. This means that the draining of the handler that was
previously added to getAvailableRollbacks can now be removed.
In order to avoid a similar race condition when rollbacks are expired, this
change also cancels corresponding NewRollbacks when expireRollbackForPackage
is called.
Bug: 136241838
Bug: 136548146
Test: atest RollbackTest
Test: manual local test with delay added to onFinished in the service callback
Change-Id: I015ee5925e38118c40f4b9e145f78fb12c0e2890
Diffstat (limited to 'startop/scripts/lib/cmd_utils.py')
0 files changed, 0 insertions, 0 deletions