-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
analysis dashboard access to satellite bucket #482
analysis dashboard access to satellite bucket #482
Conversation
… of s3_nwp_buckets
Do you need to remove this one - https://github.com/openclimatefix/ocf-infrastructure/blob/external-pr/terraform/modules/services/eb_app/iam.tf#L128 |
Yes, ill make the change , thank you for pointing it out |
I have removed this |
* analysis dashboard access to satellite bucket (#482) * added var s3_nwp_buckets * analysis dashboard access to satellite bucket * added variables.tfvars * Refactor: Modify s3_nwp_bucket variable to accept a list of bucket ARNs * updated modules in india/dev & nowcasting/dev to include the new list of s3_nwp_buckets * removed conflicting role policy attachment code (#487) * added var s3_nwp_buckets * analysis dashboard access to satellite bucket * added variables.tfvars * Refactor: Modify s3_nwp_bucket variable to accept a list of bucket ARNs * updated modules in india/dev & nowcasting/dev to include the new list of s3_nwp_buckets * removed old role policy attachment resource conflicting code --------- Co-authored-by: Peter Dudfield <34686298+peterdudfield@users.noreply.github.com> --------- Co-authored-by: Aryan Bhosale <36108149+aryanbhosale@users.noreply.github.com>
Pull Request
Description
Please delete the italicised instruction text!
Please include a summary of the change and which issue is fixed.
Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration
If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?
Checklist: