Created attachment 327 [details] accounting and sreport patch for print energy used by user/group This patch add field consumed_energy on table *_assoc_usage_*_table and *_usage_*_table. These field contains sum of energy for step corresponing to jobs launched during hour/day/month (as cpu time used). The field is printed with: sreport user topusage sreport cluster UserUtilizationByAccount sreport cluster AccountUtilizationByUser Can you add this enhancement to 2.6? Thomas
Thomas, this is what I would consider a major change. I'll need to study it more, but it will not be put in 2.6. Introducing the energy from steps to that of the job roll up will need some testing to make sure there aren't any gotcha's. Please avoid lines that are +80 chars.
Thomas there were a few issues with this patch (commit 5fd469c6680e8cc30449f8787cb160d9ea4ed097), but on the most part it isn't bad. Packing and unpacking wasn't made backward compatible. Fixed in 379664bd8dfcf67094a6d824505cecdc00dce822. And the biggest was the placement of the "and" clause in the sql statement. This would make it so job allocations with no steps in them would not be added to the mix. Fixed in fbe292d18cd997f62d6af91df0165a44836774e4. This has been pushed to the master 13.12 branch. Let me know if you notice any issues.
