Skip to content

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Sep 9, 2016

PR #2288 broke the Travis build since the key stored (from 2014) didn't have the project ID as a value.

Also adding better instructing about the encrypted file.

PR googleapis#2288 broke the Travis build since the key stored (from 2014) didn't have the project ID as a value.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 9, 2016
@dhermes
Copy link
Contributor Author

dhermes commented Sep 9, 2016

I've also deleted the old and no longer used file encryption env. vars:

  • encrypted_a1b222e8c14d_key
  • encrypted_a1b222e8c14d_iv
  • encrypted_d4f2fba7a94f_key
  • encrypted_d4f2fba7a94f_iv
@dhermes
Copy link
Contributor Author

dhermes commented Sep 9, 2016

Merging now to stop the permanent state of b0rken-ness. @tseaver and @daspecster please LMK if the "emergency" merge was problematic.

@dhermes dhermes merged commit 49b3f1b into googleapis:master Sep 9, 2016
@dhermes dhermes deleted the add-new-travis-key branch September 9, 2016 23:06
@dhermes
Copy link
Contributor Author

dhermes commented Sep 9, 2016

Was ready to say "Yay we're green!" but then:
https://travis-ci.org/GoogleCloudPlatform/google-cloud-python/builds/158880190

It's just #2080.

@tseaver
Copy link
Contributor

tseaver commented Sep 10, 2016

We could merge #2248 to make that one go away. ;)

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

4 participants