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.

Sign up for GitHub

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

Jump to bottom

cmd/tsuru: unit-add gives "unparseable chunk" error when sending "please wait" #1151

Closed
fsouza opened this issue Mar 26, 2015 · 1 comment
Closed

cmd/tsuru: unit-add gives "unparseable chunk" error when sending "please wait" #1151

fsouza opened this issue Mar 26, 2015 · 1 comment
Labels
bug
Milestone
0.12.0

Comments

@fsouza
Copy link
Contributor

fsouza commented Mar 26, 2015 

% tsuru unit-add 3

---- Starting 3 new units ----
 ---> Started unit 26cf3f4a16...
 ---> Started unit 456cd4f6de...
Error: Unparseable chunk: "please wait...\n"
@fsouza fsouza added the bug label Mar 26, 2015
@cezarsa
Copy link
Member

cezarsa commented Mar 26, 2015

I'm rather intrigued by this bug. unit-add doesn't create a KeepAliveWriter instance. The only place it's created is in the deploy handler in api/deploy.go. Also, the only place "please wait" string appears is in this same handler.
I don't see how a "please wait" string can show up during a unit-add operation. There is something really wrong going on.

@tarsisazevedo tarsisazevedo modified the milestone: 0.12.0 May 7, 2015
fsouza pushed a commit that referenced this issue May 14, 2015
api: Stop the keepAliveWriter before exitting the deploy handler
77ecb81 
Should fix issue #1151.
@fsouza fsouza closed this as completed May 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
0.12.0
Development

No branches or pull requests
3 participants
@cezarsa @fsouza @tarsisazevedo