Browse Source

Fix review comments

Manjunath A Kumatagi 8 years ago
parent
commit
43e9680b4e
1 changed files with 2 additions and 2 deletions
  1. 2 2
      roles/docker/tasks/main.yml

+ 2 - 2
roles/docker/tasks/main.yml

@@ -67,7 +67,7 @@
     dest: /etc/sysconfig/docker
     regexp: '^{{ item.reg_conf_var }}=.*$'
     line: "{{ item.reg_conf_var }}='{{ item.reg_fact_val | oo_prepend_strings_in_list(item.reg_flag ~ ' ') | join(' ') }}'"
-  when: item.reg_fact_val != '' and  (docker_check.stat.isreg is defined and docker_check.stat.isreg)
+  when: item.reg_fact_val != '' and docker_check.stat.isreg is defined and docker_check.stat.isreg
   with_items:
   - reg_conf_var: ADD_REGISTRY
     reg_fact_val: "{{ docker_additional_registries | default(None, true)}}"
@@ -97,7 +97,7 @@
   notify:
     - restart docker
   when:
-    - (docker_check.stat.isreg is defined and docker_check.stat.isreg) and "{{ 'http_proxy' in openshift.common or 'https_proxy' in openshift.common }}"
+    - docker_check.stat.isreg is defined and docker_check.stat.isreg and '"http_proxy" in openshift.common or "https_proxy" in openshift.common'
 
 - name: Set various Docker options
   lineinfile: