Fix partly #225.
What
Generate a final report for k8s-scaling.json result in different subdirectories which follows the current hard-coded test name k8s-scaling.
Edited files:
report/report_dockerfile/tidy_scaling.Rreport/report_dockerfile/dut-details.R
Testing done: Having the next results structure:
$ tree results/
results/
├── Env.R
├── scaling
│ └── k8s-scaling.json
└── scaling2
└── k8s-scaling2.json
2 directories, 3 files
the k8s-scaling2.json does not follow the hard-coded test name, and only for it a warning message will be displayed in the final report, see report attached.
metrics_report.pdf
Ah, I meant to leave notes on the PR here, but left them on the Issue #225 instead - please take a look @marcemq
ping @marcemq - did you see the notes on #225 ? Make sense?
whilst here, I notice this PR and commit subject names are not very descriptive (Fix partly #225. ). Can you make it a meaningful short sentence. Have a look at the repo history for examples (git log --oneline for instance). thanks!
Note, btw, I see you did add a comprehensive set of info to the PR body - that afaik does not make it into the git log history itself - sometimes it is better to add, or duplicate, that information in the commit bodies, as that does make it into git.
@grahamwhaley I'll consider your comments about PR and git meaningful short sentence.
In terms of this fix and your comments at #225, I think we need to define a configurable report, so it works with this edge cases and the behaviour of the report phase is clearer and well define, I haven't had a chance to write it formally because right know I'm focusing on scale the test at #248.