Created attachment 1037727 [details] screenshot of C&U charts with multiple daily records 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.View C&U daily charts for VMs,hosts etc.While viewing the C&U charts,select any of the time profiles for which C&U data roll up is enabled. Actual results: ---------------- The charts contain multiple daily data records instead of a single record. The attached screenshot shows 3 records for every day since C&U collection was started.This is from an appliance with 3 time profiles for which C&U data roll up has been enabled. Expected results: ---------------- The charts should contain single daily record. Additional info: ----------------
Created attachment 1037915 [details] screenshot of C&U graph
It's my understanding that the rollup code creates a data point per user with a unique timezone. I checked the metrics rollup table and there are 3 records for the 3 different time profiles on my appliance.Does that mean everything is okay with the backend? vmdb_production=# select timestamp, created_on from metric_rollups_06 where resource_id = 201 and capture_interval_name = 'daily'\x\g\x timestamp | 2015-06-09 00:00:00 created_on | 2015-06-10 00:01:47.353607 -[ RECORD 9 ]-------------------------- timestamp | 2015-06-09 00:00:00 created_on | 2015-06-10 00:01:47.294415 -[ RECORD 10 ]------------------------- timestamp | 2015-06-09 04:00:00 created_on | 2015-06-10 04:01:43.417271 -[ RECORD 11 ]------------------------- timestamp | 2015-06-10 00:00:00 created_on | 2015-06-11 00:03:33.835257 -[ RECORD 12 ]------------------------- timestamp | 2015-06-10 00:00:00 created_on | 2015-06-11 00:03:33.478702 -[ RECORD 13 ]------------------------- timestamp | 2015-06-10 04:00:00 created_on | 2015-06-11 04:01:54.494068 -[ RECORD 14 ]------------------------- timestamp | 2015-06-11 00:00:00 created_on | 2015-06-12 00:02:44.967826 -[ RECORD 15 ]------------------------- timestamp | 2015-06-11 00:00:00 created_on | 2015-06-12 00:02:45.112775 -[ RECORD 16 ]------------------------- timestamp | 2015-06-11 04:00:00 created_on | 2015-06-12 04:01:45.412357
Would it be possible to get an export of the DB that was used to recreate this bug? I've created a new DB and started collecting C&U but I need at least 2 days of collection in order to see daily data on reports. If you still have it, great. If not, I'll wait until I have enough data and hopefully recreate the bug. Thanks,
(In reply to Gregg Tanzillo from comment #4) > Would it be possible to get an export of the DB that was used to recreate > this bug? I've created a new DB and started collecting C&U but I need at > least 2 days of collection in order to see daily data on reports. If you > still have it, great. If not, I'll wait until I have enough data and > hopefully recreate the bug. Thanks, Don't worry about getting the DB. I was able to recreate with my local DB.
Merge request: http://gitlab.cloudforms.lab.eng.rdu2.redhat.com/cloudforms/cfme/merge_requests/205
Resetting the NEEDINFO flag based on Gregg's comment 5
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> AuthorDate: Thu Jul 30 08:23:01 2015 -0400 Commit: Gregg Tanzillo <gtanzill> 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> AuthorDate: Fri Jul 31 16:50:04 2015 -0400 Commit: Gregg Tanzillo <gtanzill> 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> AuthorDate: Mon Aug 3 14:55:15 2015 -0400 Commit: Dan Clarizio <dclarizi> 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> AuthorDate: Fri Jul 31 17:05:06 2015 -0400 Commit: Gregg Tanzillo <gtanzill> 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> AuthorDate: Thu Jul 30 16:36:51 2015 -0400 Commit: Gregg Tanzillo <gtanzill> 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(-)
Verification blocked by https://bugzilla.redhat.com/show_bug.cgi?id=1264218
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