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

Documentation for adding AWS_PROFILE #95

Merged
merged 2 commits into from
Jun 11, 2018

Conversation

asobrien
Copy link
Contributor

@asobrien asobrien commented Jun 7, 2018

Fixes #94

Signed-off-by: Anthony O'Brien [email protected]

@asobrien asobrien force-pushed the feature/enable_profiles branch from a3b0fcc to 2748b56 Compare June 7, 2018 21:01
@christopherhein
Copy link
Member

@asobrien check the issue for feedback… I would say change this and the title to Documentation for adding AWS_PROFILE

@asobrien asobrien force-pushed the feature/enable_profiles branch from 2748b56 to 6586238 Compare June 8, 2018 13:47
@asobrien asobrien changed the title add support for profiles Documentation for adding AWS_PROFILE Jun 8, 2018
@asobrien
Copy link
Contributor Author

asobrien commented Jun 8, 2018

@christopherhein I've updated the PR so that it just adds documentation that covers specifying credentials and using a AWS_PROFILE. The rendered preview is here.

@christopherhein
Copy link
Member

Awesome @asobrien LGTM. @nckturner any chance you can give this a quick review?

@nckturner
Copy link
Contributor

@christopherhein Yup taking a look

Copy link
Contributor

@nckturner nckturner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, very clear and with a good example. Thanks!

@christopherhein
Copy link
Member

The travis issue … looks like a flake? Then again the goreleaser version seems out of date by quite a bit…

Signed-off-by: Anthony O'Brien <[email protected]>
@asobrien asobrien force-pushed the feature/enable_profiles branch from 47139cb to 766fe3e Compare June 11, 2018 18:39
@asobrien
Copy link
Contributor Author

@christopherhein I added a --retry to the curl command when fetching goreleaser. That should cut down on some of the flakiness in the tests.

@mattlandis mattlandis merged commit bae670c into kubernetes-sigs:master Jun 11, 2018
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

Successfully merging this pull request may close these issues.

4 participants