diff options
author | Felix Edel <felix.edel@bmw.de> | 2021-10-20 09:14:18 +0200 |
---|---|---|
committer | Felix Edel <felix.edel@bmw.de> | 2021-10-20 09:21:24 +0200 |
commit | 55242af7cbb7a81aab1a16760b114c876083d7fa (patch) | |
tree | cdf26fbdcb917329a075d927f109f2c4207c68d4 /web/src/containers/build/Buildset.jsx | |
parent | 22552b3ffafbb7e0bc9dd5949f0bd479ec4fed9f (diff) | |
download | zuul-55242af7cbb7a81aab1a16760b114c876083d7fa.tar.gz |
UI: Ignore empty timestamps in build time calculation on buildset page
Currently, if a buildset contains at least one build without sufficient
time information (e.g. a SKIPPED build), this build will be shown as
"Starting build" with an "Invalid date". Additionally, the calculated
duration of such a buildset will always be shown as 0.
To fix this behaviour, we simply ignore missing timestamps when
trying to find the first and last build in the buildset.
Change-Id: I9a4479c913743ebf1fc8f0133f544362dcf61aaf
Diffstat (limited to 'web/src/containers/build/Buildset.jsx')
-rw-r--r-- | web/src/containers/build/Buildset.jsx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/web/src/containers/build/Buildset.jsx b/web/src/containers/build/Buildset.jsx index 94d1994e4..a980af705 100644 --- a/web/src/containers/build/Buildset.jsx +++ b/web/src/containers/build/Buildset.jsx @@ -48,10 +48,10 @@ function Buildset({ buildset, timezone, tenant }) { function renderBuildTimes() { const firstStartBuild = buildset.builds.reduce((prev, cur) => - prev.start_time < cur.start_time ? prev : cur + !cur.start_time || prev.start_time < cur.start_time ? prev : cur ) const lastEndBuild = buildset.builds.reduce((prev, cur) => - prev.end_time > cur.end_time ? prev : cur + !cur.end_time || prev.end_time > cur.end_time ? prev : cur ) const totalDuration = (moment.utc(lastEndBuild.end_time).tz(timezone) - |