PTP alarms not triggered with Debian

Bug #1978576 reported by Douglas Henrique Koerich
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Medium
Douglas Henrique Koerich

Bug Description

Brief Description
-----------------
PTP "host not locked to GM" alarm is not triggered for Debian system. OOT alarm was also not seen.

Severity
--------
Major

Steps to Reproduce
------------------
Update host to use clock_synchronization=ptp without any PTP instance created and/or assigned to the host.

Expected Behavior
------------------
PTP "Host not locked to GM" alarm should be triggered.

Actual Behavior
----------------
No alarm is generated.

Reproducibility
---------------
Reproducible

System Configuration
--------------------
Any (e.g. AIO-SX)

Branch/Pull Time/Commit
-----------------------
r/stx7.0

Last Pass
---------
First tests with PTP alarming in Debian.

Timestamp/Logs
--------------
N/A

Test Activity
-------------
Developer Testing

Workaround
----------
N/A

Changed in starlingx:
assignee: nobody → Douglas Henrique Koerich (dkoerich-wr)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to monitoring (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to monitoring (master)

Reviewed: https://review.opendev.org/c/starlingx/monitoring/+/846042
Committed: https://opendev.org/starlingx/monitoring/commit/f81d3c8e5e3ef6d5ca66e8dc0ce7e9abb30113f6
Submitter: "Zuul (22348)"
Branch: master

commit f81d3c8e5e3ef6d5ca66e8dc0ce7e9abb30113f6
Author: Douglas Henrique Koerich <email address hidden>
Date: Wed Jun 15 14:10:24 2022 -0300

    Fixing PTP conf files in Debian for alarming

    In Debian the PTP *.conf files moved from /etc/ptpinstance/ to
    /etc/linuxptp/ptpinstance, and therefore collectd plugin would not be
    able to load the configuration and could not generate the proper alarms.
    This change sets the path of configuration files according to the OS
    release ('centos' or 'debian') as read from /etc/os-release.
    As a bonus, it also fixes the calls to subprocess.check_output() to
    decode() the result, which in Python3 is of type "bytes", and that could
    generate a type exception when used with string functions like split().

    Test Plan:
    PASS: Checked alarm generation with PTP configuration and clock skew.

    Closes-Bug: 1978576
    Signed-off-by: Douglas Henrique Koerich <email address hidden>
    Change-Id: I049eb4f1841c2fe0d07bdf3a337d53eea36f783d

Changed in starlingx:
status: In Progress → Fix Released
Ghada Khalil (gkhalil)
Changed in starlingx:
importance: Undecided → Medium
tags: added: stx.7.0 stx.debian stx.networking
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.