moodlehq / moodle-performance-comparison

Set of shell scripts to run Moodle performance tests using different hardware and configurations and compare results.
GNU General Public License v2.0
75 stars 39 forks source link

Ability to export data #24

Closed dmonllao closed 10 years ago

dmonllao commented 10 years ago

Many things to discuss here... Runs data is all contained in PHP files, one per run, so the fastest way to code it would be allowing exports & imports of those .php files.

I doubt whether it is worth to invest resources in creating a new import/export format as those files are pretty readable, containing the run data at the beginning. Waiting for feedback.

stronk7 commented 10 years ago

Aha, I really haven't looked inside run files (doing it now).

My main point about this is that we could decide to store all those results somewhere and perform some alternative data crunching different from current one (comparing two runs).

Things like showing along the time how every "base" evolves, or showing the results like test failures (converting to xUnit format)... are two simple ideas about what we could expect from exported data.

But, after looking to current format (var_export, it seems), surely it's enough to be able to perform those different "reports". So you can consider this to be closed, as far as they may be enough.

Just a silly comment... looking at current runs format... could we improve it a bit and normalize things like: $baseversion, $siteversion, $sitecommit ? Right now each one has its own format, and it would be far better to have both the base and site commits, and then, if needed, the base and site versions and/or the base and site releases.. but each one inside its own variable (instead of current "mix").

dmonllao commented 10 years ago
dmonllao commented 10 years ago

https://github.com/dmonllao/moodle-performance-comparison/commit/5ab85e95b7061656c321411863283b1d9737790e

dmonllao commented 10 years ago

Feel free to merge the changes if you are happy or let me know if you want me to merge them

stronk7 commented 10 years ago

if you can do it yourself... +1!

dmonllao commented 10 years ago

Thanks Eloy, merged in 25 and master. Closing