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

Jump to bottom

Fixing issue of openshift-storage not found in awscli_pod_fixture function #8901

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Fixing issue of openshift-storage not found in awscli_pod_fixture function #8901

merged 1 commit into from
Nov 21, 2023

Conversation

suchita-g
Copy link
Contributor

@suchita-g suchita-g commented Nov 21, 2023 •
edited
Loading

This Fixes the issue 8863 for 45 Test cases , error at openshift-storage not found while running on client cluster

@pull-request-size pull-request-size bot added the size/S PR that changes 10-29 lines label Nov 21, 2023
@suchita-g suchita-g force-pushed the Debug_openshift-storage-issue branch 2 times, most recently from f19b937 to 1a61bd8 Compare November 21, 2023 08:00
@pull-request-size pull-request-size bot added size/M PR that changes 30-99 lines and removed size/S PR that changes 10-29 lines labels Nov 21, 2023
@suchita-g suchita-g force-pushed the Debug_openshift-storage-issue branch 2 times, most recently from a7c3cb1 to e81107b Compare November 21, 2023 08:19
@pull-request-size pull-request-size bot added size/S PR that changes 10-29 lines and removed size/M PR that changes 30-99 lines labels Nov 21, 2023
@suchita-g suchita-g marked this pull request as ready for review November 21, 2023 08:35
@suchita-g suchita-g requested review from a team as code owners November 21, 2023 08:35
@suchita-g suchita-g added the Verified Mark when PR was verified and log provided label Nov 21, 2023
@suchita-g
Copy link
Contributor Author

Verifed: https://ocs4-jenkins-csb-odf-qe.apps.ocp-c1.prod.psi.redhat.com/job/qe-odf-multicluster/2098/
https://ocs4-jenkins-csb-odf-qe.apps.ocp-c1.prod.psi.redhat.com/job/qe-odf-multicluster/2098/testReport/
the test failed Error is Error from server (AlreadyExists): error when creating "/tmp/StatefulSets_4ubtx3": statefulsets.apps "s3cli" already exists"
However this PR is fixing the existing issue where openshift-storage namespace is used in yaml template by default causing the issue "openshift-storage" not found on client cluster

@suchita-g
Fixing issue of openshift-storage not found in awscli_pod_fixture fun…
9b99086 
…ction

Signed-off-by: suchita-g <sgatfane@redhat.com>
This Fixes the issue 8863 for 45 Test cases
Signed-off-by: suchita-g <sgatfane@redhat.com>
@pull-request-size pull-request-size bot added size/XS and removed size/S PR that changes 10-29 lines labels Nov 21, 2023
@suchita-g suchita-g changed the title Debug openshift storage issue Fixing issue of openshift-storage not found in awscli_pod_fixture function Nov 21, 2023
@openshift-ci openshift-ci bot added the lgtm label Nov 21, 2023
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Nov 21, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fbalak, jilju, suchita-g

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

@fbalak fbalak merged commit 772dbdb into red-hat-storage:master Nov 21, 2023
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fbalak fbalak fbalak approved these changes

@jilju jilju jilju approved these changes

Assignees

@fbalak fbalak

@jilju jilju

Labels
lgtm size/XS Verified Mark when PR was verified and log provided
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@suchita-g @fbalak @jilju