Skip to content

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Dec 8, 2016

Labeled "don't merge" in the hopes that 1.0.2 will be released soon and we don't need to depend on a release candidate. With this release, running e.g. system test eliminates the previous error (and hang on exit)

$ tox -e system-tests -- datastore
...
E1208 10:30:09.454683798   17019 network_status_tracker.c:48] Memory leaked as all network endpoints were not shut down

/cc @bjwatson @nathanielmanistaatgoogle

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 8, 2016
@dhermes dhermes added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Dec 8, 2016
deps =
grpcio
grpcio >= 1.0.2rc0
mock

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@dhermes
Copy link
Contributor Author

dhermes commented Dec 8, 2016

I deleted the CircleCI webhook, turned off the project and turned it back on, but CircleCI still won't trigger a build. What the heck

@daspecster
Copy link
Contributor

@dhermes their support email is sayhi@circleci.com.

@dhermes
Copy link
Contributor Author

dhermes commented Dec 8, 2016

@daspecster Already sent an email, thanks though

@dhermes
Copy link
Contributor Author

dhermes commented Dec 10, 2016

Now that 1.0.2 is out I'm putting it into #2846

@dhermes dhermes closed this Dec 10, 2016
@dhermes dhermes deleted the upgrade-grpcio branch December 10, 2016 00:54
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. do not merge Indicates a pull request not ready for merge, due to either quality or timing. grpc packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants