-
Notifications
You must be signed in to change notification settings - Fork 45
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
Update nf-syn23664726-s3.yaml #601
Conversation
Maybe this will resolve permission errors in Tower?
config/prod/nf-syn23664726-s3.yaml
Outdated
@@ -21,7 +21,9 @@ parameters: | |||
- 'arn:aws:iam::694704239947:user/giri' ## Nebula Genomics access | |||
- 'arn:aws:iam::969434029702:user/audrisc' ##Stanford access (Audris Chiang) | |||
- 'arn:aws:iam::728882028485:role/ntap-add5-project-TowerForgeBatchHeadJobRole-X9Y068JVFXHC' ## Sage NF Tower access | |||
- 'arn:aws:iam::728882028485:role/ntap-add5-project-TowerForgeBatchWorkJobRole-TYLG8H2QIS7R' ## Sage NF Tower access | |||
- 'arn:aws:iam::728882028485:role/ntap-add5-project-TowerForgeBatchWorkJobRole-TYLG8H2QIS7R' ## Sage NF Tower access |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there already a TowerForgeBatchHeadJobRole defined in the line above. is this one intended to replace the one above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @zaro0508, there are three roles I am trying to add: TowerForgeBatchHeadJobRole, TowerForgeBatchWorkJobRole, TowerForgeServiceRole.
My end goal is to use Tower to index a non-Tower bucket (s3://nf-syn23664726-s3-bucket-n9uakf7bowwd/). Do you think that these three roles will be sufficient to do this? I tried just TowerForgeBatchHeadJobRole and TowerForgeBatchWorkJobRole and that didn't appear to work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm sorry, I don't know Tower well enough to answer your question @allaway. I would suggest asking @thomasyu888
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After talking with Adam, I think I need to make the config change in this PR, but I also think I need to change the config for the nextflow project.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
please fix the pre-commit failures @allaway then we can merge |
Oops, trailing whitespace. Fixed! |
Maybe this will resolve permission errors in Tower?