SNMP OIDs missing for wrsAlarmMessage

[Agustin Carranza]

Brief Description

Some OID's are missing from `wrsAlarmMessage` type traps. If I'm reading this correctly, wrsAlarmMessage should be sending 12 OID's but only 8 are appearing. I think the UUID one is added in stx7.0 - so it seems like 3 are missing.

Severity

<Minor: System/Feature is usable with minor issue>

Steps to Reproduce

Configure SNMP and have Alarms sent via snmp trap. Check fm-manager.log and observe that wrsAlarmMessage is missing some OID's.

Expected Behavior

12 OID's should be visible.

Actual Behavior

8 OIDs are visible.

Reproducibility

Occurs on customer system.

System Configuration

AIO-DX

Load info (eg: 2022-03-10_20-00-07)

stx6.0

$ cat etc/build.info| grep "BUILD_ID"
BUILD_ID="2022-01-10_02-00-35"
SRC_BUILD_ID="45"

Last Pass

NA

Timestamp/Logs

One example from /var/log/fm-manager.log :

2023-07-24T04:22:49.815 fmSnmpUtils.cpp(325): SNMP trap metadata sent succesfully {
  "operation_type": "wrsAlarmMessage",
  "alarm": {
   "wrsEventEventId": "401.005",
   "wrsEventEntityInstanceId": "*****************",
   "wrsEventDateAndTime": "2023-07-24,09:22:49.0,+0:0",
   "wrsEventSeverity": "4",
   "wrsEventReasonText": "Communication established with peer over port ***** on host controller-0",
   "wrsEventEventType": "3",
   "wrsEventProbableCause": "65",
   "wrsEventServiceAffecting": "1"
  }
 } 530

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/snmp-armada-app/+/892624

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/fault/+/892710

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/starlingx/snmp-armada-app/+/892624
Committed: https://opendev.org/starlingx/snmp-armada-app/commit/b9f45f28c48b9ca111ffa0dda89c09333eee6f8c
Submitter: "Zuul (22348)"
Branch: master

commit b9f45f28c48b9ca111ffa0dda89c09333eee6f8c
Author: Agustin Carranza <email address hidden>
Date: Sat Jul 15 17:32:31 2023 -0300

    Add missing fields for Event traps

    wrsEventMessage traps are being managed as wrsAlarmMessages.
    Events do not contain wrsEventProposedRepairAction and
    wrsEventSuppressionAllowedf fields.
    They are now receiving a default value generated in FM.
    This commits adds those fields in order to format the event traps
    like the alarm traps.

    Test plan:
    PASS: Install snmp application. Generate an event and check that
    the trap is specified with the new values.

    Partial-bug: 2032844

    Signed-off-by: Agustin Carranza <email address hidden>
    Change-Id: I5467b5a7bc6e903c629965f86dfaafd468738c8e

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/root/+/893818

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/snmp-armada-app/+/893975

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/starlingx/fault/+/892710
Committed: https://opendev.org/starlingx/fault/commit/496702c9322f7fe7cad1cb921499c7b72897557c
Submitter: "Zuul (22348)"
Branch: master

commit 496702c9322f7fe7cad1cb921499c7b72897557c
Author: Agustin Carranza <email address hidden>
Date: Thu Aug 24 15:58:53 2023 -0300

    Add missing fields for Event traps

    wrsEventMessage traps are being managed as wrsAlarmMessages.
    Events do not contain wrsEventProposedRepairAction and
    wrsEventSuppressionAllowed fields, so they need to generate a default
    value in FM.
    This commit adds those fields in order to create the event traps
    with the same format as the alarm traps.

    Depends-on: https://review.opendev.org/c/starlingx/snmp-armada-app/+/892624

    Partial-bug: 2032844

    Signed-off-by: Agustin Carranza <email address hidden>
    Change-Id: I58577406cc75c597f6f430015ddd51d0029d4539

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/starlingx/root/+/893818
Committed: https://opendev.org/starlingx/root/commit/d949ab146e7efd49f5c64e7ee37b21fd61191e2e
Submitter: "Zuul (22348)"
Branch: master

commit d949ab146e7efd49f5c64e7ee37b21fd61191e2e
Author: Agustin Carranza <email address hidden>
Date: Tue Sep 5 21:30:33 2023 -0300

    Update static tag for stx-fm-trap-subagent image

    The content of the stx-fm-trap-subagent image has been updated due to
    the fix of a bug:
    https://review.opendev.org/c/starlingx/snmp-armada-app/+/892624

    It is updated the yaml file with new tag using existing
    master-debian-stable-20230905T060000Z.0.

    Partial-bug: 2032844

    Change-Id: I203db537b90ad11fa8c417726ad09ff3eedc25aa
    Signed-off-by: Agustin Carranza <email address hidden>

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/starlingx/snmp-armada-app/+/893975
Committed: https://opendev.org/starlingx/snmp-armada-app/commit/14d8d43585ddec146f4551c3efa848c779dc718d
Submitter: "Zuul (22348)"
Branch: master

commit 14d8d43585ddec146f4551c3efa848c779dc718d
Author: Agustin Carranza <email address hidden>
Date: Wed Sep 6 16:11:44 2023 -0300

    Update stx-fm-trap-subagent tag to stx.9.0-v1.0.2

    This change updates the stx-fm-trap-subagent image tag to stx.9.0-v1.0.2
    due to the fix of the following bug:
    https://review.opendev.org/c/starlingx/snmp-armada-app/+/892624

    Test Plan:
    PASS: Apply snmp app with new tags introduced. Verify pod is up and
    running. Describe pod and verify that the new tag is used.

    Depends-on: https://review.opendev.org/c/starlingx/root/+/893818
    Depends-on: https://review.opendev.org/c/starlingx/fault/+/892710

    Closes-bug: 2032844

    Signed-off-by: Agustin Carranza <email address hidden>
    Change-Id: Idaa83357d632944b7ac530d8eea22869a5aabf0f