Bug 1309530

Summary:	recipe start time is a random handful of seconds later than rebooted time
Product:	[Retired] Beaker	Reporter:	Dan Callaghan <dcallagh>
Component:	scheduler	Assignee:	Dan Callaghan <dcallagh>
Status:	CLOSED CURRENTRELEASE	QA Contact:	tools-bugs <tools-bugs>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	22	CC:	dcallagh, dowang, mjia, rjoost
Target Milestone:	23.0	Keywords:	Patch
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-07-07 23:10:18 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Dan Callaghan 2016-02-18 03:33:15 UTC

Recipes have an overall start and finish time (start_time and finish_time in the database). Beaker also tracks the timestamp at which the system was successfully rebooted in order to kick off the installation at the beginning of the recipe.

Currently the recipe start time is approximately 0-20 seconds later than the rebooted time, because recipe.start_time is populated by beakerd's update_dirty_jobs thread. It runs in a loop every 20 seconds, and when it finds a recipe which is running but has no start time, it sets the start time to now.

This is particularly noticeable in the new recipe UI, which presents the installation progress including relative timestamps. For all our existing recipes, Beaker claims that the reboot happened a few seconds *before* the recipe started which doesn't make much sense.

Comment 1 Dan Callaghan 2016-02-18 04:49:38 UTC

I think the best option is to record the start time as the earliest of rebooted time, first task start time, or now. (Note that for systems without power control, there is no reboot command so there is no rebooted timestamp, the system simply appears and the first Beaker sees is when the system has checked in after booting Anaconda.)

http://gerrit.beaker-project.org/4692

Comment 2 Dan Callaghan 2016-02-19 06:43:57 UTC

Patch is merged, but only into results-reporting-improvements-take2 branch for now.

Comment 3 Mike McCune 2016-03-28 22:26:18 UTC

This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions

Comment 4 Dan Callaghan 2016-04-05 07:54:50 UTC

The results-reporting-improvements-take3 branch has been merged to develop.

Comment 7 Dan Callaghan 2016-07-07 23:10:18 UTC

Beaker 23.0 has been released.