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

MS037 - Adding Missing Permission & Ability to Assume Role from Central Terraform-running Account #7148

Merged

Conversation

julialawrence
Copy link
Contributor

Original details:
#7090
Tracking issue: ministryofjustice/analytical-platform#4358

Follow-on to:
#7145

iam:GetRole and iam:GetRolePolicy broken out to a separate statement block and arn of the LakeFormation service role corrected.

@julialawrence julialawrence requested review from a team as code owners July 19, 2024 13:47
@github-actions github-actions bot added the environments-repository Used to exclude PRs from this repo in our Slack PR update label Jul 19, 2024
@julialawrence julialawrence had a problem deploying to digital-prison-reporting-development July 19, 2024 13:49 — with GitHub Actions Error
Copy link
Contributor

@harichintala1 harichintala1 left a comment

Choose a reason for hiding this comment

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

LGTM

@julialawrence julialawrence merged commit 7ff497f into main Jul 19, 2024
12 of 16 checks passed
@julialawrence julialawrence deleted the feature/add-additional-permissions-to-share-policy-2 branch July 19, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
environments-repository Used to exclude PRs from this repo in our Slack PR update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants