From f1821ae63a4e376df854f8fc937d88d6777e8f0b Mon Sep 17 00:00:00 2001
From: SamYaple <sam@yaple.net>
Date: Fri, 1 Jan 2016 03:17:03 +0000
Subject: [PATCH] Remove unused params for kolla_docker

Update the group_vars and globals docuementation as well.

Change-Id: I3ffd49b8d99667425596a2753845767a62e05bf1
Partially-Implements: blueprint kolla-docker-module
---
 ansible/group_vars/all.yml      | 7 +------
 ansible/library/kolla_docker.py | 1 -
 etc/kolla/globals.yml           | 8 --------
 3 files changed, 1 insertion(+), 15 deletions(-)

diff --git a/ansible/group_vars/all.yml b/ansible/group_vars/all.yml
index 71c1e53c67..57e630bb33 100644
--- a/ansible/group_vars/all.yml
+++ b/ansible/group_vars/all.yml
@@ -44,12 +44,8 @@ docker_registry_email:
 docker_registry:
 docker_namespace: "kollaglue"
 docker_registry_username:
-docker_insecure_registry: "False"
 
-# Valid options are [ missing, always ]
-docker_pull_policy: "always"
-
-# Valid options are [ no, on-failure, always ]
+# Valid options are [ never, on-failure, always ]
 docker_restart_policy: "always"
 
 # '0' means unlimited retries
@@ -63,7 +59,6 @@ docker_common_options:
     auth_username: "{{ docker_registry_username }}"
     environment:
       KOLLA_CONFIG_STRATEGY: "{{ config_strategy }}"
-    insecure_registry: "{{ docker_insecure_registry }}"
     restart_policy: "{{ docker_restart_policy }}"
     restart_retries: "{{ docker_restart_policy_retry }}"
 
diff --git a/ansible/library/kolla_docker.py b/ansible/library/kolla_docker.py
index b2006b4359..c6b92216b4 100644
--- a/ansible/library/kolla_docker.py
+++ b/ansible/library/kolla_docker.py
@@ -369,7 +369,6 @@ def generate_module():
         name=dict(required=True, type='str'),
         environment=dict(required=False, type='dict'),
         image=dict(required=False, type='str'),
-        insecure_registry=dict(required=False, type='bool', default=False),
         pid_mode=dict(required=False, type='str', choices=['host']),
         privileged=dict(required=False, type='bool', default=False),
         remove_on_exit=dict(required=False, type='bool', default=True),
diff --git a/etc/kolla/globals.yml b/etc/kolla/globals.yml
index 62feb5ddbd..a5179e7324 100644
--- a/etc/kolla/globals.yml
+++ b/etc/kolla/globals.yml
@@ -37,14 +37,6 @@ kolla_internal_address: "10.10.10.254"
 # docker_namespace: "companyname"
 # docker_registry_username: "sam"
 # docker_registry_password: "correcthorsebatterystaple"
-# docker_insecure_registry: "False"
-
-# The default behaviour when running the playbooks is to pull the latest image
-# available when running the playbooks. If a newer image is found, it will pull
-# that new image and recreate the containers. If this is not the behaviour you
-# want, this should be set to 'missing'
-# Valid options are [ always, missing ]
-#docker_pull_policy: "always"
 
 
 ####################