From 9516282c82d34cad3bfc391d5204922ee4620208 Mon Sep 17 00:00:00 2001
From: Harry Huang <huangxiangyu5@huawei.com>
Date: Thu, 1 Mar 2018 16:42:06 +0800
Subject: Fix workaround of image upload through horizon

1. using horizon group limit
2. set fact openstack_release to hostvars of
localhost

Change-Id: I0af0da1a508e862d0e2bb975f29897ec2948617f
Signed-off-by: Harry Huang <huangxiangyu5@huawei.com>
---
 .../adapters/ansible/openstack/HA-ansible-multinodes.yml  |  1 +
 deploy/adapters/ansible/roles/config-osa/tasks/main.yml   |  2 --
 .../roles/config-osa/tasks/set_openstack_release.yml      | 15 ---------------
 .../adapters/ansible/roles/post-openstack/tasks/main.yml  |  4 ++--
 .../adapters/ansible/roles/post-openstack/vars/main.yml   |  4 ++++
 deploy/adapters/ansible/roles/post-osa/vars/main.yml      |  4 +++-
 6 files changed, 10 insertions(+), 20 deletions(-)
 delete mode 100644 deploy/adapters/ansible/roles/config-osa/tasks/set_openstack_release.yml

(limited to 'deploy')

diff --git a/deploy/adapters/ansible/openstack/HA-ansible-multinodes.yml b/deploy/adapters/ansible/openstack/HA-ansible-multinodes.yml
index 6ea57c04..c14f958e 100644
--- a/deploy/adapters/ansible/openstack/HA-ansible-multinodes.yml
+++ b/deploy/adapters/ansible/openstack/HA-ansible-multinodes.yml
@@ -106,6 +106,7 @@
 - hosts:
     - utility_all[0]
     - network_hosts[0]
+    - horizon
   remote_user: root
   roles:
     - post-openstack
diff --git a/deploy/adapters/ansible/roles/config-osa/tasks/main.yml b/deploy/adapters/ansible/roles/config-osa/tasks/main.yml
index 5f4f2fb2..74d930e2 100755
--- a/deploy/adapters/ansible/roles/config-osa/tasks/main.yml
+++ b/deploy/adapters/ansible/roles/config-osa/tasks/main.yml
@@ -366,5 +366,3 @@
   copy:
     src: redhat-7.yml
     dest: /etc/ansible/roles/os_tacker/vars/redhat-7.yml
-
-- include: set_openstack_release.yml
diff --git a/deploy/adapters/ansible/roles/config-osa/tasks/set_openstack_release.yml b/deploy/adapters/ansible/roles/config-osa/tasks/set_openstack_release.yml
deleted file mode 100644
index c886eabf..00000000
--- a/deploy/adapters/ansible/roles/config-osa/tasks/set_openstack_release.yml
+++ /dev/null
@@ -1,15 +0,0 @@
-############################################################################
-# Copyright (c) 2018 HUAWEI TECHNOLOGIES CO.,LTD and others.
-#
-# All rights reserved. This program and the accompanying materials
-# are made available under the terms of the Apache License, Version 2.0
-# which accompanies this distribution, and is available at
-# http://www.apache.org/licenses/LICENSE-2.0
-##############################################################################
----
-- name: set openstack release
-  lineinfile:
-    dest: "{{ run_dir }}/group_vars/all"
-    line: "openstack_release: {{ openstack_release }}"
-
-- meta: refresh_inventory
diff --git a/deploy/adapters/ansible/roles/post-openstack/tasks/main.yml b/deploy/adapters/ansible/roles/post-openstack/tasks/main.yml
index d5582391..0bd9aeff 100644
--- a/deploy/adapters/ansible/roles/post-openstack/tasks/main.yml
+++ b/deploy/adapters/ansible/roles/post-openstack/tasks/main.yml
@@ -77,7 +77,7 @@
     regexp: "^HORIZON_IMAGES_UPLOAD_MODE"
     line: "HORIZON_IMAGES_UPLOAD_MODE = 'legacy'"
   when:
-    - inventory_hostname in groups['dashboard_containers']
+    - inventory_hostname in groups['horizon']
 # yamllint enable rule:line-length
 
 - name: restart apache2
@@ -85,4 +85,4 @@
     name: apache2
     state: restarted
   when:
-    - inventory_hostname in groups['dashboard_containers']
+    - inventory_hostname in groups['horizon']
diff --git a/deploy/adapters/ansible/roles/post-openstack/vars/main.yml b/deploy/adapters/ansible/roles/post-openstack/vars/main.yml
index d9c36d42..ed64c8db 100644
--- a/deploy/adapters/ansible/roles/post-openstack/vars/main.yml
+++ b/deploy/adapters/ansible/roles/post-openstack/vars/main.yml
@@ -43,3 +43,7 @@ flavors:
     vcpus: 8
     ram: 16384
     disk: 160
+
+# yamllint disable rule:line-length
+openstack_release: "{{ lookup('yamlfile', '/opt/openstack-ansible/group_vars/all/all.yml key=openstack_release') }}"
+# yamllint enable rule:line-length
diff --git a/deploy/adapters/ansible/roles/post-osa/vars/main.yml b/deploy/adapters/ansible/roles/post-osa/vars/main.yml
index 95f51530..f5ffa335 100644
--- a/deploy/adapters/ansible/roles/post-osa/vars/main.yml
+++ b/deploy/adapters/ansible/roles/post-osa/vars/main.yml
@@ -1,6 +1,8 @@
 ---
 
-os_ver: 16.0.5
+# yamllint disable rule:line-length
+os_ver: "{{ lookup('yamlfile', '/opt/openstack-ansible/group_vars/all/all.yml key=openstack_release') }}"
+# yamllint enable rule:line-length
 os_name: pike
 
 # yamllint disable rule:line-length
-- 
cgit