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

Launch policy defined twice - each different #7

Open
robsyme opened this issue Apr 4, 2023 · 3 comments
Open

Launch policy defined twice - each different #7

robsyme opened this issue Apr 4, 2023 · 3 comments

Comments

@robsyme
Copy link
Member

robsyme commented Apr 4, 2023

The launch policy looks to be defined twice:

once in launch/launch-policy.json:

"Sid": "VisualEditor0",
"Effect": "Allow",
"Action": [
"batch:DescribeJobQueues",

and once in forge/forge-policy.json (this json document contains both the forge policy and the second launch policy:

"Sid": "TowerLaunch0",
"Effect": "Allow",
"Action": [
"s3:Get*",
"s3:List*",

There are differences between the two launch policies:
image

Is there any benefit in defining the launch policy in forge/forge-policy.json? Perhaps it would be best to remove it and rely on just the one copy in launch/launch-policy.json.

@pditommaso
Copy link
Collaborator

Because they are meant to be used indipendly

@robsyme
Copy link
Member Author

robsyme commented Apr 4, 2023

I understand that the forge policy is to be used independently from the launch policy, but what is the difference between the two launch policies? Why would you need to define a launch policy if you are using Tower Forge - doesn't it create those policies for you anyway?

@pditommaso
Copy link
Collaborator

Why would you need to define a launch policy if you are using Tower Forge - doesn't it create those policies for you anyway?

Tower creates the instance policy and other permission policies used by the child jobs, but it cannot grant permissions to itself. Said differently, that's the policy to allow Forge to access Batch, Cloudwatch and other APIs required to launch the pipeline execution.

The launch policy is a subset of the forge policy because Forge requires more permissions to carry out some operation in place of the user.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants