Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reconstruct MU guest migration test: merge chained tests into combined single testsuite #19568

Merged

Conversation

alice-suse
Copy link
Contributor

@alice-suse alice-suse commented Jun 20, 2024

This PR mainly implements A3 task https://progress.opensuse.org/issues/160850, so it does mainly below:

  • move below testsuites into one kvm testsuite, eg with name qam-kvm-migration-src
    qam_virt_kvm_install_host_migration_src
    qam_virt_kvm_install_guests_migration_src
    qam_virt_kvm_guests_migration_src
  • move below testsuites into one kvm testsuite, eg with name qam-kvm-migration-dst
    qam_virt_kvm_install_host_migration_dst
    qam_virt_kvm_install_guests_migration_dst
    qam_virt_kvm_guests_migration_dst
  • qam-kvm-migration-src has PARALLEL_WITH relation with qam-kvm-migration-dst
  • move below testsuites into one xen testsuite, eg with name qam-xen-migration-src
    qam_virt_kvm_install_host_migration_src
    qam_virt_kvm_install_guests_migration_src
    qam_virt_kvm_guests_migration_src
  • move below testsuites into one xen testsuite, eg with name qam-xen-migration-dst
    qam_virt_xen_install_host_migration_dst
    qam_virt_xen_install_guests_migration_dst
    qam_virt_xen_guests_migration_dst
  • qam-xen-migration-src has PARALLEL_WITH relation with qam-xen-migration-dst
  • within each testsuite, it supports switch settings to enable or disable host installation, guest installation.

Job Group Yaml file(in development group now, will be updated to official job group when PR merged):

Verification run:

verification run on http://10.67.129.77/

PASS
qa2-dhcp-77:/var/lib/openqa/tests/sle # openqa-cli api  -X POST isos ARCH="x86_64" DISTRI="sle" VERSION="15-SP5" FLAVOR=Server-DVD-Virt-KVM-GuestMigration-Incidents TERADATA=  PATCH_WITH_ZYPPER=0  GUEST_ADMINISTRATION=0 INTERVAL_LOG=0 TEST=qam-kvm-migration-src,qam-kvm-migration-dst BUILD=":123456:qemu"
{"count":2,"failed":[],"ids":[769,770],"scheduled_product_id":315}

PASS
qa2-dhcp-77:/var/lib/openqa/tests/sle # openqa-cli api  -X POST isos ARCH="x86_64" DISTRI="sle" VERSION="15-SP5" FLAVOR=Server-DVD-Virt-XEN-GuestMigration-Incidents TERADATA=  PATCH_WITH_ZYPPER=0  GUEST_ADMINISTRATION=0 INTERVAL_LOG=0 TEST=qam-xen-migration-src,qam-xen-migration-dst BUILD=":123456:xen"
{"count":2,"failed":[],"ids":[777,778],"scheduled_product_id":316}

PASS
qa2-dhcp-77:/var/lib/openqa/tests/sle # openqa-cli api  -X POST isos ARCH="x86_64" DISTRI="sle" VERSION="15-SP5" FLAVOR=Server-DVD-Virt-KVM-GuestMigration-Incidents TERADATA=  PATCH_WITH_ZYPPER=0  GUEST_ADMINISTRATION=0 INTERVAL_LOG=0 TEST=qam-kvm-migration-src,qam-kvm-migration-dst BUILD=":123456:libvirt"
{"count":2,"failed":[],"ids":[779,780],"scheduled_product_id":321}

PASS(PATCH_WITH_ZYPPER=1, UPDATE_PACKAGE=libvirt, ignore the last migration sub-test failure)
http://10.67.129.77/tests/801#
http://10.67.129.77/tests/802#


WIP  (PATCH_WITH_ZYPPER=1,UPDATE_PACKAGE=xen)
http://openqa.qam.suse.cz/tests/77898
http://openqa.qam.suse.cz/tests/77897

Copy link

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files.

@alice-suse alice-suse force-pushed the alice-migration-base-on-roy-A1 branch 2 times, most recently from 0731ff7 to 8e074a0 Compare June 24, 2024 02:20
@alice-suse alice-suse changed the title [WIP] Alice migration base on roy a1 [WIP] Reconstruct MU guest migration test: merge 3 into 1 Jun 24, 2024
@alice-suse alice-suse force-pushed the alice-migration-base-on-roy-A1 branch 2 times, most recently from 292bf79 to 4c69c5d Compare June 24, 2024 03:04
@alice-suse alice-suse force-pushed the alice-migration-base-on-roy-A1 branch 2 times, most recently from 77ff7f9 to 8d01ab1 Compare July 15, 2024 02:42
@alice-suse alice-suse changed the title [WIP] Reconstruct MU guest migration test: merge 3 into 1 Reconstruct MU guest migration test: merge 3 into 1 Jul 15, 2024
@alice-suse alice-suse changed the title Reconstruct MU guest migration test: merge 3 into 1 Reconstruct MU guest migration test: merge chained tests into combined single testsuite Jul 15, 2024
@alice-suse alice-suse force-pushed the alice-migration-base-on-roy-A1 branch 2 times, most recently from dc9554d to daa8e9c Compare July 15, 2024 10:27
@alice-suse alice-suse force-pushed the alice-migration-base-on-roy-A1 branch from 026c4ff to 65c0f2c Compare July 16, 2024 05:49
@alice-suse
Copy link
Contributor Author

@RoyCai7 @tbaev @martinsmarcelo @waynechen55 @Julie-CAO Hello guys, welcome to review!

@RoyCai7
Copy link
Contributor

RoyCai7 commented Jul 16, 2024

LGTM

@RoyCai7
Copy link
Contributor

RoyCai7 commented Jul 22, 2024

Current command to trigger migration test is
openqa-cli api --host http://openqa.qam.suse.cz -X POST isos ARCH="x86_64" DISTRI="sle" VERSION="15-SP5" FLAVOR="Server-DVD-Virt-XEN-GuestMigration-Incidents" BUILD="::live_migrat" INCIDENT_ID="34724" WORKER_CLASS=live_migrat TERADATA=
it does not set specify test, so if new incident is coming, it will also trigger test:qam-xen-migration-src and qam-xen-migration-dst.
It is best to merge this PR asap.

@alice-suse
Copy link
Contributor Author

Any more review from anyone? If no, consider to merge by EOB today. @RoyCai7 Please help with the merge action! Thanks.

@RoyCai7 RoyCai7 merged commit f1a74c2 into os-autoinst:master Jul 22, 2024
10 checks passed
@martinsmarcelo
Copy link
Contributor

no comments, LGTM

@alice-suse
Copy link
Contributor Author

@tbaev Notice that you are testing incidents this week. Please be aware of this PR merge. From next job, it will be using the new migration testsuites. Let me know if any problems in https://progress.opensuse.org/issues/160850.

@tbaev
Copy link
Contributor

tbaev commented Jul 22, 2024

@alice-suse thanks for the heads up, I am aware of the PR and okay with the merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants