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

Implement tests for NooBaa component rescheduling improvements #4966

Merged
merged 2 commits into from Nov 3, 2021

Conversation

sidhant-agrawal
Copy link
Contributor

@sidhant-agrawal sidhant-agrawal commented Oct 17, 2021

This is related to rescheduling improvements in NooBaa component introduced in KNIP-1707
This PR introduces following changes:

  • Add new tests in mcg/test_host_node_failure.py (RHSTOR-1887)
  • Reduce timeout from 300 seconds to 90 seconds in existing mcg/test_mcg_resources_disruptions.py (RHSTOR-1691)

Signed-off-by: Sidhant Agrawal sagrawal@redhat.com

 - Add new tests in mcg/test_host_node_failure.py
 - Reduce timeout from 300 seconds to 90 seconds in existing mcg/test_mcg_resources_disruptions.py

Signed-off-by: Sidhant Agrawal <sagrawal@redhat.com>
@sidhant-agrawal sidhant-agrawal requested a review from a team as a code owner October 17, 2021 04:55
@pull-request-size pull-request-size bot added the size/L PR that changes 100-499 lines label Oct 17, 2021
 - Use constant values for Noobaa resources

Signed-off-by: Sidhant Agrawal <sagrawal@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/mcg/test_host_node_failure.py tests/manage/mcg/test_mcg_resources_disruptions.py::TestMCGResourcesDisruptions::test_delete_noobaa_resources
Additional Test Params:
OCP VERSION: 4.9
OCS VERSION: 4.9

Job PASSED.

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/mcg/test_host_node_failure.py tests/manage/mcg/test_mcg_resources_disruptions.py::TestMCGResourcesDisruptions::test_delete_noobaa_resources
Additional Test Params:
OCP VERSION: 4.8
OCS VERSION: 4.8

Job PASSED.

@sidhant-agrawal sidhant-agrawal added the Verified Mark when PR was verified and log provided label Oct 20, 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 Verified Mark when PR was verified and log provided
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants