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 "SKIPPED" status in Post upgrade suite #1738

Merged
merged 2 commits into from
Aug 21, 2024

Conversation

bdattoma
Copy link
Contributor

@bdattoma bdattoma commented Aug 21, 2024

Currently, some tests in post upgrade suite have "SKIPPED" status because a piece of the teardown gets skipped if RHOAI operator is not Managed.
This PR replaces the call to SKIP keyword with IF statement so to avoid the SKIPPED status which doesn't reflect the right test status

PR validation:

  • pre-upgrade suite + post-upgrade suite (without actual upgrade): rhods-ci-pr-test/3270 OK - 1 failure expected for automation issue

image

@bdattoma bdattoma self-assigned this Aug 21, 2024
@bdattoma bdattoma added needs testing Needs to be tested in Jenkins enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) labels Aug 21, 2024
Copy link
Contributor

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
522 0 0 522 100

Copy link

@bdattoma bdattoma changed the title Fix "SKIP" status in Post upgrade suite Fix "SKIPPED" status in Post upgrade suite Aug 21, 2024
@asanzgom
Copy link
Contributor

Waiting for rhods-ci-pr-test/3270/ to finish successfully to approve

@bdattoma bdattoma merged commit 55534fe into red-hat-data-services:master Aug 21, 2024
10 checks passed
jgarciao pushed a commit to jgarciao/ods-ci that referenced this pull request Sep 1, 2024
* replace SKIP with IF

* add missing documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) needs testing Needs to be tested in Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants