Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 8523ea02 authored by Michael Eastwood's avatar Michael Eastwood
Browse files

Bump the timeout on IDumpstateDevice 

We're seeing increasingly frequent timeouts due to what appears to be
organic growth in the amount data dumped. These timeouts severely
restrict our ability to triage certain kinds of bugs, so upping the
timeout here.

Bug: 223417101
Test: No timeouts seen locally.
Change-Id: I75aceef2eaa78e5e20f6f27e76b62131f781a14f
parent 2362edf7

cmds/dumpstate/dumpstate.cpp

+1 −1
Original line number Original line Diff line number Diff line
@@ -2418,7 +2418,7 @@ void Dumpstate::DumpstateBoard(int out_fd) { 
    // Given that bugreport is required to diagnose failures, it's better to set an arbitrary amount

    // Given that bugreport is required to diagnose failures, it's better to set an arbitrary amount

    // of timeout for IDumpstateDevice than to block the rest of bugreport. In the timeout case, we

    // of timeout for IDumpstateDevice than to block the rest of bugreport. In the timeout case, we

    // will kill the HAL and grab whatever it dumped in time.

    // will kill the HAL and grab whatever it dumped in time.

    constexpr size_t timeout_sec = 30;

    constexpr size_t timeout_sec = 45;





    if (dumpstate_hal_handle_aidl != nullptr) {

    if (dumpstate_hal_handle_aidl != nullptr) {

        DoDumpstateBoardAidl(dumpstate_hal_handle_aidl, dumpstate_fds, options_->bugreport_mode,

        DoDumpstateBoardAidl(dumpstate_hal_handle_aidl, dumpstate_fds, options_->bugreport_mode,