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

Making emulator scripts wait until emulators are ready. #1417

Merged
merged 1 commit into from
Jan 28, 2016

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Jan 25, 2016

Also manually handling clean-up of child processes spawned. The gcloud tool was orphan-ing some processes.


NOTE: Has #1400 as diffbase.

Filed googleapis/google-cloud-datastore#87 in response to orphan-ed processes.

@dhermes dhermes added api: datastore Issues related to the Datastore API. testing api: pubsub Issues related to the Pub/Sub API. labels Jan 25, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 25, 2016
@dhermes
Copy link
Contributor Author

dhermes commented Jan 25, 2016

Another option would be to create a process group before calling subprocess, but I haven't tried to implement, so I'm not sure it'll work.

Also manually handling clean-up of child processes spawned.
The ``gcloud`` tool was orphan-ing some processes.
@dhermes dhermes force-pushed the emulator-less-flaky branch from 034d83f to 92d956d Compare January 28, 2016 20:13
dhermes added a commit that referenced this pull request Jan 28, 2016
Making emulator scripts wait until emulators are ready.
@dhermes dhermes merged commit 2911bd9 into googleapis:master Jan 28, 2016
@dhermes dhermes deleted the emulator-less-flaky branch January 28, 2016 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: datastore Issues related to the Datastore API. api: pubsub Issues related to the Pub/Sub API. cla: yes This human has signed the Contributor License Agreement. testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants