Skip to content

Conversation

@chemelnucfin
Copy link
Contributor

Thanks @jonparrott.

@chemelnucfin chemelnucfin added api: spanner Issues related to the Spanner API. type: process A process-related concern. May include testing, release, or the like. labels Feb 7, 2018
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 7, 2018
@theacodes
Copy link
Contributor

Updated the changelog slightly, as the gapic changes happened during the last release.

@chemelnucfin
Copy link
Contributor Author

@jonparrott Should I squash it on my end and repush? otherwise it'd be hard to tag?

@theacodes
Copy link
Contributor

nah we'll squash on merge.

@theacodes theacodes merged commit 4718cdc into googleapis:master Feb 7, 2018
@chemelnucfin chemelnucfin deleted the spanner_release branch February 7, 2018 18:01
@theacodes
Copy link
Contributor

@chemelnucfin the system tests in the tag build failed https://circleci.com/gh/GoogleCloudPlatform/google-cloud-python/5408

this seems to just be due to flakiness considering it passed on Python 3.6, so I'm going to manually push the library to pypi, but could you take a look at the test?

@chemelnucfin
Copy link
Contributor Author

on it.

@chemelnucfin
Copy link
Contributor Author

It might be a server issue as my tests pass. I'll see if it can be retried on that error.

@theacodes
Copy link
Contributor

theacodes commented Feb 7, 2018 via email

@chemelnucfin
Copy link
Contributor Author

looks like the test retries on ServerErrors, which does not retry on ClientError which 409 is...

chemelnucfin added a commit to chemelnucfin/google-cloud-python that referenced this pull request Feb 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the Spanner API. cla: yes This human has signed the Contributor License Agreement. type: process A process-related concern. May include testing, release, or the like.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants