public_git
/
actions-runner-controller
mirror of https://github.com/actions-runner-controller/actions-runner-controller.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Use timestamps from Workflow Job event as defaults for queued,started,completed time 

Default these values to the timestamps contained in the event. They will
be updated if the log contains more accurate values, however the
"Waiting for a runner to pick up this job..." and "Job is about to start
running on the runner:" lines are only present in the logs until the job
has finished, at which point they are removed from the job logs by
Github, so they can't be used to calculate the job duration after the
job has finished.

Fixes actions/actions-runner-controller#3522
This commit is contained in:
Jeremy Donahue 2024-05-15 16:26:29 -07:00
parent a6d87c46cd
commit ab58a4c537
1 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
pkg/actionsmetrics/event_reader.go
View File

@ -244,6 +244,22 @@ func (reader *EventReader) fetchAndParseWorkflowJobLogs(ctx context.Context, e *
startedTime time.Time
completedTime time.Time
)
// Default these values to the timestamps contained in the event. They will
// be updated if the log contains more accurate values, however the
// "Waiting for a runner to pick up this job..." and "Job is about to start
// running on the runner:" lines are only present in the logs until the job
// has finished, at which point they are removed from the job logs by
// Github, so they can't be used to calculate the job duration after the
// job has finished.
if e.WorkflowJob.CreatedAt != nil {
queuedTime = e.WorkflowJob.CreatedAt.Time
}
if e.WorkflowJob.StartedAt != nil {
startedTime = e.WorkflowJob.StartedAt.Time
}
if e.WorkflowJob.CompletedAt != nil {
completedTime = e.WorkflowJob.CompletedAt.Time
}
func() {
// Read jobLogs.Body line by line