Fix human readable task_runtime - Githubissues

cnti-testcatalog / testsuite

📞📱☎️📡🌐 Cloud Native Telecom Initiative (CNTI) Test Catalog is a tool to check for and provide feedback on the use of K8s + cloud native best practices in networking applications and platforms

https://wiki.lfnetworking.org/display/LN/Test+Catalog

Apache License 2.0

173 stars 71 forks source link

Fix human readable task_runtime #2011

Closed martin-mat closed 4 months ago

martin-mat commented 5 months ago

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 commented 4 months ago

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