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

Fixes test_ocs_operator_is_not_present for utilizing custom namespace #9877

Conversation

suchita-g
Copy link
Contributor

In Test case test_ocs_operator_is_not_present the use of namespace "openshift-storage" is hardcoded, hence this test is failing on platform where custom name is used for ODF deployment, like ROSA/AWS HCP Fix for: #9876

Signed-off-by: suchita-g [email protected]

Signed-off-by: suchita-g <[email protected]>

In Test case test_ocs_operator_is_not_present the use of namespace
"openshift-storage" is hardcoded, hence this test is failing on
platform where custom name is used for ODF deployment, like ROSA/AWS HCP
Fix for: red-hat-storage#9876

Signed-off-by: suchita-g <[email protected]>
@suchita-g suchita-g self-assigned this May 30, 2024
@suchita-g suchita-g requested a review from a team as a code owner May 30, 2024 18:05
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: vkathole-416
Cluster Configuration:
PR Test Suite: tier1
PR Test Path: tests/cross_functional/ui/test_validation_ui.py::TestUserInterfaceValidation::test_ocs_operator_is_not_present
Additional Test Params:
OCP VERSION: 4.16
OCS VERSION: 4.16
tested against branch: master

Job PASSED.

@suchita-g suchita-g force-pushed the Fix_9876_test_ocs_operator_is_not_present branch from 811810a to 3d3df15 Compare May 31, 2024 01:00
@suchita-g suchita-g added the Verified Mark when PR was verified and log provided label May 31, 2024
Copy link

openshift-ci bot commented May 31, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: am-agrawa, ebondare, 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

@dahorak dahorak merged commit a309162 into red-hat-storage:master May 31, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm size/XS ui_automation Verified Mark when PR was verified and log provided
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants