Menu

#1897 Incorrect ER messages in syslog

5.0.2
fixed
Gary Lee
None
defect
amf
-
major
2017-01-20
2016-06-28
elunlen
No

Several incorrectly prioritized syslog messages found in SG_2N::si_swap in sg_2n_fsm.cc
LOG_ER is used when problem is easily recoverable e.g. return value is SA_AIS_ERR_TRY_AGAIN

Related

Tickets: #1897
Wiki: ChangeLog-5.0.2
Wiki: ChangeLog-5.1.1

Discussion

  • Anders Widell

    Anders Widell - 2016-09-20
    • Milestone: 4.7.2 --> 5.0.2
     
  • Long H Buu Nguyen

    • status: unassigned --> assigned
     
  • Long H Buu Nguyen

    I think amfd should use LOG_NO in this case since it returns TRY_AGAIN.
    It is also to be consistent to si_swap() function in sg_nway_fsm.cc.

     
  • Long H Buu Nguyen

    • status: assigned --> review
     
  • Long H Buu Nguyen

    • status: review --> fixed
     
  • Gary Lee

    Gary Lee - 2017-01-20
    • status: fixed --> accepted
    • assigned_to: Gary Lee
     
  • Gary Lee

    Gary Lee - 2017-01-20

    Reopening as changes in this ticket were reverted when #1294 was pushed. We need to reapply the changes.

     
  • Gary Lee

    Gary Lee - 2017-01-20

    changeset: 8526:88555159b006
    branch: opensaf-5.0.x
    parent: 8516:72b4ef9124d8
    user: Gary Lee gary.lee@dektech.com.au
    date: Fri Jan 20 16:02:54 2017 +1100
    summary: amfd: use more appropriate log severity levels in si_swap_validate() [#1897]

    changeset: 8525:7336414d6a07
    branch: opensaf-5.1.x
    parent: 8520:46d2a32bd152
    user: Gary Lee gary.lee@dektech.com.au
    date: Fri Jan 20 15:37:16 2017 +1100
    summary: amfd: use more appropriate log severity levels in si_swap_validate() [#1897]

    changeset: 8524:2605957a234b
    user: Gary Lee gary.lee@dektech.com.au
    date: Fri Jan 20 15:17:07 2017 +1100
    summary: amfd: use more appropriate log severity levels in si_swap_validate() [#1897]

     

    Related

    Tickets: #1897

  • Gary Lee

    Gary Lee - 2017-01-20
    • status: accepted --> fixed
     
  • Rafael Odzakow

    Rafael Odzakow - 2017-01-20

    Consider to change LOG_NO to TRACE in case multiple try again would spam the system log.

     

Log in to post a comment.