[keyserver] Increase timeout for getInitialReduxState
ClosedPublic
Actions

Authored by ashoat on Sep 20 2023, 11:57 AM.

Details

Reviewers

michal
ginsu
atul

Commits

rCOMM9595f5ed54d1: [keyserver] Increase timeout for getInitialReduxState

Summary

The default timeout was too short, which was causing ENG-4991.

Test Plan

I monkey-patched this onto prod and it solved the issue

Diff Detail

Repository

rCOMM Comm

Branch

ashoat/fixmichal

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Sep 20 2023, 11:57 AM

Herald added a subscriber: tomek. · View Herald TranscriptSep 20 2023, 11:57 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Sep 20 2023, 12:35 PM

Harbormaster failed remote builds in B22766: Diff 31324!

beginning review now bc android CI failed + is slow

beginning review now bc android CI failed + is slow

Nvm I thought there was an option for that...

Restarted CI and will review once it passes

ashoat requested review of this revision.Sep 20 2023, 1:19 PM

ginsu accepted this revision.Sep 20 2023, 1:20 PM

This revision is now accepted and ready to land.Sep 20 2023, 1:20 PM

Harbormaster completed remote builds in B22766: Diff 31324.Sep 20 2023, 2:01 PM

Closed by commit rCOMM9595f5ed54d1: [keyserver] Increase timeout for getInitialReduxState (authored by ashoat). · Explain WhySep 20 2023, 2:32 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM9595f5ed54d1: [keyserver] Increase timeout for getInitialReduxState.

Revision Contents
Changeset List

Path

Size

web/

redux/

action-types.js

2 lines

Diff 31324

View Options

[keyserver] Increase timeout for getInitialReduxStateClosedPublicActions