Page MenuHomePhabricator

[dagster-aws] Increase timeout on dagster-aws info CLI command
ClosedPublic

Authored by schrockn on Oct 15 2019, 3:10 AM.

Details

Summary

I think this is responsible for dagster-aws info sometimes reporting an instance as down, when the response takes longer than 100ms.

Test Plan

manually tested

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

natekupp created this revision.Oct 15 2019, 3:10 AM
natekupp retitled this revision from [dagster-aws] Increase timeout on dagster-aws info CLI command to 500ms to [dagster-aws] Increase timeout on dagster-aws info CLI command.Oct 15 2019, 3:15 AM
natekupp edited the summary of this revision. (Show Details)
natekupp added a reviewer: Restricted Project.
natekupp updated this revision to Diff 5843.Oct 15 2019, 3:15 AM
natekupp edited the summary of this revision. (Show Details)

up

schrockn commandeered this revision.Oct 15 2019, 12:29 PM
schrockn edited reviewers, added: natekupp; removed: schrockn.
This revision was not accepted when it landed; it landed in state Needs Review.Oct 15 2019, 12:30 PM
This revision was automatically updated to reflect the committed changes.