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

Stop using a designated tests project ID. #2288

Merged
merged 1 commit into from
Sep 9, 2016

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