Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Handle empty ceph_version in ceph_mon_metadata to avoid raising misleading alert #92

Merged
merged 6 commits into from
Feb 2, 2022
Merged

Handle empty ceph_version in ceph_mon_metadata to avoid raising misleading alert #92

merged 6 commits into from
Feb 2, 2022

Commits on Oct 6, 2021

  1. Adding 'namespace' to the 'ceph_node_down' query 

    Signed-off-by: Arun Kumar Mohan <amohan@redhat.com>
    @aruniiird @GowthamShanmugam
    aruniiird authored and GowthamShanmugam committed Oct 6, 2021
    Configuration menu
    Copy the full SHA
    937f993 View commit details
    Browse the repository at this point in the history

  2. Change CephAbsentMgr to use 'up' query 

    Instead of using 'absent' query, we are trying to use 'up' which should
provide us with the needed 'namespace' field in the resultant metrics

Signed-off-by: aruniiird <amohan@redhat.com>
    @aruniiird @GowthamShanmugam
    aruniiird authored and GowthamShanmugam committed Oct 6, 2021
    Configuration menu
    Copy the full SHA
    fcf1565 View commit details
    Browse the repository at this point in the history

  3. Adding namespace field into other alert queries 

    Signed-off-by: Arun Kumar Mohan <amohan@redhat.com>
    @aruniiird @GowthamShanmugam
    aruniiird authored and GowthamShanmugam committed Oct 6, 2021
    Configuration menu
    Copy the full SHA
    e0b35f4 View commit details
    Browse the repository at this point in the history

  4. Increasing the auto-resolvable alerts' delay to 15m 

    The following alerts,

CephMonHighNumberOfLeaderChanges
CephOSDDiskNotResponding
CephClusterWarningState

, which are resolved automatically, in most cases,
are causing unnecessary admin events. So we are increasing the
alert delay time to '15m'.

Signed-off-by: aruniiird <amohan@redhat.com>
    @aruniiird @GowthamShanmugam
    aruniiird authored and GowthamShanmugam committed Oct 6, 2021
    Configuration menu
    Copy the full SHA
    ae51e28 View commit details
    Browse the repository at this point in the history

  5. Reverting the time delay of 'CephMonHighNumberOfLeaderChanges' 

    Reverting the time delay of
'CephMonHighNumberOfLeaderChanges' back to 5m

Signed-off-by: Arun Kumar Mohan <amohan@redhat.com>
    @aruniiird @GowthamShanmugam
    aruniiird authored and GowthamShanmugam committed Oct 6, 2021
    Configuration menu
    Copy the full SHA
    082c58a View commit details
    Browse the repository at this point in the history

  6. Bug 1970354: Handle empty ceph_version in ceph_mon_metadata to avoid … 

    …raising misleading alert

Signed-off-by: Gowtham Shanmugasundaram <gshanmug@redhat.com>
    @GowthamShanmugam
    GowthamShanmugam committed Oct 6, 2021
    Configuration menu
    Copy the full SHA
    a5fa42d View commit details
    Browse the repository at this point in the history