Acked-by: Jiří Župka <[email protected]>

----- Original Message -----
> Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>
> ---
>  client/virt/kvm_vm.py | 11 +++++------
>  1 file changed, 5 insertions(+), 6 deletions(-)
> 
> diff --git a/client/virt/kvm_vm.py b/client/virt/kvm_vm.py
> index bd8046e..fe8c259 100644
> --- a/client/virt/kvm_vm.py
> +++ b/client/virt/kvm_vm.py
> @@ -2528,10 +2528,10 @@ class VM(virt_vm.BaseVM):
>          # For compatibility with versions of QEMU that do not
>          recognize all
>          # key names: replace keyname with the hex value from the
>          dict, which
>          # QEMU will definitely accept
> -        dict = {"comma": "0x33",
> -                "dot":   "0x34",
> -                "slash": "0x35"}
> -        for key, value in dict.items():
> +        key_mapping = {"comma": "0x33",
> +                       "dot":   "0x34",
> +                       "slash": "0x35"}
> +        for key, value in key_mapping.items():
>              keystr = keystr.replace(key, value)
>          self.monitor.sendkey(keystr)
>          time.sleep(0.2)
> @@ -2557,7 +2557,7 @@ class VM(virt_vm.BaseVM):
>          logging.debug("Saving VM %s to %s" % (self.name, path))
>          # Can only check status if background migration
>          self.monitor.migrate("exec:cat>%s" % path, wait=False)
> -        result = utils_misc.wait_for(
> +        utils_misc.wait_for(
>              # no monitor.migrate-status method
>              lambda : "status: completed" in
>              self.monitor.info("migrate"),
>              self.MIGRATE_TIMEOUT, 2, 2,
> @@ -2567,7 +2567,6 @@ class VM(virt_vm.BaseVM):
>          self.monitor.migrate_set_downtime(0.03)
>          # Base class defines VM must be off after a save
>          self.monitor.cmd("system_reset")
> -        state = self.monitor.get_status()
>          self.verify_status('paused') # Throws exception if not
>  
>      def restore_from_file(self, path):
> --
> 1.7.11.4
> 
> _______________________________________________
> Autotest-kernel mailing list
> [email protected]
> https://www.redhat.com/mailman/listinfo/autotest-kernel
> 

_______________________________________________
Autotest-kernel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/autotest-kernel

Reply via email to