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

DWX-20183 Deprecate standalone default database catalog resource #213

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tevesz
Copy link
Contributor

@tevesz tevesz commented Feb 17, 2025

The default database catalog is now managed as part of the cluster since its lifecycle is inherently tied to the cluster's lifecycle. All properties are now computed. New data warehouse cluster setups should use the embedded catalog properties instead.

  • deprecate the database catalog resource
  • add default database catalog properties under dw aws cluster resource
  • Add test fixes and acceptance test changes

Instance type properties are also computed. These values were not populated from the API before. There are default instance types that we should store in the state. The additional instance type attribute was not used, this property got dropped.

The default database catalog is now managed as part of the cluster
since its lifecycle is inherently tied to the cluster's lifecycle.
All properties are now computed. New data warehouse cluster setups
should use the embedded catalog properties instead.

- deprecate the database catalog resource
- add default database catalog properties under dw aws cluster
  resource
- Add test fixes and acceptance test changes

Instance type properties are also computed. These values were not
populated from the API before. There are default instance types that we
should store in the state. The additional instance type attribute was
not used, this property got dropped.
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.

1 participant