Bug 1527094 - Metrics capture from container manages have gaps
Summary: Metrics capture from container manages have gaps
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: C&U Capacity and Utilization
Version: 5.9.0
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: GA
: 5.9.0
Assignee: Yaacov Zamir
QA Contact: Gilad Shefer
URL:
Whiteboard:
Depends On: 1524183
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-12-18 14:45 UTC by Satoe Imaishi
Modified: 2018-03-06 15:08 UTC (History)
5 users (show)

Fixed In Version: 5.9.0.14
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1524183
Environment:
Last Closed: 2018-03-06 15:08:24 UTC
Category: ---
Cloudforms Team: Container Management
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Comment 2 CFME Bot 2017-12-18 14:55:29 UTC
New commit detected on ManageIQ/manageiq-providers-kubernetes/gaprindashvili:
https://github.com/ManageIQ/manageiq-providers-kubernetes/commit/767ed8106f6808a2d83175b9773f191a86893cbd

commit 767ed8106f6808a2d83175b9773f191a86893cbd
Author:     Mooli Tayer <mtayer>
AuthorDate: Thu Dec 14 14:00:16 2017 +0200
Commit:     Satoe Imaishi <simaishi>
CommitDate: Mon Dec 18 09:45:48 2017 -0500

    Merge pull request #187 from yaacov/metric-capture-interval-60s
    
    Make capture interval a multiple of 30s
    (cherry picked from commit 9be0e668b23e822aeaf3eef960fcb3a0f1fa04cb)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1527094

 .../container_manager/metrics_capture.rb           |  5 +-
 .../metrics_capture/capture_context_mixin.rb       | 29 +++++++---
 .../metrics_capture/hawkular_capture_context.rb    |  2 +-
 .../metrics_capture/prometheus_capture_context.rb  |  3 +-
 .../prometheus_capture_context_spec.rb             |  3 +-
 .../container_manager/metrics_capture_spec.rb      | 66 +++++++++++-----------
 6 files changed, 62 insertions(+), 46 deletions(-)

Comment 3 Yaacov Zamir 2017-12-19 09:47:48 UTC
backported upstream:
https://github.com/ManageIQ/manageiq-providers-kubernetes/pull/187


Note You need to log in before you can comment on or make changes to this bug.