[OSD-20863] Must use the cluster region or s3 cleanup fails.

[bergmannf]

When attempting to cleanup s3 buckets in a different region than the default region it will fail otherwise.

What is being added?

Is this a fix for a bug? What's the bug? Is this a new feature? Please describe it. Is this just a small typo fix? That's fine too!

Checklist before requesting review

• [ ] I have tested this locally
• [ ] I have included unit tests
• [ ] I have updated any corresponding documentation

Steps To Manually Test

Please provide us steps to reproduce the scenarios needed to test this. If an integration test is provided, let us know how to run it. If not, enumerate the steps to validate this does what it's supposed to.

1. Start the operator
2. Run the thing
3. Observe X in the spec for the thing
4. Clean up the thing

Ref OSD-0000

[bergmannf]

I've tested updateing the resource-record with an aws CLI that is set to another region than us-east-1 and that still worked. Assuming they don't do any special handling in the CLI I hope this also works with the SDK.

[codecov-commenter]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (c5f00d9) 47.77% compared to head (712606b) 47.82%.

Additional details and impacted files

[](https://app.codecov.io/gh/openshift/aws-account-operator/pull/817?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openshift) ```diff @@ Coverage Diff @@ ## master #817 +/- ## ========================================== + Coverage 47.77% 47.82% +0.04% ========================================== Files 43 43 Lines 6296 6296 ========================================== + Hits 3008 3011 +3 + Misses 2972 2971 -1 + Partials 316 314 -2 ``` | [Files](https://app.codecov.io/gh/openshift/aws-account-operator/pull/817?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openshift) | Coverage Δ | | |---|---|---| | [controllers/accountclaim/reuse.go](https://app.codecov.io/gh/openshift/aws-account-operator/pull/817?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openshift#diff-Y29udHJvbGxlcnMvYWNjb3VudGNsYWltL3JldXNlLmdv) | `53.49% <75.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/openshift/aws-account-operator/pull/817/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openshift)

[openshift-ci[bot]]

@bergmannf: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).

[iamkirkbater]

/lgtm

[openshift-ci[bot]]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bergmannf, iamkirkbater

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openshift/aws-account-operator/blob/master/OWNERS)~~ [bergmannf,iamkirkbater] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment