Skip to content

chore(e2e): re-enable bulk import tests on OSD#4504

Open
PatAKnight wants to merge 1 commit intoredhat-developer:mainfrom
PatAKnight:reenable-bulk-import-osd
Open

chore(e2e): re-enable bulk import tests on OSD#4504
PatAKnight wants to merge 1 commit intoredhat-developer:mainfrom
PatAKnight:reenable-bulk-import-osd

Conversation

@PatAKnight
Copy link
Copy Markdown
Member

Description

Attempts to re-enable the Bulk Import tests on OSD

Which issue(s) does this PR fix

  • Fixes #?

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Signed-off-by: Patrick Knight <pknight@redhat.com>
@rhdh-qodo-merge
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🔒 No security concerns identified
⚡ Recommended focus areas for review

📚 Focus areas based on broader codebase context

Test Stability

The OSD-specific skip guard was removed, so these bulk import E2E tests will now run on osd-gcp jobs. Please confirm the underlying OSD failure condition is resolved, or replace the skip with a more targeted/temporary condition to avoid reintroducing known CI flakes. (Ref 1)

test.describe.serial("Bulk Import plugin", () => {
  test.describe.configure({ retries: process.env.CI ? 5 : 0 });

Reference reasoning: Existing bulk import E2E coverage includes an explicit conditional skip for OSD jobs due to a tracked OSD environment issue. That pattern indicates the suite previously required environment-based gating to keep CI stable, so removing it should be validated against the same job matrix.

📄 References
  1. redhat-developer/rhdh/e2e-tests/playwright/e2e/plugins/bulk-import.spec.ts [284-351]

@rhdh-qodo-merge
Copy link
Copy Markdown

rhdh-qodo-merge bot commented Mar 30, 2026

PR Type

Tests


Description

  • Remove skip condition for bulk import tests on OSD

  • Tests will now run on OSD GCP environment

  • Resolves RHDHBUGS-555 blocking condition


File Walkthrough

Relevant files
Tests
bulk-import.spec.ts
Remove OSD skip condition from bulk import tests                 

e2e-tests/playwright/e2e/plugins/bulk-import.spec.ts

  • Removed test.skip() condition that was skipping tests on OSD GCP
    environment
  • Tests previously blocked due to [RHDHBUGS-555](https://redhat.atlassian.net/browse/RHDHBUGS-555) will now execute
+0/-4     

@rhdh-qodo-merge
Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

@PatAKnight
Copy link
Copy Markdown
Member Author

/test e2e-osd-gcp-helm-nightly

2 similar comments
@PatAKnight
Copy link
Copy Markdown
Member Author

/test e2e-osd-gcp-helm-nightly

@PatAKnight
Copy link
Copy Markdown
Member Author

/test e2e-osd-gcp-helm-nightly

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 30, 2026

@PatAKnight: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-osd-gcp-helm-nightly a9f56d5 link false /test e2e-osd-gcp-helm-nightly

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant