Skip to content

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Sep 9, 2016

This is because the project ID can be determined directly from the file pointed to by GOOGLE_APPLICATION_CREDENTIALS.

Fixes #2263.

@dhermes dhermes added the testing label Sep 9, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 9, 2016
@dhermes dhermes changed the title Stop uisng a designated tests project ID. Stop using a designated tests project ID. Sep 9, 2016
@tseaver
Copy link
Contributor

tseaver commented Sep 9, 2016

LGTM. Lots of pink lines!

@dhermes
Copy link
Contributor Author

dhermes commented Sep 9, 2016

Blerg! Not quite enough red lines: lint says we got some unused imports.

This is because the project ID can be determined directly from
the file pointed to by GOOGLE_APPLICATION_CREDENTIALS.

Fixes googleapis#2263.
@dhermes dhermes merged commit d42b020 into googleapis:master Sep 9, 2016
@dhermes dhermes deleted the fix-2263 branch September 9, 2016 20:37
@dhermes
Copy link
Contributor Author

dhermes commented Sep 9, 2016

OK this broke things big time:
https://travis-ci.org/GoogleCloudPlatform/google-cloud-python/builds/158845211

Looking into it.

@dhermes
Copy link
Contributor Author

dhermes commented Sep 9, 2016

OK I think I know what happened, the key stored in key.json.enc was an older style key and doesn't have a project_id key.

dhermes added a commit to dhermes/google-cloud-python that referenced this pull request Sep 9, 2016
PR googleapis#2288 broke the Travis build since the key stored (from 2014)
didn't have the project ID as a value.
@dhermes dhermes mentioned this pull request Sep 9, 2016
@dhermes
Copy link
Contributor Author

dhermes commented Sep 9, 2016

Sent #2292 to fix.

@dhermes
Copy link
Contributor Author

dhermes commented Sep 9, 2016

b0rken here as well:
https://travis-ci.org/GoogleCloudPlatform/google-cloud-python/builds/158865134

Though this is even more b0rken since I deleted the IV and key env. vars. from Travis.

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

Labels

cla: yes This human has signed the Contributor License Agreement. testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants