Comment 6 for bug 1759461

Note it seems to fail in different ways now:

----------------------------------------------------------------------
FAIL: showoutput_test.go:59: ShowOutputSuite.TestRun

test 0 (model flag -m): should handle wait-time formatting errors
test 0 (model flag --model): should handle wait-time formatting errors
test 1 (model flag -m): should timeout if result never comes
test 1 (model flag --model): should timeout if result never comes
test 2 (model flag -m): should pass api error through properly
test 2 (model flag --model): should pass api error through properly
test 3 (model flag -m): should fail with no tag matches
test 3 (model flag --model): should fail with no tag matches
test 4 (model flag -m): should fail with no results
test 4 (model flag --model): should fail with no results
test 5 (model flag -m): should error correctly with multiple results
test 5 (model flag --model): should error correctly with multiple results
showoutput_test.go:275:
    testRunHelper(
        c, s,
        makeFakeClient(
            t.withAPIDelay,
            t.withAPITimeout,
            t.withTags,
            t.withAPIResponse,
            params.ActionsByNames{},
            t.withAPIError),
        t.expectedErr,
        t.expectedOutput,
        t.withClientWait,
        t.withClientQueryID,
        modelFlag,
    )
showoutput_test.go:304:
    c.Check(err, gc.ErrorMatches, expectedErr)
... value = nil
... regex string = "too many results for action
f47ac10b-58cc-4372-a567-0e02b2c3d479"
... Error value is nil

On Wed, Mar 28, 2018 at 2:30 PM, John A Meinel <email address hidden>
wrote:

> looks like this was already modified in develop where it was changed to
> be strictly a 3s timeout instead of 4s.
>
>
> ** Changed in: juju
> Status: In Progress => Fix Committed
>
> ** Changed in: juju
> Milestone: None => 2.3.6
>
> ** Changed in: juju
> Assignee: (unassigned) => John A Meinel (jameinel)
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1759461
>
> Title:
> intermittent failure github.com/juju/juju/cmd/juju/action
> ShowOutputSuite.TestRun
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/juju/+bug/1759461/+subscriptions
>