From 18e6a737505ad7a6aee4d16b7efd38dca680db41 Mon Sep 17 00:00:00 2001
From: Vivek Reddy <vkarri@nvidia.com>
Date: Thu, 5 Sep 2024 14:08:54 -0700
Subject: [PATCH] Fix ntp conf file path

---
 show/main.py       |  2 +-
 tests/ntp.conf     | 37 +++++++++++++++++++++++++++++++++++++
 tests/show_test.py | 14 ++++++++++++++
 3 files changed, 52 insertions(+), 1 deletion(-)
 create mode 100644 tests/ntp.conf

diff --git a/show/main.py b/show/main.py
index c9e5e2086c..7e6cf7d203 100755
--- a/show/main.py
+++ b/show/main.py
@@ -1539,7 +1539,7 @@ def ntp(verbose):
     """Show NTP running configuration"""
     ntp_servers = []
     ntp_dict = {}
-    with open("/etc/ntp.conf") as ntp_file:
+    with open("/etc/ntpsec/ntp.conf") as ntp_file:
         data = ntp_file.readlines()
     for line in data:
         if line.startswith("server "):
diff --git a/tests/ntp.conf b/tests/ntp.conf
new file mode 100644
index 0000000000..58bf276dce
--- /dev/null
+++ b/tests/ntp.conf
@@ -0,0 +1,37 @@
+###############################################################################
+# This file was AUTOMATICALLY GENERATED. DO NOT MODIFY.
+# Controlled by ntp-config.service
+###############################################################################
+
+# /etc/ntp.conf, configuration for ntpd; see ntp.conf(5) for help
+
+# To avoid ntpd from panic and exit if the drift between new time and
+# current system time is large.
+tinker panic 0
+
+driftfile /var/lib/ntpsec/ntp.drift
+leapfile /usr/share/zoneinfo/leap-seconds.list
+
+server 10.1.1.1 iburst
+restrict 10.1.1.1 kod limited nomodify noquery
+
+server 10.22.1.12 iburst
+restrict 10.22.1.12 kod limited nomodify noquery
+
+
+interface ignore wildcard
+
+
+interface listen eth0
+interface listen 127.0.0.1
+
+
+# Access control configuration
+# By default, exchange time with everybody, but don't allow configuration.
+# NTPsec doesn't establish peer associations, and so nopeer has no effect, and
+# has been removed from here
+restrict default kod nomodify noquery limited
+
+# Local users may interrogate the ntp server more closely.
+restrict 127.0.0.1
+restrict ::1
diff --git a/tests/show_test.py b/tests/show_test.py
index 4cd29ac45e..d81192367a 100644
--- a/tests/show_test.py
+++ b/tests/show_test.py
@@ -1064,6 +1064,20 @@ def test_rc_syslog(self, mock_rc):
         assert result.exit_code == 0
         assert '[1.1.1.1]' in result.output
 
+    @patch('builtins.open', mock_open(
+        read_data=open('tests/ntp.conf').read()))
+    def test_ntp(self):
+        runner = CliRunner()
+
+        result = runner.invoke(
+            show.cli.commands['runningconfiguration'].commands['ntp'])
+        print(result.exit_code)
+        print(result.output)
+
+        assert result.exit_code == 0
+        assert '10.1.1.1' in result.output
+        assert '10.22.1.12' in result.output
+
     @classmethod
     def teardown_class(cls):
         print('TEARDOWN')