[JENKINS-36732] Use start time instead of scheduled time in build history.

[ikedam]

JENKINS-36732

There's a build

• scheduled at 10:00
• started at 11:00 (nodes were busy for long time)
• finished at 11:30

Before this fix, the build history displays the build during 10:00 - 10:30.
This fix displays the build during 11:00 - 11:30 correctly.

Before this fix:

After this fix:

[oleg-nenashev]

Looks reasonable for the default behavior. But I would vote for retaining the original behavior and making it available via request parameter or UI control (e.g. "Display with queue wait times")

[oleg-nenashev]

Ideally, it would be great to...

• display Queue Wait time on the chart with a different color (e.g. "blue" for build time and "blue with hatching" for queue wait time)
• Display "Scheduling time" and "Execution started" times in the popup

[ikedam]

But I would vote for retaining the original behavior
it would be great to...

It might be.
But I don't plan to work for those new features. They should require so many changes.
I'd like you to take over the ticket for those new features if they're required to fix the current behavior.

[daniel-beck]

👍 on this change. It's the only sensible behavior without introducing an additional wait time visualization.

The proposed option doesn't seem to make sense, as the data is simply wrong right now.

[KostyaSha]

. But I would vote for retaining the original behavior

How often do you use this trend? :)

+1 for change as it Build history and not start history.

display Queue Wait time on the chart with a different color (e.g. "blue" for build time and "blue with hatching" for queue wait time)

Queue wait timing is separate plugin feature AFAIR

[oleg-nenashev]

But I don't plan to work for those new features. They should require so many changes.
I'd like you to take over the ticket for those new features if they're required to fix the current behavior.

Nope, do not have time for it :(
As I said before, I'm fine with changing the default behavior. If others do not think the current layout makes sense, so let it be.

How often do you use this trend? :)

Rarely, because it's not very useful

[KostyaSha]

Rarely, because it's not very useful

That's no need to spend time on it and add complexities :)