-
Notifications
You must be signed in to change notification settings - Fork 4
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
🛂 Grant App roles trust permissions to be assumed by the app repo #4023
Comments
This issue is being marked as stale because it has been open for 60 days with no activity. Remove stale label or comment to keep the issue open. |
This issue is being closed because it has been open for a further 7 days with no activity. If this is still a valid issue, please reopen it, Thank you! |
This issue is being marked as stale because it has been open for 60 days with no activity. Remove stale label or comment to keep the issue open. |
This issue is being closed because it has been open for a further 7 days with no activity. If this is still a valid issue, please reopen it, Thank you! |
User Story
As an app admin
I want to allow my github runner access to assume my app role
So that I can make use of app permissions in github runner actions
Value / Purpose
Users have asked us to create custom roles that can be assumed by github runners. By allowing the app repo to assume the role that we already create for the app, we wont need to manage additional roles for runners
Useful Contacts
@julialawrence @michaeljcollinsuk
User Types
No response
Hypothesis
If we... [do a thing]
Then... [this will happen]
Proposal
When creating an IAM role for an app, setup the trust policy so that it can assumed by the app repo by default. App roles are already created via the Control Panel, when a user registers their app. So this existing implementation will need to be updated to add the extra permissions statement to the trust policy. We store the repo url when an app is registered, so we can use this in the trust policy definition.
For existing apps, we will need to decide if we retrospectively apply the same changes, or only update for apps that have requested it.
Additional Information
See discussion about this in Slack https://mojdt.slack.com/archives/C04M8224WCV/p1712686094130389.
There may be related support/feature requests but I cannot find them at this time.
Definition of Done
The text was updated successfully, but these errors were encountered: