Stretch cluster + CNV automation #10077

mashetty330 · 2024-07-11T11:56:57Z

No description provided.

tests/functional/disaster-recovery/sc_arbiter/test_netsplit.py

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-stj29
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_netsplit.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job UNSTABLE (some or all tests failed).

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-stj29
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_netsplit.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job FAILED (installation failed, tests not executed).

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-stj30
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_netsplit.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job UNSTABLE (some or all tests failed).

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-stj30
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_netsplit.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job UNSTABLE (some or all tests failed).

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-stj30
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_netsplit.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job UNSTABLE (some or all tests failed).

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-stj30
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_netsplit.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job UNSTABLE (some or all tests failed).

PrasadDesala · 2024-08-05T13:44:38Z

ocs_ci/deployment/cnv.py

+        cnv_namespace.delete(resource_name=constants.CNV_NAMESPACE)
+        logger.info(f"Deleted the namespace {constants.CNV_NAMESPACE}")
+
+    def cleanup_cnv(self, check_cnv_installed=False):


can we rename it to uninstall_cnv

ocs_ci/deployment/cnv.py

PrasadDesala · 2024-08-05T13:52:04Z

tests/conftest.py

+        volumeimportsource_obj = get_volumeimportsource(pvc_obj=pvc_obj)
+        vm_obj.delete()
+        log.info(f"Successfully deleted VM {vm_obj.name}")
+        pvc_obj.delete()


we do not need to wait for the resource to delete?

by default wait=True, so delete itself makes sure the resource is deleted. correct?

No, the wait here is for the delete command to complete. It will not wait for the resource to delete.
Please check https://github.com/mashetty330/ocs-ci/blob/cdc6601c397f41b37d0c2522de0a94035a2a8ae4/ocs_ci/ocs/ocp.py#L434

def delete(self): """ Delete the VirtualMachine """ self.vm_ocp_obj.delete(resource_name=self._vm_name) self.vm_ocp_obj.wait_for_delete(resource_name=self._vm_name, timeout=180)

PrasadDesala · 2024-08-05T13:54:32Z

tests/conftest.py

+
+
+@pytest.fixture()
+def setup_vms_standalone_pvc(request, project_factory, setup_cnv):


what is the need to pass the setup_cnv here?

PrasadDesala · 2024-08-05T13:55:43Z

tests/conftest.py

+    vm_obj = None
+
+    def factory():
+        nonlocal vm_obj


doc string missing

ocs_ci/deployment/cnv.py

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-sts2
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_zone_shutdown_and_crash.py
Additional Test Params:
OCP VERSION: 4.16
OCS VERSION: 4.16
tested against branch: master

Job UNSTABLE (some or all tests failed).

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-sts2
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_zone_shutdown_and_crash.py:: TestZoneShutdownsAndCrashes::test_zone_crashes
Additional Test Params:
OCP VERSION: 4.16
OCS VERSION: 4.16
tested against branch: master

Job FAILED (installation failed, tests not executed).

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-sts2
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_zone_shutdown_and_crash.py:: TestZoneShutdownsAndCrashes::test_zone_crashes
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job FAILED (installation failed, tests not executed).

Signed-off-by: Mahesh Shetty <[email protected]>

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-sts2
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_zone_shutdown_and_crash.py:: TestZoneShutdownsAndCrashes::test_zone_crashes
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job FAILED (installation failed, tests not executed).

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-sts2
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_zone_shutdown_and_crash.py::TestZoneShutdownsAndCrashes::test_zone_crashes
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job UNSTABLE (some or all tests failed).

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-sts2
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_zone_shutdown_and_crash.py::TestZoneShutdownsAndCrashes::test_zone_crashes
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job PASSED.

ocs-ci

PR validation on existing cluster

Cluster Name: mashetty-sts2
Cluster Configuration:
PR Test Suite:
PR Test Path: tests/functional/disaster-recovery/sc_arbiter/test_zone_shutdown_and_crash.py::TestZoneShutdownsAndCrashes::test_zone_shutdowns
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master

Job PASSED.

openshift-ci · 2024-09-06T05:13:27Z

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Akarsha-rai, mashetty330, PrasadDesala

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

mashetty330 added the team/e2e E2E team related issues/PRs label Jul 11, 2024

mashetty330 self-assigned this Jul 11, 2024

openshift-ci bot added the do-not-merge/work-in-progress label Jul 11, 2024

pull-request-size bot added size/M PR that changes 30-99 lines size/L PR that changes 100-499 lines and removed size/M PR that changes 30-99 lines labels Jul 11, 2024

mashetty330 force-pushed the sc_cnv branch from 6e00a1f to 259c750 Compare July 16, 2024 04:25

hnallurv reviewed Jul 17, 2024

View reviewed changes

tests/functional/disaster-recovery/sc_arbiter/test_netsplit.py Outdated Show resolved Hide resolved

mashetty330 force-pushed the sc_cnv branch from 259c750 to de2a6d5 Compare July 18, 2024 07:44

mashetty330 marked this pull request as ready for review July 26, 2024 09:15

mashetty330 requested review from a team as code owners July 26, 2024 09:15

openshift-ci bot removed the do-not-merge/work-in-progress label Jul 26, 2024

mashetty330 requested review from PrasadDesala and Akarsha-rai July 26, 2024 09:15

mashetty330 force-pushed the sc_cnv branch from 8e096b1 to 13139bc Compare July 30, 2024 08:21

ocs-ci reviewed Jul 30, 2024

View reviewed changes

ocs-ci reviewed Jul 31, 2024

View reviewed changes

PrasadDesala reviewed Aug 5, 2024

View reviewed changes

Akarsha-rai reviewed Aug 6, 2024

View reviewed changes

ocs_ci/deployment/cnv.py Show resolved Hide resolved

mashetty330 force-pushed the sc_cnv branch 2 times, most recently from cdc6601 to 6b1a390 Compare August 12, 2024 06:34

pull-request-size bot added size/XL and removed size/L PR that changes 100-499 lines labels Aug 12, 2024

openshift-merge-robot added the needs-rebase label Aug 12, 2024

mashetty330 force-pushed the sc_cnv branch from 6b1a390 to 1524efa Compare August 14, 2024 05:18

openshift-ci bot added the lgtm label Sep 2, 2024

ocs-ci reviewed Sep 2, 2024

View reviewed changes

ocs-ci reviewed Sep 3, 2024

View reviewed changes

openshift-merge-robot added the needs-rebase label Sep 3, 2024

mashetty330 and others added 12 commits September 4, 2024 15:00

create fixture for creating vms using standalone pvc

d17ab94

Signed-off-by: Mahesh Shetty <[email protected]>

Test CNV workload during netsplit

abb57b2

Signed-off-by: Mahesh Shetty <[email protected]>

Update docstrings for the netsplit test case

8a58223

Signed-off-by: Mahesh Shetty <[email protected]>

Add CNV workloads to the zone shutdown and zone crash tests

3b4a9b8

Signed-off-by: Mahesh Shetty <[email protected]>

add cnv deployment required marker

99c523d

Signed-off-by: Mahesh Shetty <[email protected]>

CNV setup, cleanup. Vm setup fixture

4e96302

Signed-off-by: Mahesh Shetty <[email protected]>

Update doc strings

a06bd6c

Signed-off-by: Mahesh Shetty <[email protected]>

Add polarion ids

e54f732

Signed-off-by: Mahesh Shetty <[email protected]>

Addressed review comments

d5884e8

Signed-off-by: Mahesh Shetty <[email protected]>

optimizing the tests

1c39d1e

Signed-off-by: Mahesh Shetty <[email protected]>

Rebase code to use new CNV workloads fixture

687b149

Signed-off-by: Mahesh Shetty <[email protected]>

Address review comments

70803b7

Signed-off-by: Mahesh Shetty <[email protected]>

mashetty330 dismissed PrasadDesala’s stale review via 70803b7 September 4, 2024 09:33

mashetty330 force-pushed the sc_cnv branch from f608aa7 to 70803b7 Compare September 4, 2024 09:33

openshift-ci bot removed the lgtm label Sep 4, 2024

openshift-merge-robot removed the needs-rebase label Sep 4, 2024

ocs-ci reviewed Sep 4, 2024

View reviewed changes

Akarsha-rai approved these changes Sep 5, 2024

View reviewed changes

openshift-ci bot added the lgtm label Sep 5, 2024

PrasadDesala approved these changes Sep 6, 2024

View reviewed changes

PrasadDesala merged commit 16b6700 into red-hat-storage:master Sep 6, 2024
6 of 7 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Stretch cluster + CNV automation #10077

Stretch cluster + CNV automation #10077

mashetty330 commented Jul 11, 2024

ocs-ci left a comment

ocs-ci left a comment

ocs-ci left a comment

ocs-ci left a comment

ocs-ci left a comment

ocs-ci left a comment

PrasadDesala Aug 5, 2024

PrasadDesala Aug 5, 2024

mashetty330 Aug 9, 2024

PrasadDesala Aug 9, 2024

mashetty330 Aug 20, 2024

PrasadDesala Aug 5, 2024

PrasadDesala Aug 5, 2024

ocs-ci left a comment

ocs-ci left a comment

ocs-ci left a comment

ocs-ci left a comment

ocs-ci left a comment

ocs-ci left a comment

ocs-ci left a comment

openshift-ci bot commented Sep 6, 2024



		@pytest.fixture()
		def setup_vms_standalone_pvc(request, project_factory, setup_cnv):

Stretch cluster + CNV automation #10077

Stretch cluster + CNV automation #10077

Conversation

mashetty330 commented Jul 11, 2024

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

PrasadDesala Aug 5, 2024

Choose a reason for hiding this comment

PrasadDesala Aug 5, 2024

Choose a reason for hiding this comment

mashetty330 Aug 9, 2024

Choose a reason for hiding this comment

PrasadDesala Aug 9, 2024

Choose a reason for hiding this comment

mashetty330 Aug 20, 2024

Choose a reason for hiding this comment

PrasadDesala Aug 5, 2024

Choose a reason for hiding this comment

PrasadDesala Aug 5, 2024

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

ocs-ci left a comment

Choose a reason for hiding this comment

openshift-ci bot commented Sep 6, 2024