teleport icon indicating copy to clipboard operation
teleport copied to clipboard

Support China and GovCloud for database access

Open nklaassen opened this issue 2 years ago • 0 comments

This PR adds support for non-default AWS partitions in resource ARNs of IAM policies and roles which are created for database access and auto-discovery on AWS.

I am hijacking this PR from https://github.com/gravitational/teleport/pull/13054 to add a minor fix and GovCloud support, and so I can more easily run CI and get this merged. The original PR was authored by @waylenguo, I will make sure he is credited as an author on the final commit.

nklaassen avatar Aug 09 '22 22:08 nklaassen

@nklaassen See the table below for backport results.

Branch Result
branch/v10 Create PR
branch/v8 Failed
branch/v9 Failed

github-actions[bot] avatar Aug 16 '22 18:08 github-actions[bot]

@greedy52 I'm considering where to backport this. I don't see that https://github.com/gravitational/teleport/pull/10560 was backported anywhere, so I think it's only in v10. This change depends on that one in more ways than one, do you think it's appropriate to only backport this to v10?

nklaassen avatar Aug 16 '22 18:08 nklaassen

@nklaassen it wasn't backported because I wasn't confident of the change since we don't have an account to test. so it was kept on master.

I don't know if any users are explicitly asking for this support still and what versions they are. @r0mant @smallinsky?

greedy52 avatar Aug 16 '22 19:08 greedy52

Hey @greedy52, we did have a specific customer with interest in this. They are on 10.x.x and can easily update.

They are not currently being blocked because they have manually added DBs for now. They are on GovCloud.

corkrean avatar Aug 16 '22 22:08 corkrean