Bug 710524

Summary: [Beaker] cmdline job-watch --nowait not working as expected
Product: [Retired] Beaker Reporter: Jeff Burke <jburke>
Component: command lineAssignee: Bill Peck <bpeck>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 0.6CC: bpeck, dcallagh, jstancek, mcsontos, pbunyan, rmancy, stl
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-09-08 02:43:18 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Jeff Burke 2011-06-03 15:37:14 UTC 
Description of problem:
The --nowait option is not working as expected.

This is from a completed job:

$ bkr job-watch J:84399
Watching tasks (this may be safely interrupted)...
J:84399 kernel-2.6.18-260.el5.kpq1 KernelTier1: Completed
--> Completed: 1 [total: 1]
*Returns you to the prompt (as expected)

$ bkr job-watch --nowait J:84399 
*Not sure why it return nothing? shouldn't it return the
same thing as the command without the --nowait

--------------------------------------------------

This is from a job that was running at the time I ran the command

$ bkr job-watch J:92788
Watching tasks (this may be safely interrupted)...
J:92788 ipa sssd testing: Running
--> Running: 1 [total: 1]
*Stays connected (as expected)

$ bkr job-watch --nowait J:92788
*should return the same thing as the command without the --nowait
but give back the prompt
Comment 1 Bill Peck 2011-06-03 17:09:41 UTC 
I'm not even sure why we have --nowait for job-watch.  The option doesn't make any sense and will be removed.
Comment 2 Marian Csontos 2011-06-03 18:30:51 UTC 
(In reply to comment #1)
> I'm not even sure why we have --nowait for job-watch.  The option doesn't make
> any sense and will be removed.

Backwards compatibility, I guess.
Comment 4 Dan Callaghan 2011-09-08 02:43:18 UTC 
Beaker 0.7.1 has been released.