Show evaluation time in the evaluations overview

[dasJ]

[grahamc]

Looks pretty good! I think the function should probably live somewhere else, to prevent confusion if someone tries to use it elsewhere. I'm not totally sure if this is the best answer, but I'd be happy enough with it.

[grahamc]

I think this should probably move to TT.pm:

--- a/src/lib/Hydra/View/TT.pm
+++ b/src/lib/Hydra/View/TT.pm
@@ -9,7 +9,7 @@ __PACKAGE__->config(
     ENCODING => 'utf-8',
     PRE_CHOMP => 1,
     POST_CHOMP => 1,
-    expose_methods => [qw/buildLogExists buildStepLogExists jobExists stripSSHUser/]);
+    expose_methods => [qw/buildLogExists buildStepLogExists jobExists stripSSHUser relativeDuration/]);
 
 sub buildLogExists {
     my ($self, $c, $build) = @_;
@@ -34,6 +34,11 @@ sub stripSSHUser {
     }
 }
 
+sub relativeDuration {
+    my ($self, $c, $seconds) = @_;
+    return Time::Seconds->new($seconds)->pretty();
+}
+

[grahamc]

I applied my change over in master...DeterminateSystems:pr-887-review -- I propose we squash these commits if you're up for it.

[dasJ]

Like this?

[grahamc]

lgtm, thanks! just waiting for CI to be green