Fix test bucket endpoints

[IBM-Deeksha]

• update bucket endpoints display for test cloud account

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 #0000

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccXXX'

...

[cjschaef]

I performed further testing with this change, and it wasn't sufficient still. The data_source also needs a similar update in order to provide the proper endpoint for use.

I made a similar change to the data_source, and tested to verify it worked properly.
cjschaef@31ae4aa

[cjschaef]

I did some testing with the additional changes you added, and that does resolve the issues we were seeing during TF create.

However, I also did some destroy testing, it it appears one of the other functions (resourceIBMCOSBucketExists) in resource_ibm_cos_bucket.go is missing the FileFallBack feature, to use an overridden endpoint.
For reference, this is the change I added that appears to allow the destroy logic to find the existing resource and then delete it.
cjschaef@59085f2

Sorry, I did not realize this was missed in the initial PR's to add the necessary support for COS endpoint override.
Whether that is a separate PR or can be included in this change, I leave up to you.

[IBM-Deeksha]

Thanks again @cjschaef for highlighting. I have made the required changes.

[cjschaef]

@IBM-Deeksha I fixed the golang errors and retested and this appears to now be sufficient with our needs to override COS endpoints.
cjschaef@5238034

I can retest if necessary, once you fix the golang errors, but I expect these changes are good now.

[IBM-Deeksha]

Done @cjschaef. Thanks for verifying.