Added support for bundle-provided end-to-end load generator #75

Merged
merged 1 commit into from Feb 3, 2017

Conversation

Projects
None yet
2 participants
Owner

johnsca commented Feb 1, 2017

No description provided.

petevg approved these changes Feb 3, 2017

All tests pass locally, and code looks good. +1

return True
+
+
+async def e2e_output(stream, log_func):
@petevg

petevg Feb 3, 2017

Collaborator

I'd like to see how well this behaves when/if we have a massive number of messages to stdout or stderr. I know that those PIPEs can get filled up, but I'm also assuming that asyncio.gather is going to be able to keep up, and keep pulling data out of them.

It doesn't block a merge, but it is maybe something to keep an eye on as we unleash this thing on real e2e tests.

@johnsca

johnsca Feb 3, 2017

Owner

Fair enough. I was torn on whether we should even capture the stdout from the e2e script, but it's helpful for debugging, at least.

@petevg petevg merged commit c94fe8b into master Feb 3, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment