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.

Sign up for GitHub

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

CNV abrupt shutdown cluster while VMs in different state. #11088

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

CNV abrupt shutdown cluster while VMs in different state. #11088

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
434fb60
[WIP] CNV abrupt shutdown cluster while VMs in different state.
avd-sagare Jan 1, 2025
29bbef8
corrected errors
avd-sagare Jan 2, 2025
b9c7e37
added checksum fix
avd-sagare Jan 3, 2025
7e66cb4
added check for vm status after restart
avd-sagare Jan 3, 2025
53a7708
changed parameterized test for ordered and abrupt shutdown as well
avd-sagare Jan 7, 2025
4bb134f
Incorporated comments
avd-sagare Jan 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
added check for vm status after restart 
Signed-off-by: Avdhoot <asagare@redhat.com>
  • Loading branch information
@avd-sagare
avd-sagare committed Jan 15, 2025
commit 7e66cb4116598fa136df0a7abb4ec07b34f243c9
12 changes: 8 additions & 4 deletions tests/functional/workloads/cnv/test_cnv_shutdown_start.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,14 +27,14 @@ class TestVmShutdownStart(E2ETest):
@pytest.mark.polarion_id("OCS-6304")
def test_vm_abrupt_shutdown_cluster(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIUC, you are doing both garceful and abrupt with parametrize, please update the test func name accordingly

self,
multi_cnv_workload,
nodes,
setup_cnv,
project_factory,
cnv_workload,
multi_cnv_workload,
clone_vm_workload,
setup_cnv,
snapshot_factory,
snapshot_restore_factory,
cnv_workload,
nodes,
):
"""
This test performs the behaviour of VMs and data integrity after abrupt shutdown of cluster
Expand Down Expand Up @@ -158,6 +158,10 @@ def test_vm_abrupt_shutdown_cluster(
all_vms[1].start()
all_vms[2].start()

# Verifies vm status after start and ssh connectivity
all_vms[1].verify_vm(verify_ssh=True)
all_vms[2].verify_vm(verify_ssh=True)

# Perform post restart data integrity check
for vm_obj in all_vms:
new_csum = cal_md5sum_vm(vm_obj=vm_obj, file_path=file_paths[0])
Expand Down