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

IBM Cloud Databases - Deprecate flat list of scaling attributes #3782

Merged
merged 8 commits into from
May 19, 2022

Conversation

omaraibrahim
Copy link
Collaborator

@omaraibrahim omaraibrahim commented May 16, 2022

With the addition of the ibm_database groups api the following attributes are set of deprecation:

  • members_memory_allocation_mb
  • members_disk_allocation_mb
  • members_cpu_allocation_count
  • node_memory_allocation_mb
  • node_disk_allocation_mb
  • node_cpu_allocation_count
  • node_count

A deprecation attribute has been added and these attributes conflict with the groups attribute (that particular set of code was added in the groups PR). Also the docs have been updated.

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccIBMDatabaseDataSource_basic'
--- PASS: TestAccIBMDatabaseDataSource_basic (662.19s)
PASS
ok  	github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/database	663.210s
make testacc TEST=./ibm/service/database TESTARGS='-run=TestAccIBMDatabaseInstance_Etcd_Basic'
--- PASS: TestAccIBMDatabaseInstance_Etcd_Basic (1086.24s)
PASS
ok  	github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/database	1087.229s

@omaraibrahim omaraibrahim changed the title Deprecate flat list attributes IBM Cloud Databases - Deprecate flat list attributes May 16, 2022
Comment on lines +39 to +45
memory {
allocation_mb = 14336
}

disk {
allocation_mb = 20480
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should make sure these allocations are same as the "Small" T-Shirt sizes.
It will be different for each database type.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done :D

@obai-1 obai-1 changed the title IBM Cloud Databases - Deprecate flat list attributes IBM Cloud Databases - Deprecate flat list of scaling attributes May 19, 2022
@obai-1 obai-1 merged commit aea4985 into IBM-Cloud:master May 19, 2022
@omaraibrahim omaraibrahim deleted the deprecate_flat_list_attributes branch May 27, 2022 14:29
SunithaGudisagarIBM pushed a commit to ibm-vpc/terraform-provider-ibm that referenced this pull request Sep 14, 2022
…Cloud#3782)

* added deprecation and updated docs
* updated allocation sizes to be in accordance to "small" template sizes
SunithaGudisagarIBM pushed a commit to ibm-vpc/terraform-provider-ibm that referenced this pull request Sep 14, 2022
…Cloud#3782)

* added deprecation and updated docs
* updated allocation sizes to be in accordance to "small" template sizes
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.

2 participants