Fix human readable task_runtime

[martin-mat]

Refs: #2010

This fixes 3 issues introduced by adding human readable task_runtime:

• commented code left out
• double "seconds" word in the logs
• inconsistent and difficult-to-parse format of task_runtime in the results log yaml

Description

(name of issue/change)

Issues:

Refs: #NNN

How has this been tested:

• Covered by existing integration testing
• Added integration testing to cover
• Verified all A/C passes
  • develop
  • master
  • tag/other branch
• Test environment
  • Shared Packet K8s cluster
  • New Packet K8s cluster
  • Kind cluster
• Have not tested

Types of changes:

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update

Checklist:

Documentation

• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• No updates required.

Code Review

• Does the test handle fatal exceptions, ie. rescue block

Issue

• Tasks in issue are checked off

[HashNuke]

LGTM

[HashNuke]

LGTM

[HashNuke]

Build passing - https://github.com/cnti-testcatalog/testsuite/actions/runs/8980928494