ntp fix.

Fixed the review comments.

Signed-off-by: Baha Mesleh <baha.mesleh@nokia.com>
Change-Id: I2212a5ef7cc41922500164cea8f3d99e01f731a0

diff --git a/roles/ntp-utils/library/change_master_servers.py b/roles/ntp-utils/library/change_master_servers.py
index 7b694e6..17dd6f0 100644 (file)
--- a/roles/ntp-utils/library/change_master_servers.py
+++ b/roles/ntp-utils/library/change_master_servers.py
@@ -1,9 +1,7 @@
 #!/bin/env python
 # pylint: skip-file
-from ansible.module_utils.basic import *
-import os, json
-import re, sys
-import syslog
+from ansible.module_utils.basic import AnsibleModule
+import os
 import re
 
 if __name__ == '__main__':

diff --git a/roles/ntp-utils/library/tune_master_server_params.py b/roles/ntp-utils/library/tune_master_server_params.py
index ca65bce..5651c58 100644 (file)
--- a/roles/ntp-utils/library/tune_master_server_params.py
+++ b/roles/ntp-utils/library/tune_master_server_params.py
@@ -1,9 +1,7 @@
 #!/bin/env python
 # pylint: skip-file
-from ansible.module_utils.basic import *
-import os, json
-import re, sys
-import syslog
+from ansible.module_utils.basic import AnsibleModule
+import os
 import re
 
 if __name__ == '__main__':

diff --git a/roles/ntp-utils/tasks/main.yml b/roles/ntp-utils/tasks/main.yml
index a971b5b..35f1a81 100644 (file)
--- a/roles/ntp-utils/tasks/main.yml
+++ b/roles/ntp-utils/tasks/main.yml
@@ -17,7 +17,7 @@
             /usr/bin/logger "ntp-check: Succeeded after $i iterations"
             exit 0
         fi
-        /usr/bin/logger "Failed, iteration $i"
+        /usr/bin/logger "ntp-check: Failed, iteration $i"
         /usr/bin/sleep 2
       done
 
@@ -34,4 +34,3 @@
   systemd:
       state: started
       name: ntpd
-