Skip to content

Commit

Permalink
Enable iburst option for NTP servers loaded from minigraph
Browse files Browse the repository at this point in the history
In #15058, the NTP server configuration was modified to add additional
options, such as conditionally enabling iburst, specifying the
association type, and specifying the NTP version. One side effect of
this was that the iburst option which was previously always enabled now
requires it to be explicitly enabled in the config_db.

To restore the old behavior, when loading from minigraph, add
`"iburst": "true"` for each NTP server loaded from minigraph.

Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
  • Loading branch information
saiarcot895 committed Jun 28, 2024
1 parent 8c45508 commit 04ef501
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/sonic-config-engine/minigraph.py
Original file line number Diff line number Diff line change
Expand Up @@ -2610,7 +2610,7 @@ def parse_xml(filename, platform=None, port_config_file=None, asic_name=None, hw
results['SYSLOG_SERVER'] = dict((item, {}) for item in syslog_servers)
results['DHCP_SERVER'] = dict((item, {}) for item in dhcp_servers)
results['DHCP_RELAY'] = dhcp_relay_table
results['NTP_SERVER'] = dict((item, {}) for item in ntp_servers)
results['NTP_SERVER'] = dict((item, {'iburst': 'true'}) for item in ntp_servers)
# Set default DNS nameserver from dns.j2
results['DNS_NAMESERVER'] = {}
if os.environ.get("CFGGEN_UNIT_TESTING", "0") == "2":
Expand Down

0 comments on commit 04ef501

Please sign in to comment.