-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add AWS credential #281
Add AWS credential #281
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## develop #281 +/- ##
========================================
Coverage 66.65% 66.65%
========================================
Files 56 56
Lines 4666 4666
========================================
Hits 3110 3110
Misses 1384 1384
Partials 172 172
Flags with carried forward coverage won't be shown. Click here to find out 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.
LGTM --
ops/scripts/aws_config
Outdated
@@ -0,0 +1,2 @@ | |||
[profile BobaGCP] |
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.
A bit of a nit, I don't think we necessarily need to include 'Boba' here, I'd think simply calling this the "GCP" profile would be good enough. Presumably if someone is overwriting the aws config already, they would simply overwrite this file
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.
Sure. I have renamed it.
📋 Add associated issues, tickets, docs URL here.
Overview
Describe what your Pull Request is about in a few sentences.
Add aws credential to image