CommandRunner.run(response=pexpect.TIMEOUT) does not DTRT
Bug #1009302 reported by
Michael Hudson-Doyle
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| LAVA Dispatcher |
Fix Released
|
High
|
Michael Hudson-Doyle | ||
Bug Description
<mwhudson> if you pass pexpect.TIMEOUT in response the code still waits for the prompt
<mwhudson> and that probably won't appear
<mwhudson> so it will raise an exception
Related branches
lp:~mwhudson/lava-dispatcher/CommandRunner-timeout-bug-1009302
- Andy Doan (community): Approve
-
Diff: 12 lines (+2/-0)1 file modifiedlava_dispatcher/client/base.py (+2/-0)
| Changed in lava-dispatcher: | |
| status: | New → Fix Committed |
| importance: | Undecided → High |
| assignee: | nobody → Yongqin Liu (liuyq0307) |
| milestone: | none → 2012.06 |
Revision history for this message
| Michael Hudson-Doyle (mwhudson) wrote | #1 |
| Changed in lava-dispatcher: | |
| status: | Fix Committed → New |
| assignee: | Yongqin Liu (liuyq0307) → nobody |
| Changed in lava-dispatcher: | |
| status: | New → Fix Committed |
| status: | Fix Committed → New |
| Changed in lava-dispatcher: | |
| assignee: | nobody → Michael Hudson-Doyle (mwhudson) |
| status: | New → Incomplete |
| status: | Incomplete → In Progress |
| Changed in lava-dispatcher: | |
| status: | In Progress → Fix Committed |
| Changed in lava-dispatcher: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Hi, that branch contained a workaround for this issue, not a fix for it.