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

Test to delete mon services manually #4711

Merged
merged 4 commits into from Aug 26, 2021

Conversation

Akarsha-rai
Copy link
Contributor

Automates bz-1969733

Signed-off-by: Akarsha-rai akrai@redhat.com

@Akarsha-rai Akarsha-rai requested a review from a team as a code owner August 12, 2021 05:39
@pull-request-size pull-request-size bot added the size/L PR that changes 100-499 lines label Aug 12, 2021
Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: tier4b
PR Test Path: tests/manage/pv_services/test_del_mon_service_and_create_pvc.py::TestPvcCreationAfterDelMonService::test_del_mon_svc
Additional Test Params:
OCP VERSION: 4.8
OCS VERSION: 4.8

Job FAILED (installation failed, tests not executed).

Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: tier4b
PR Test Path: tests/manage/pv_services/test_del_mon_service_and_create_pvc.py::TestPvcCreationAfterDelMonService::test_del_mon_svc
Additional Test Params:
OCP VERSION: 4.8
OCS VERSION: 4.8

Job FAILED (installation failed, tests not executed).

Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: tier4a
PR Test Path: tests/manage/pv_services/test_del_mon_service_and_create_pvc.py::TestPvcCreationAfterDelMonService::test_del_mon_svc
Additional Test Params:
OCP VERSION: 4.8
OCS VERSION: 4.8

Job UNSTABLE (some or all tests failed).

Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: tier4a
PR Test Path: tests/manage/pv_services/test_del_mon_service_and_create_pvc.py::TestPvcCreationAfterDelMonService::test_del_mon_svc
Additional Test Params:
OCP VERSION: 4.8
OCS VERSION: 4.8

Job PASSED.


@bugzilla("1969733")
@pytest.mark.polarion_id("OCS-2611")
def test_del_mon_svc(self, multi_pvc_factory):
Copy link
Contributor

Choose a reason for hiding this comment

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

this test looks quite disruptive so in case it fails in the middle, there is no teardown to bring the cluster back to health state. Please add such a teardown

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added teardown function to validate mon services are up or not, if not restart the operator pod.

Signed-off-by: Akarsha-rai <akrai@redhat.com>
Signed-off-by: Akarsha-rai <akrai@redhat.com>
Signed-off-by: Akarsha-rai <akrai@redhat.com>
Signed-off-by: Akarsha-rai <akrai@redhat.com>
Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: tier4a
PR Test Path: tests/manage/pv_services/test_del_mon_service_and_create_pvc.py::TestPvcCreationAfterDelMonService::test_del_mon_svc
Additional Test Params:
OCP VERSION: 4.8
OCS VERSION: 4.8

Job FAILED (installation failed, tests not executed).

Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: tier4a
PR Test Path: tests/manage/pv_services/test_del_mon_service_and_create_pvc.py::TestPvcCreationAfterDelMonService::test_del_mon_svc
Additional Test Params:
OCP VERSION: 4.8
OCS VERSION: 4.8

Job FAILED (installation failed, tests not executed).

Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: tier4a
PR Test Path: tests/manage/pv_services/test_del_mon_service_and_create_pvc.py::TestPvcCreationAfterDelMonService::test_del_mon_svc
Additional Test Params:
OCP VERSION: 4.8
OCS VERSION: 4.8

Job PASSED.

@PrasadDesala PrasadDesala merged commit 9df2925 into red-hat-storage:master Aug 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L PR that changes 100-499 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants