Skip to content

14629:test_negative_custom_taint need adjustment for provider/multiclient setup#14716

Open
suchita-g wants to merge 1 commit intored-hat-storage:masterfrom
suchita-g:14629_negative_custom_taint_provider
Open

14629:test_negative_custom_taint need adjustment for provider/multiclient setup#14716
suchita-g wants to merge 1 commit intored-hat-storage:masterfrom
suchita-g:14629_negative_custom_taint_provider

Conversation

@suchita-g
Copy link
Contributor

-Fixes: #14629

  • The fix adds a runtime check in init_sanity that skips the test when running on an HCI client cluster. This is needed because:
    • The existing @skipif_hci_provider_and_client class-level marker is static, evaluated at import time when the multi-cluster config may not be fully populated
    • The runtime check uses config.ENV_DATA which reflects the current cluster context at test execution time
    • It checks specifically for HCI_CLIENT cluster type, since that's the cluster where CephCluster CRD doesn't exist

…ient setup

-Fixes: red-hat-storage#14629
- The fix adds a runtime check in init_sanity that skips the test when running on an HCI client cluster.
  This is needed because:
  - The existing @skipif_hci_provider_and_client class-level marker is static,
    evaluated at import time when the multi-cluster config may not be fully populated
  - The runtime check uses config.ENV_DATA which reflects the current cluster context
    at test execution time
  - It checks specifically for HCI_CLIENT cluster type, since that's the cluster
    where CephCluster CRD doesn't exist

Signed-off-by: suchita-g <sgatfane@redhat.com>
@suchita-g suchita-g self-assigned this Mar 18, 2026
@suchita-g suchita-g requested a review from a team as a code owner March 18, 2026 10:49
@openshift-ci
Copy link

openshift-ci bot commented Mar 18, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: suchita-g

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

Details 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: test_negative_custom_taint need adjustment for provider/multiclient setup

1 participant