Re: [spice 3/8] tests: migrate: add support to run with remote-viewer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Wed, Sep 25, 2019 at 05:29:56AM -0400, Frediano Ziglio wrote:
> > 
> > From: Victor Toso <me@xxxxxxxxxxxxxx>
> > 
> > Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx>
> > ---
> >  tests/migrate.py | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> > 
> > diff --git a/tests/migrate.py b/tests/migrate.py
> > index 5a0f5930..f893b2ac 100755
> > --- a/tests/migrate.py
> > +++ b/tests/migrate.py
> > @@ -58,7 +58,7 @@ def get_args():
> >      parser.add_argument('--qemu', dest='qemu',
> >      default='../../qemu/x86_64-softmmu/qemu-system-x86_64')
> >      parser.add_argument('--log_filename', dest='log_filename',
> >      default='migrate.log')
> >      parser.add_argument('--image', dest='image', default='')
> > -    parser.add_argument('--client', dest='client', default='spicy',
> > choices=['spicy'])
> > +    parser.add_argument('--client', dest='client', default='spicy',
> > choices=['spicy', 'remote-viewer'])
> >      parser.add_argument('--vdagent', choices=['on', 'off'], default='on')
> >      args = parser.parse_args(sys.argv[1:])
> >      if os.path.exists(args.qemu):
> > @@ -97,6 +97,9 @@ def start_qemu(qemu_exec, image, spice_port, qmp_filename,
> > incoming_port=None, e
> >  
> >  def start_client(client, spice_port):
> >      client_cmd = "spicy --uri spice://localhost:%s" % (spice_port)
> > +    if client == "remote-viewer":
> > +        client_cmd = "%s spice://localhost:%s" % (client, spice_port)
> 
> Maybe here can just be
> 
>   client_cmd = "remote-viewer spice://localhost:%s" % (spice_port)

Ah, yes, before it was != "spicy" but I changed.
Either way is fine for me.

> > +
> >      return Popen(client_cmd.split(), executable=client)
> >  
> >  def wait_active(q, active):
> 
> Otherwise,
>   Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> 
> Frediano

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]