Skip to content

Conversation

@lornakelly
Copy link
Contributor

@lornakelly lornakelly commented Oct 9, 2025

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates OR Closes: https://github.ibm.com/compose/App/issues/2466

  • Add options include_hidden and include_beta parameters to fetching deployment capability versions
  • Include hidden and beta versions when fetching deployment capability versions for in place upgrade validation
  • Update test with new version for in place upgrade
  • Bump go sdk version

Output from acceptance testing:

--- PASS: TestAccIBMDatabaseInstance_Redis_Basic (838.17s)
--- PASS: TestAccIBMDatabaseInstanceRedisImport (366.70s)
--- PASS: TestAccIBMDatabaseInstanceMongodbBasic (964.42s)
--- PASS: TestAccIBMDatabaseInstancePostgresBasic (1766.58s)
--- PASS: TestAccIBMDatabaseInstancePostgresGroup (1755.44s)
--- PASS: TestAccIBMDatabaseInstanceMongodbUpgrade (865.11s)

Output from unit tests

$ go test github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/database
ok      github.com/IBM-Cloud/terraform-provider-ibm/ibm/service/database        2.575s

...

Copy link

@oaibrahim oaibrahim left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@omaraibrahim omaraibrahim left a comment

Choose a reason for hiding this comment

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

LGTM

@omaraibrahim omaraibrahim merged commit 1f2bcf1 into IBM-Cloud:master Oct 10, 2025
1 check passed
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.

3 participants