Bug 2051593 - Summary Interval Hardcoded in PTP Operator if Set in the Global Body Instead of Command Line
Summary: Summary Interval Hardcoded in PTP Operator if Set in the Global Body Instead ...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Networking
Version: 4.10
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: 4.11.0
Assignee: Joseph Richard
QA Contact: obochan
URL:
Whiteboard:
Depends On:
Blocks: 2055358
TreeView+ depends on / blocked
 
Reported: 2022-02-07 15:14 UTC by Ken Young
Modified: 2022-08-10 10:48 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-08-10 10:47:50 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
a yaml to reproduce the problem (3.04 KB, text/plain)
2022-02-07 15:14 UTC, Ken Young
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Github openshift linuxptp-daemon pull 73 0 None open Bug 2051593: Don't set --summary_interval 1 if set in ptp4lconf 2022-02-14 19:05:16 UTC
Red Hat Product Errata RHSA-2022:5069 0 None None None 2022-08-10 10:48:15 UTC

Description Ken Young 2022-02-07 15:14:14 UTC
Created attachment 1859597 [details]
a yaml to reproduce the problem

Description of problem:
When summary interval is not set on the command line, the PTP operator adds "--summary_interval=1" to the command line over-riding the settings in

Version-Release number of selected component (if applicable):
4.10 - needs to be backported 

How reproducible:
100%

Steps to Reproduce:
1.  remove any summary interval specification in the ptp4l command line
2.  Set summary interval in the global section of the ptp configuration yaml (i.e., -4)
3.

Actual results:
The running ptp4l daemon will have summary interval = 1

Expected results:
The running ptp4l daemon will have summary interval = -4

Additional info:

Comment 3 obochan 2022-03-30 08:19:57 UTC
The issue is verified for 4.10 

The summary interval Global config don't have priority over the command line in case it is configured if the summary configuration is missing in the global config, the default config will work which is 1.

Comment 4 yliu1 2022-03-31 19:07:41 UTC
@

Comment 7 errata-xmlrpc 2022-08-10 10:47:50 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Important: OpenShift Container Platform 4.11.0 bug fix and security update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2022:5069


Note You need to log in before you can comment on or make changes to this bug.