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

Fix: Scale ceph pod respin test while loop is not exited #8634

Merged
merged 2 commits into from
Oct 9, 2023

Conversation

ramkiperiy
Copy link
Contributor

@ramkiperiy ramkiperiy commented Oct 5, 2023

Issues Observed: github issue #2201
PVC not reaching Bound state due to issues in environment like Ceph Cluster health not in good shape POD not reaching Running state due to lack of resources for running pods in cluster

In above scenarios TC while loop is continued till it creates the required number of resources(PVCs and PODs) in the cluster, due to this TC continued to execute for more than 4 days.

In the PR will be adding a iteration counter to terminate the while loop.

Issues Observed: github issue red-hat-storage#2201
PVC not reaching Bound state due to issues in environment like Ceph Cluster health not in good shape
POD not reaching Running state due to lack of resources for running pods in cluster

In above scenarios TC while loop is continued till it creates the required number of resources(PVCs and PODs) in the cluster, due to this TC continued to execute for more than 4 days.

In the PR will be adding a iteration counter to terminate the while loop.

Signed-off-by: Ramakrishnan Periyasamy <[email protected]>
@ramkiperiy ramkiperiy requested a review from a team as a code owner October 5, 2023 14:14
@pull-request-size pull-request-size bot added the size/M PR that changes 30-99 lines label Oct 5, 2023
@ramkiperiy ramkiperiy added bug Something isn't working scale labels Oct 5, 2023
Signed-off-by: Ramakrishnan Periyasamy <[email protected]>
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.

Unknown PR validation

Cluster Name: rperiyas-pr
Cluster Configuration:
PR Test Suite: scale
PR Test Path: tests/e2e/scale/test_pv_scale_and_respin_ceph_pods.py
Additional Test Params:
OCP VERSION: 4.14
OCS VERSION: 4.14
tested against branch: master

Job state: ABORTED.

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 on existing cluster

Cluster Name: jopinto-t411
Cluster Configuration: conf/deployment/vsphere/upi_1az_rhcos_vsan_3m_3w.yaml
PR Test Suite: scale
PR Test Path: tests/e2e/scale/test_pv_scale_and_respin_ceph_pods.py
Additional Test Params:
OCP VERSION: 4.11
OCS VERSION: 4.11
tested against branch: master

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: rperiyas-8634
Cluster Configuration:
PR Test Suite: scale
PR Test Path: tests/e2e/scale/test_pv_scale_and_respin_ceph_pods.py
Additional Test Params:
OCP VERSION: 4.14
OCS VERSION: 4.14
tested against branch: master

Job PASSED.

@openshift-ci
Copy link

openshift-ci bot commented Oct 9, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: keesturam, petr-balogh, ramkiperiy, tiffanyn108

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

Needs approval from an approver in each of these files:

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

@keesturam keesturam merged commit 00dcab6 into red-hat-storage:master Oct 9, 2023
6 of 7 checks passed
ramkiperiy added a commit to ramkiperiy/ocs-ci that referenced this pull request Oct 9, 2023
Cherry Pick PR red-hat-storage#8634
for 4.12 release branch

Signed-off-by: Ramakrishnan Periyasamy <[email protected]>
ramkiperiy added a commit to ramkiperiy/ocs-ci that referenced this pull request Oct 9, 2023
Cherry Pick PR red-hat-storage#8634 to
4.13 release branch

Signed-off-by: Ramakrishnan Periyasamy <[email protected]>
ramkiperiy added a commit that referenced this pull request Oct 9, 2023
* Cherry Pick PR-8634 for release 4.12

Cherry Pick PR #8634
for 4.12 release branch

Signed-off-by: Ramakrishnan Periyasamy <[email protected]>
ramkiperiy added a commit that referenced this pull request Oct 10, 2023
* Cherry Pick PR 8634 to 4.13 release branch

Cherry Pick PR #8634 to
4.13 release branch

Signed-off-by: Ramakrishnan Periyasamy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working lgtm scale size/M PR that changes 30-99 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants