Container logs operation hangs until timeout

Bug #1501498 reported by Andrew Melton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Undecided
Andrew Melton

Bug Description

It appears we are using the old style method to get logs where dockerpy attaches to the container and attempts to stream the logs. That appears to hang with our current set up. This doesn't appear to happen with the newer 'logs' operation exposed by dockerpy.

Steps to reproduce:
1) Spawn Swarm Bay
2) Create a container
3) Run magnum container-start <container>
4) Run magnum container-logs <container>
5) Magnum client should hang until it gets a timeout exception from Magnum

Changed in magnum:
assignee: nobody → Andrew Melton (andrew-melton)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to magnum (master)

Fix proposed to branch: master
Review: https://review.openstack.org/229627

Changed in magnum:
status: New → In Progress
Revision history for this message
Egor Guz (eghobo) wrote :

@Andrew: just curious is it's atomic 2 or 5. Keep in mind that 5 has Docker 1.7.1

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

Reviewed: https://review.openstack.org/229627
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=63cd7f4017aeff5eac9fe5c8713ec27954c58d09
Submitter: Jenkins
Branch: master

commit 63cd7f4017aeff5eac9fe5c8713ec27954c58d09
Author: Andrew Melton <email address hidden>
Date: Wed Sep 30 16:44:40 2015 -0400

    Use dockerpy logs operation instead of attach

    Attach is the old way of retrieving container logs, and it hangs
    when used with Swarm. Dockerpy handles the code to switch between
    the old attach and the new logs operation depending on remote
    version, so this should be a backwards compatible change.

    Change-Id: I224ee41c634a2c7cc1aa63e0495668063f66db80
    Closes-Bug: #1501498

Changed in magnum:
status: In Progress → Fix Committed
Adrian Otto (aotto)
Changed in magnum:
milestone: none → mitaka-1
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.