Bug #17682
closed[arvados-client] shell stutter
100%
Description
When connecting via a container request UUID, the "connecting to container ..." line is printed twice:
$ ./arvados-client shell ce8i5-xvhdp-e6wnujfslyyqn4b connecting to container ce8i5-dz642-h1cl0sa62d4i430 connecting to container ce8i5-dz642-h1cl0sa62d4i430 root@0f13dcd755fa:~#
When using the container request, nothing like that is printed:
$ ./arvados-client shell ce8i5-dz642-h1cl0sa62d4i430 root@0f13dcd755fa:~#
Updated by Ward Vandewege over 3 years ago
- Related to Story #17207: External access to web services running in containers added
Updated by Tom Clegg over 3 years ago
As a side effect, this fix also avoids an extra API call and avoids the possibility, when a container request UUID is supplied, of setting up a "probe" tunnel to container A and then (due to cancel/retry) setting up the real tunnel to a different container B. (This seems pretty harmless but could produce mildly confusing results.)
17682-fix-stutter @ e9ab3d864a8deb2810d7a49b0529a2aef0c0b41c -- developer-run-tests: #2480
Updated by Tom Clegg over 3 years ago
- Status changed from New to In Progress
- Assigned To set to Tom Clegg
Updated by Ward Vandewege over 3 years ago
Tom Clegg wrote:
As a side effect, this fix also avoids an extra API call and avoids the possibility, when a container request UUID is supplied, of setting up a "probe" tunnel to container A and then (due to cancel/retry) setting up the real tunnel to a different container B. (This seems pretty harmless but could produce mildly confusing results.)
17682-fix-stutter @ e9ab3d864a8deb2810d7a49b0529a2aef0c0b41c -- developer-run-tests: #2480
LGTM, thanks!
Updated by Tom Clegg over 3 years ago
- Status changed from In Progress to Resolved
Applied in changeset arvados|ba56503f90d099a215fb7375f5cb1cc1ac667e2c.