Merge pull request #15 from mlichvar/default_provider

Add empty timesync_ntp_provider to defaults
linux-system-roles · May 25, 2019 · 00a646c · 00a646c
2 parents f5179b0 + 3a84020
commit 00a646c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/defaults/main.yml b/defaults/main.yml
@@ -4,3 +4,4 @@ timesync_ptp_domains: []
 timesync_dhcp_ntp_servers: false
 timesync_step_threshold: -1.0
 timesync_min_sources: 1
+timesync_ntp_provider: ''
diff --git a/tasks/main.yml b/tasks/main.yml
@@ -17,12 +17,12 @@
 
 - name: Determine current NTP provider
   timesync_provider:
-  when: timesync_mode != 2 and (timesync_ntp_provider is not defined or timesync_ntp_provider == '')
+  when: timesync_mode != 2 and timesync_ntp_provider == ''
 
 - name: Select NTP provider
   set_fact:
     timesync_ntp_provider: "{{ timesync_ntp_provider_os_default if timesync_ntp_provider_current == '' else timesync_ntp_provider_current }}"
-  when: timesync_mode != 2 and (timesync_ntp_provider is not defined or timesync_ntp_provider == '')
+  when: timesync_mode != 2 and timesync_ntp_provider == ''
 
 - name: Install chrony
   package: