diff options
| author | Jhon Honce <jhonce@redhat.com> | 2020-07-31 10:11:21 -0700 |
|---|---|---|
| committer | Jhon Honce <jhonce@redhat.com> | 2020-07-31 10:22:27 -0700 |
| commit | 02eefe650c8e13cf046844080656c6d1114f1625 (patch) | |
| tree | 911006787766a65aa1ba3f3f79ef1bf0b38d7b05 | |
| parent | 7a15be546adffe4f884abfbd4ed02f69ac7659e0 (diff) | |
| download | podman-02eefe650c8e13cf046844080656c6d1114f1625.tar.gz podman-02eefe650c8e13cf046844080656c6d1114f1625.tar.bz2 podman-02eefe650c8e13cf046844080656c6d1114f1625.zip | |
Fix podman service --valink timeout
Documentation and unit files call for a millisecond timeout while the
code was using a second resolution. Code change is smaller given
varlink has been deprecated.
Signed-off-by: Jhon Honce <jhonce@redhat.com>
| -rw-r--r-- | cmd/podman/system/service.go | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/cmd/podman/system/service.go b/cmd/podman/system/service.go index 2d511f0ec..7c692b07e 100644 --- a/cmd/podman/system/service.go +++ b/cmd/podman/system/service.go @@ -49,7 +49,7 @@ func init() { flags := srvCmd.Flags() flags.Int64VarP(&srvArgs.Timeout, "time", "t", 5, "Time until the service session expires in seconds. Use 0 to disable the timeout") - flags.BoolVar(&srvArgs.Varlink, "varlink", false, "Use legacy varlink service instead of REST") + flags.BoolVar(&srvArgs.Varlink, "varlink", false, "Use legacy varlink service instead of REST. Unit of --time changes from seconds to milliseconds.") _ = flags.MarkDeprecated("varlink", "valink API is deprecated.") flags.SetNormalizeFunc(aliasTimeoutFlag) @@ -88,14 +88,15 @@ func service(cmd *cobra.Command, args []string) error { opts := entities.ServiceOptions{ URI: apiURI, - Timeout: time.Duration(srvArgs.Timeout) * time.Second, Command: cmd, } if srvArgs.Varlink { + opts.Timeout = time.Duration(srvArgs.Timeout) * time.Millisecond return registry.ContainerEngine().VarlinkService(registry.GetContext(), opts) } + opts.Timeout = time.Duration(srvArgs.Timeout) * time.Second return restService(opts, cmd.Flags(), registry.PodmanConfig()) } |