pmon alarm clear

Bug #1826708 reported by yanxubin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Won't Fix
Low
yanxubin

Bug Description

Title
-----

it's logic error to judge clear alarm in manage_alarm

Brief Description
-----------------
file :pmonHdlr.cpp
function: manage_alarm(process_config_type * ptr, int action)

 if ( action == PMON_CLEAR )
    {
        if ( ptr->alarm_severity != FM_ALARM_SEVERITY_CLEAR )
        {
            ilog ("%s from '%s' to 'clear'\n", ptr->process, alarmUtil_getSev_str(ptr->alarm_severity).c_str());
            pmonAlarm_clear ( ctrl_ptr->my_hostname, PMON_ALARM_ID__PMOND, processInfo );
            ptr->alarm_severity = FM_ALARM_SEVERITY_CLEAR ;
        }
        ptr->failed = false ;
    }

if action is PMON_CLEAR and ptr->alarm_severity is FM_ALARM_SEVERITY_CLEAR, we should call pmonAlarm_clear(),
so change "if ( ptr->alarm_severity != FM_ALARM_SEVERITY_CLEAR )" to ptr->alarm_severity == FM_ALARM_SEVERITY_CLEAR

Severity
--------
Major

Tags: stx.fault
yanxubin (yanxubin)
Changed in starlingx:
assignee: nobody → yanxubin (yanxubin)
Revision history for this message
Frank Miller (sensfan22) wrote :

Please use the StarlingX bug reporting template when creating new LPs.
Template is here: https://wiki.openstack.org/wiki/StarlingX/BugTemplate
Process is here: https://wiki.openstack.org/wiki/StarlingX/Defect_Handling_Process

Changed in starlingx:
status: New → Incomplete
Revision history for this message
Ghada Khalil (gkhalil) wrote :

Not release gating given a reproduction scenario is not provided. Issue appears to be found by code inspection. Would be nice to fix, but not high priority. Please feel free to fix and review the code changes with the fault core (Tao Liu)

tags: added: stx.fault
Changed in starlingx:
importance: Undecided → Low
status: Incomplete → Triaged
Revision history for this message
Ramaswamy Subramanian (rsubrama) wrote :

No progress on this bug for more than 2 years. Candidate for closure.

If there is no update, this issue is targeted to be closed by Jan 11th.

Revision history for this message
Ramaswamy Subramanian (rsubrama) wrote :

Changing the status to 'Won't Fix' as there is no activity.

Changed in starlingx:
status: Triaged → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.