This was a feature that was present in 0.25.x and was inadvertently dropped from 2.6.x.
Signed-off-by: Paul Berry <[email protected]> --- lib/puppet/transaction/report.rb | 2 ++ spec/unit/transaction/report_spec.rb | 9 +++++++++ 2 files changed, 11 insertions(+), 0 deletions(-) diff --git a/lib/puppet/transaction/report.rb b/lib/puppet/transaction/report.rb index 8a92845..bcdefd0 100644 --- a/lib/puppet/transaction/report.rb +++ b/lib/puppet/transaction/report.rb @@ -149,6 +149,8 @@ class Puppet::Transaction::Report metrics[name.to_s.downcase] = value end + metrics["total"] = metrics.values.inject(0) { |a,b| a+b } + add_metric(:time, metrics) end end diff --git a/spec/unit/transaction/report_spec.rb b/spec/unit/transaction/report_spec.rb index f87f103..1678c09 100755 --- a/spec/unit/transaction/report_spec.rb +++ b/spec/unit/transaction/report_spec.rb @@ -200,6 +200,15 @@ describe Puppet::Transaction::Report do @report.calculate_metrics metric(:time, "foobar").should == 50 end + + it "should have a total time" do + add_statuses(3, :file) do |status| + status.evaluation_time = 1.25 + end + @report.add_times :config_retrieval, 0.5 + @report.calculate_metrics + metric(:time, "total").should == 4.25 + end end describe "for events" do -- 1.7.2 -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.
