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

[FEATURE]: Create uber-IAM profile for performing external table migration on AWS #879

Closed
1 task done
Tracked by #893
nfx opened this issue Feb 5, 2024 · 0 comments · Fixed by #993
Closed
1 task done
Tracked by #893

[FEATURE]: Create uber-IAM profile for performing external table migration on AWS #879

nfx opened this issue Feb 5, 2024 · 0 comments · Fixed by #993
Assignees
Labels
enhancement New feature or request migrate/external go/uc/upgrade SYNC EXTERNAL TABLES step

Comments

@nfx
Copy link
Collaborator

nfx commented Feb 5, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Problem statement

Related issues:

Proposed Solution

Create AWS IAM role/profile that has read access for all of the S3 buckets for external table migration and add it to UCX cluster policy:

Additional Context

No response

@nfx nfx added enhancement New feature or request migrate/external go/uc/upgrade SYNC EXTERNAL TABLES step credentials labels Feb 5, 2024
@nfx nfx added this to UCX Feb 5, 2024
@github-project-automation github-project-automation bot moved this to Triage in UCX Feb 5, 2024
@nfx nfx changed the title [FEATURE]: Create uber-IAM profile for performing external migration [FEATURE]: Create uber-IAM profile for performing external table migration on AWS Feb 5, 2024
@mwojtyczka mwojtyczka self-assigned this Feb 29, 2024
@nfx nfx closed this as completed in #993 Mar 11, 2024
nfx pushed a commit that referenced this issue Mar 11, 2024
…pal` command (#993)

## Changes
Added CLI command `databricks labs ucx create-uber-principal` for
creating uber-IAM profile for performing external table migration on
AWS.

Logic:
* Stop if UCX migration cluster policy is not found
* Collect paths of all locations/paths used in tables (call
`external_location.snapshot`)
* If cluster policy has an existing iam instance profile/role specified,
then add/update migration policy providing access to the locations
* If cluster policy does not have iam instance profile/role specified,
then create new iam profile/role and migration policy, and add it to the
cluster policy

### Linked issues

Resolves #879 

Related issues:
- #976
- #693

### Functionality 

- [x] added new CLI command

### Tests

- [x] manually tested
- [x] added unit tests

### TODO
- [x] added integration tests
- [x] verified on staging environment (screenshot attached)

---------

Co-authored-by: Vuong <vuong.nguyen@databricks.com>
@github-project-automation github-project-automation bot moved this from Triage to Archive in UCX Mar 11, 2024
dmoore247 pushed a commit that referenced this issue Mar 23, 2024
…pal` command (#993)

## Changes
Added CLI command `databricks labs ucx create-uber-principal` for
creating uber-IAM profile for performing external table migration on
AWS.

Logic:
* Stop if UCX migration cluster policy is not found
* Collect paths of all locations/paths used in tables (call
`external_location.snapshot`)
* If cluster policy has an existing iam instance profile/role specified,
then add/update migration policy providing access to the locations
* If cluster policy does not have iam instance profile/role specified,
then create new iam profile/role and migration policy, and add it to the
cluster policy

### Linked issues

Resolves #879 

Related issues:
- #976
- #693

### Functionality 

- [x] added new CLI command

### Tests

- [x] manually tested
- [x] added unit tests

### TODO
- [x] added integration tests
- [x] verified on staging environment (screenshot attached)

---------

Co-authored-by: Vuong <vuong.nguyen@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request migrate/external go/uc/upgrade SYNC EXTERNAL TABLES step
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants