Skip to content
This repository was archived by the owner on Jul 6, 2018. It is now read-only.

Monkey patch EventOutputStream #49

Closed
mwrock opened this issue May 31, 2015 · 1 comment
Closed

Monkey patch EventOutputStream #49

mwrock opened this issue May 31, 2015 · 1 comment

Comments

@mwrock
Copy link
Contributor

mwrock commented May 31, 2015

As much as I dont think this is a great idea. Unless its planned to merge chef/chef#2100 soon, seems like this would be a reasonable temporary workaround. Otherwise one is left flying blind during convergence. For example the couchbase cookbook waits for its port to respond and can hang quite a while and one is left wondering, "is it still downloading the couch package or is it hanging on the port"?
@mwrock
Copy link
Contributor Author

mwrock commented Jun 8, 2015

i'm working on a PR to make the transport more simple and leverage Docker::Container.exec which would send output via Transport.stream_chunk.

@mwrock mwrock closed this as completed Jun 8, 2015
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mwrock