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

Remove Deprovision on failure from OCP provisioning task #1924

Merged

Conversation

bdattoma
Copy link
Contributor

@bdattoma bdattoma commented Oct 14, 2024

Removing the deprovisioning of the OCP cluster - when the provisioning fails - from the task for 2 reasons:

  1. it always fails to run for some 401 Unauthorized error
  2. from long time now, the CI leverages on another deprovisioning script which is not in ODS-CI

PR validation:

  • provisioning regression test: rhoai-test-flow/826 PASS - rhoai deployment failed because I put a wrong setting, but cluster provisioning worked just fine)

Copy link
Contributor

Robot Results

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

@bdattoma bdattoma requested a review from kobihk October 14, 2024 15:38
@bdattoma bdattoma requested a review from kobihk October 14, 2024 16:35
Log Cluster '${cluster_name}' has previously failed to be provisioned - Cleaning Hive resources
... console=True
Delete Cluster Configuration
FAIL Cluster '${cluster_name}' has previously failed to be provisioned but some Hive and/or Cloud resources are still present.

Check warning

Code scanning / Robocop

Line is too long ({{ line_length }}/{{ allowed_length }}) Warning

Line is too long (137/120)
@bdattoma bdattoma self-assigned this Oct 15, 2024
@bdattoma bdattoma added verified This PR has been tested with Jenkins enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) labels Oct 15, 2024
Copy link

sonarcloud bot commented Oct 16, 2024

@bdattoma bdattoma merged commit 4950bcc into red-hat-data-services:master Oct 16, 2024
8 checks passed
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) verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants