From d16f8e36d68311b1d9130ea28cec9e338b9f870e Mon Sep 17 00:00:00 2001 From: Wesley Wang <2791412+WesleyW@users.noreply.github.com> Date: Mon, 9 Mar 2020 19:01:48 -0700 Subject: [PATCH] Correctly disable ntp install for onprem Summary: D8085 incorrectly wrapped the when conditional for disabling onprem. Fixedd it so the braces completely encompass the conditional. Test Plan: onprem install - make sure it doens't have ntp Reviewers: ram, bogdan, alan Reviewed By: alan Subscribers: jenkins-bot, yugaware Differential Revision: https://phabricator.dev.yugabyte.com/D8110 --- managed/devops/roles/provision-cluster-server/tasks/main.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/managed/devops/roles/provision-cluster-server/tasks/main.yml b/managed/devops/roles/provision-cluster-server/tasks/main.yml index 0ec3b7fc5cb9..613ba797f400 100644 --- a/managed/devops/roles/provision-cluster-server/tasks/main.yml +++ b/managed/devops/roles/provision-cluster-server/tasks/main.yml @@ -38,7 +38,7 @@ name: ntp state: restarted when: ansible_distribution == "Ubuntu" - when: not should_setup_chrony and not air_gap and "{{ cloud_type != 'onprem' }}" + when: "{{ not should_setup_chrony and not air_gap and cloud_type != 'onprem' }}" - block: - name: Uninstall NTP if available