Red Hat Bugzilla – Bug 1231069
Duplicate data and graphs on Optimize->Utilization pages with time profiles that have C&U data roll up enabled
Last modified: 2015-12-08 08:13:32 EST
Created attachment 1037917 [details] screenshot of optimize summary tab Description of problem: ----------------------- Version-Release number of selected component (if applicable): ------------------------ 5.4.0.5 How reproducible: ----------------- Always Steps to Reproduce: ------------------ 1.Enable C&U collection 2.Create mutliple time profiles,enable C&U data roll up for the time profiles 3.Navigate to the Optimize->Utilization page for a host. Actual results: -------------- 1)Observe that the Report tab has duplicate data. The following data is duplicated in the CPU and Memory sections. Total,Max used,Avg used 2)Summary tab has duplicate charts for memory and CPU Utilization The 'Total' row is present twice in the CPU and Memory sections. Expected results: ---------------- No duplicate data should be present in the Optimize->Utilization page. Additional info: --------------- Also,refer to https://bugzilla.redhat.com/show_bug.cgi?id=1230786 where I've reported a similar issue related to daily C&U graphs.
Created attachment 1037918 [details] screenshot of Optimize report tab
This is most likely a duplicate of https://bugzilla.redhat.com/show_bug.cgi?id=1230786
Merge request: http://gitlab.cloudforms.lab.eng.rdu2.redhat.com/cloudforms/cfme/merge_requests/205
Resetting the NEEDINFO flag
New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=3b1c6250f57bc3f1dce260480b0de70af28e635c commit 3b1c6250f57bc3f1dce260480b0de70af28e635c Author: Gregg Tanzillo <gtanzill@redhat.com> AuthorDate: Thu Jul 30 08:23:01 2015 -0400 Commit: Gregg Tanzillo <gtanzill@redhat.com> CommitDate: Thu Jul 30 15:59:30 2015 -0400 Treat classes that have "derived" instances (instances_are_derived? => true) the same as classes that derive from ActsAsArModel and call custom find method. This was preventing the custom find method in VimPerformanceDaily from being called which caused time profiles to be ignored and return multiple rows for the same day when multiple time profiles exist. https://bugzilla.redhat.com/show_bug.cgi?id=1231069 https://bugzilla.redhat.com/show_bug.cgi?id=1230786 vmdb/app/models/rbac.rb | 2 +- vmdb/app/models/vim_performance_daily.rb | 2 ++ vmdb/spec/models/rbac_spec.rb | 7 +++++++ 3 files changed, 10 insertions(+), 1 deletion(-)
New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=95a733b9bca72866af6e0d1ac1d3084384d8c46a commit 95a733b9bca72866af6e0d1ac1d3084384d8c46a Author: Gregg Tanzillo <gtanzill@redhat.com> AuthorDate: Fri Jul 31 16:50:04 2015 -0400 Commit: Gregg Tanzillo <gtanzill@redhat.com> CommitDate: Fri Jul 31 16:50:04 2015 -0400 Added tests for daily performance find with time profile, limit and offset. https://bugzilla.redhat.com/show_bug.cgi?id=1231069 https://bugzilla.redhat.com/show_bug.cgi?id=1230786 vmdb/spec/models/vim_performance_daily_spec.rb | 77 ++++++++++++++++++++++++++ 1 file changed, 77 insertions(+) create mode 100644 vmdb/spec/models/vim_performance_daily_spec.rb
New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=67e3cbd9074acd96b9982e82f1908b769058d0ef commit 67e3cbd9074acd96b9982e82f1908b769058d0ef Merge: fa41d2f 9a94873 Author: Dan Clarizio <dclarizi@redhat.com> AuthorDate: Mon Aug 3 14:55:15 2015 -0400 Commit: Dan Clarizio <dclarizi@redhat.com> CommitDate: Mon Aug 3 14:55:15 2015 -0400 Merge branch 'fix-timelines-in-daily-cu' into '5.4.z' Fixes issue where timelines are ignored and multiple days from daily performance are displayed in C&U detailed reports. Treat classes that have "derived" instances (instances_are_derived? => true) the… same as classes that derive from ActsAsArModel and call custom find method. This was preventing the custom find method in VimPerformanceDaily from being called which caused time profiles to be ignored and return multiple rows for the same day when multiple time profiles exist. https://bugzilla.redhat.com/show_bug.cgi?id=1231069 https://bugzilla.redhat.com/show_bug.cgi?id=1230786 See merge request !205 vmdb/app/models/rbac.rb | 2 +- vmdb/app/models/vim_performance_daily.rb | 2 + vmdb/spec/models/rbac_spec.rb | 7 +++ vmdb/spec/models/vim_performance_daily_spec.rb | 79 ++++++++++++++++++++++++++ 4 files changed, 89 insertions(+), 1 deletion(-)
New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=9a9487320d7f01103d3b8ab8ce924e890aece311 commit 9a9487320d7f01103d3b8ab8ce924e890aece311 Author: Gregg Tanzillo <gtanzill@redhat.com> AuthorDate: Fri Jul 31 17:05:06 2015 -0400 Commit: Gregg Tanzillo <gtanzill@redhat.com> CommitDate: Fri Jul 31 17:05:06 2015 -0400 Added explicit ordering in the test to ensure the expected result is in the correct sequece. https://bugzilla.redhat.com/show_bug.cgi?id=1231069 https://bugzilla.redhat.com/show_bug.cgi?id=1230786 vmdb/spec/models/vim_performance_daily_spec.rb | 2 ++ 1 file changed, 2 insertions(+)
New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=70766d7650a7de235fd0dea88765098a07ef2f54 commit 70766d7650a7de235fd0dea88765098a07ef2f54 Author: Gregg Tanzillo <gtanzill@redhat.com> AuthorDate: Thu Jul 30 16:36:51 2015 -0400 Commit: Gregg Tanzillo <gtanzill@redhat.com> CommitDate: Thu Jul 30 16:38:15 2015 -0400 Fixed incorrect call to limit on offset line. https://bugzilla.redhat.com/show_bug.cgi?id=1231069 https://bugzilla.redhat.com/show_bug.cgi?id=1230786 vmdb/app/models/vim_performance_daily.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Verified in 5.5.0.5
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2015:2551