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] Delete databricks_sql_endpoint that failed to start #4520

Merged
merged 2 commits into from
Feb 25, 2025

Conversation

nkvuong
Copy link
Contributor

@nkvuong nkvuong commented Feb 24, 2025

Changes

Tests

  • make test run locally
  • using Go SDK

@nkvuong nkvuong requested review from a team as code owners February 24, 2025 03:34
@nkvuong nkvuong requested review from parthban-db and removed request for a team February 24, 2025 03:34
// Rollback by deleting the warehouse
rollbackErr := w.Warehouses.DeleteById(ctx, wait.Id)
if rollbackErr != nil {
return fmt.Errorf("failed deleting warehouse: %w", rollbackErr)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will hide an original error. Maybe we can put it here as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agreed, updated the error message :)

Copy link

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/terraform

Inputs:

  • PR number: 4520
  • Commit SHA: 19f7d603e3d7e2d39adee6c38860d5d6e287c792

Checks will be approved automatically on success.

@alexott alexott added this pull request to the merge queue Feb 25, 2025
Merged via the queue into main with commit 3004cb9 Feb 25, 2025
13 checks passed
@alexott alexott deleted the fix/delete-failed-sql-warehouses branch February 25, 2025 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ISSUE] Issue with databricks_sql_endpoint resource ("CLASSIC" type)
2 participants