Skip to content

Install and configure Google Cloud SDK #1

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

Merged
merged 2 commits into from
Mar 14, 2017
Merged

Conversation

mlocher
Copy link
Contributor

@mlocher mlocher commented Apr 3, 2015

This is a first draft, installation works, but the authentication hasn't been tested yet. To upgrade to a newer version you need to reset the dependency cache on Codeship.

@mlocher mlocher changed the title Install and configure Google Cloud SDK (first draft) Install and configure Google Cloud SDK Apr 3, 2015
@mlocher mlocher force-pushed the google-cloud-sdk branch from 9990fac to 0cf2908 Compare April 3, 2015 18:53
sromocki pushed a commit to kevy/scripts that referenced this pull request Feb 6, 2016
Upping elasticsearch version [#110561816]
@mlocher mlocher added the blocked label Jun 7, 2016
@joesiewert joesiewert force-pushed the google-cloud-sdk branch 2 times, most recently from 8ce4346 to 2a4223a Compare March 14, 2017 03:57
@joesiewert
Copy link
Contributor

Ready for another look @mlocher.

@mlocher
Copy link
Contributor Author

mlocher commented Mar 14, 2017

can't approve, as I'm one of the authors :D, looks great though.

Two things we might want to keep in mind and address in a future PR

  • the cached download doesn't expire at the moment
  • should we exit with an error code if ${GOOGLE_CLOUD_KEY} and ${GOOGLE_CLOUD_PROJECT_ID} are not set?

@ethangj could you take a look?

@mlocher mlocher requested a review from ethangj March 14, 2017 07:53
Copy link
Contributor

@ethangj ethangj left a comment

Choose a reason for hiding this comment

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

lgtm!

@joesiewert joesiewert merged commit 13d463e into master Mar 14, 2017
@joesiewert joesiewert deleted the google-cloud-sdk branch March 14, 2017 22:00
@mlocher
Copy link
Contributor Author

mlocher commented Mar 15, 2017

great we finally got the PR #1 shipped (and a little sad it took that long to get it finished). Thanks @joesiewert for taking over!

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

Successfully merging this pull request may close these issues.

3 participants