----- "Lucas Meneghel Rodrigues" <l...@redhat.com> wrote:
> system_reset, system_powerdown and shutdown *must*
> be the last ones defined, since the effect of such
> tests can leave the VM on a bad state.
>
> Signed-off-by: Lucas Meneghel Rodrigues <l...@redhat.com>
> ---
> client/tests/kvm/tests_base.cfg.sample | 35
> +++++++++++++++++--------------
> 1 files changed, 19 insertions(+), 16 deletions(-)
>
> diff --git a/client/tests/kvm/tests_base.cfg.sample
> b/client/tests/kvm/tests_base.cfg.sample
> index 8a7a13d..fdd04cb 100644
> --- a/client/tests/kvm/tests_base.cfg.sample
> +++ b/client/tests/kvm/tests_base.cfg.sample
> @@ -236,6 +236,24 @@ variants:
> - fmt_raw:
> image_format_stg = raw
>
> + - vlan_tag: install setup unattended_install
> + type = vlan_tag
> + # subnet should not be used by host
> + subnet = 192.168.123
> + vlans = "10 20"
> + nic_mode = tap
> + vms += " vm2"
> + extra_params_vm1 += " -snapshot"
> + extra_params_vm2 += " -snapshot"
> + kill_vm_gracefully_vm2 = no
> + address_index_vm2 = 1
> +
> + - physical_resources_check: install setup unattended_install
> + type = physical_resources_check
> + catch_uuid_cmd = dmidecode | awk -F: '/UUID/ {print $2}'
> +
> + # system_reset, system_powerdown and shutdown *must* be the last
> ones
> + # defined, since the effect of such tests can leave the VM on a
> bad state.
I think the order should be system_powerdown, system_reset, shutdown.
We should minimize the number of tests running after system_reset.
> - system_reset: install setup unattended_install
> type = boot
> reboot_method = system_reset
> @@ -254,22 +272,7 @@ variants:
> shutdown_method = shell
> kill_vm = yes
> kill_vm_gracefully = no
> -
> - - vlan_tag: install setup unattended_install
> - type = vlan_tag
> - # subnet should not be used by host
> - subnet = 192.168.123
> - vlans = "10 20"
> - nic_mode = tap
> - vms += " vm2"
> - extra_params_vm1 += " -snapshot"
> - extra_params_vm2 += " -snapshot"
> - kill_vm_gracefully_vm2 = no
> - address_index_vm2 = 1
> -
> - - physical_resources_check: install setup unattended_install
> - type = physical_resources_check
> - catch_uuid_cmd = dmidecode | awk -F: '/UUID/ {print $2}'
> + # Do not define test variants below shutdown
>
> # NICs
> variants:
> --
> 1.6.6
>
> _______________________________________________
> Autotest mailing list
> autot...@test.kernel.org
> http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
