Allow streaming when using exec

[Intrepidd]

Related to #263

[Intrepidd]

Cassettes need to be re-recorded, do you want me to do it ?

[bfulton]

Thanks! Yes, please re-record.

[Intrepidd]

Running rake vcr:spec on master throws some errors for me :

rspec ./spec/docker/container_spec.rb:113 # Docker::Container#rename renames the container
rspec ./spec/docker/image_spec.rb:314 # Docker::Image.create when the Image does not yet exist and the body is a Hash sets the id and sends Docker.creds
rspec ./spec/docker/image_spec.rb:375 # Docker::Image.save when a filename is specified exports tarball of image to specified file
rspec ./spec/docker/image_spec.rb:383 # Docker::Image.save when no filename is specified returns raw binary data as string
rspec ./spec/docker_spec.rb:181 # Docker#info returns the info as a Hash

I'm on OSX Yosemite with the latest boot2docker.

I've done everything stated in TESTING.md

Not sure if the culprit is boot2docker or anything else going wrong on my system.

[Intrepidd]

I only re-recorded the cassettes I needed and now it's green :)

[bfulton]

Hmm. CI is still failing. I can take a look this evening.

[Intrepidd]

It's because of a complexity violation :/

On Mon, Apr 6, 2015, 15:48 Bright Fulton [email protected] wrote:

Hmm. CI is still failing. I can take a look this evening.

—
Reply to this email directly or view it on GitHub
#272 (comment).

[tlunter]

Can you share how you're going about the streaming?

I used the following with a simple container running just bash and it's streaming for me just fine.

c = Docker::Container.all.first
c.exec(["/bin/bash","-c","while [ 1 ]; do echo hello; sleep 5; done"]) do |stream, chunk|
  puts "#{stream}: #{chunk}"
end

[Intrepidd]

I'm doing exactly the same way, and I only get a response when the command
is finished.

I'm on OSX and using boot2docker, not sure if this changes anything

On Tue, Apr 7, 2015, 21:49 Todd Lunter [email protected] wrote:

Can you share how you're going about the streaming?

I used the following with a simple container running just bash and it's
streaming for me just fine.

c = Docker::Container.all.first
c.exec(["/bin/bash","-c","while [ 1 ]; do echo hello; sleep 5; done"]) do |stream, chunk|
puts "#{stream}: #{chunk}"
end

—
Reply to this email directly or view it on GitHub
#272 (comment).

[Intrepidd]

I ran your code to be extra sure and I'm having no output at all since it's an infinite loop.

This is very strange that you're getting it to work on your development environment

[tlunter]

I wonder if this has to do with SSL. You're using one of the latest boot2docker, right?

[Intrepidd]

The latest, and I have to disable ssl verification

On Tue, Apr 7, 2015, 23:04 Todd Lunter [email protected] wrote:

I wonder if this has to do with SSL. You're using one of the latest
boot2docker, right?

—
Reply to this email directly or view it on GitHub
#272 (comment).

[tlunter]

I have done the same and using the code above, this PR isn't working for me. I am thinking this has to do with Excon's SSLSocket versus regular Socket. Using tcpdump I can see the 5 second echo payload, but cannot see Excon producing anything from it. Will need to look into deeper.

[danielmanesku]

@tlunter any updates on this one?
I tried to apply those changes, and it does not seem to work for me.

[tlunter]

@danielmanesku I haven't been able to look into this one since my last post. Would love it if you could take a deep dive if you're available to do so.

[danielmanesku]

Thanks for a quick reply! I am busy at the moment with something else, but I added this on my TODO list and I will try to investigate in the coming weeks.