Introduced default values and remove unnecessary ones from ansible code image build...

Change-Id: Id711d6fdacaddeafb8550290503d6cb6c814ea70
Signed-off-by: Maciej Skrocki <maciej.skrocki@intel.com>

diff --git a/ansible/group_vars/all.yml b/ansible/group_vars/all.yml
new file mode 100644 (file)
index 0000000..2690228
--- /dev/null
+++ b/ansible/group_vars/all.yml
@@ -0,0 +1,5 @@
+---\r
+clone_dest: /opt/tempT\r
+release: xenial\r
+target_os: Ubuntu\r
+ubuntu_image_file: /tmp/workspace/yardstick/yardstick-trusty-server.raw\r

diff --git a/ansible/image_uploaders/upload_ubuntu_image.yml b/ansible/image_uploaders/upload_ubuntu_image.yml
index dcfe648..1cf7d0a 100644 (file)
--- a/ansible/image_uploaders/upload_ubuntu_image.yml
+++ b/ansible/image_uploaders/upload_ubuntu_image.yml
@@ -21,7 +21,7 @@
       when: fdio
 
     - os_image:
-        name: "{{ ubuntu_image }}"
+        name: "{{ ubuntu_image_file | basename }}"
         is_public: yes
         disk_format: qcow2
         container_format: bare

diff --git a/ansible/roles/add_custom_repos/templates/sources.list.j2 b/ansible/roles/add_custom_repos/templates/sources.list.j2
index 86be348..af741cb 100644 (file)
--- a/ansible/roles/add_custom_repos/templates/sources.list.j2
+++ b/ansible/roles/add_custom_repos/templates/sources.list.j2
@@ -1,5 +1,5 @@
 {% if YARD_IMG_ARCH == "arm64" %}
-deb [arch={{ YARD_IMG_ARCH }}] http://ports.ubuntu.com/ {{ ubuntu_release[target_os_version] }}-backports main restricted universe multiverse
+deb [arch={{ YARD_IMG_ARCH }}] http://ports.ubuntu.com/ {{ release }}-backports main restricted universe multiverse
 {% else %}
-deb http://archive.ubuntu.com/ubuntu/ {{ ubuntu_release[target_os_version] }}-backports main restricted universe multiverse
+deb http://archive.ubuntu.com/ubuntu/ {{ release }}-backports main restricted universe multiverse
 {% endif %}

diff --git a/ansible/roles/add_custom_repos/vars/main.yml b/ansible/roles/add_custom_repos/vars/main.yml
deleted file mode 100644 (file)
index 03ed6e4..0000000
--- a/ansible/roles/add_custom_repos/vars/main.yml
+++ /dev/null
@@ -1,3 +0,0 @@
-ubuntu_release:
-  "16.04": xenial
-  "14.04": trusty
\ No newline at end of file

diff --git a/ansible/roles/install_dependencies/tasks/RedHat.yml b/ansible/roles/install_dependencies/tasks/RedHat.yml
index b725933..a5d4d0b 100644 (file)
--- a/ansible/roles/install_dependencies/tasks/RedHat.yml
+++ b/ansible/roles/install_dependencies/tasks/RedHat.yml
@@ -17,8 +17,6 @@
   when: ansible_distribution in ['RedHat', 'CentOS', 'ScientificLinux']
 
 - name: Install core packages
-  # we need to recompile the kernel, but we don't need libvirtd it will
-  # run in a container
   action: "{{ ansible_pkg_mgr }} name={{ item }} state=present"
   with_items:
     - deltarpm
@@ -30,7 +28,6 @@
     - screen
     - git
     - socat
-    #
     - sshpass
     - qemu-kvm
     - qemu-img
@@ -38,7 +35,6 @@
     - libxslt-devel
     - libffi-devel
     - openssl-devel
-    - zeromq2-devel
     - nginx
     - uwsgi
     - uwsgi-plugin-python
@@ -47,25 +43,4 @@
     - libffi-devel
     - python-devel
     - kpartx
-    # don't install kernel-devel here it will trigger unwanted kernel upgrade
-    # Mandatory Packages:
-    # Don't use yum groups, they don't work, expand them manually
-    # CentOS 'Development Tools' group
-#    - autoconf
-#    - automake
-#    - binutils
-#    - bison
-#    - flex
-#    - gcc
-#    - gcc-c++
-#    - gettext
-#    - libtool
-#    - make
-#    - patch
-#    - pkgconfig
-#    - redhat-rpm-config
-#    - rpm-build
-#    - rpm-sign
-    # Fedora 'C Development Tools' group
-    #  don't use ccache, it can get weird for kernel compiles
-#      - ccache
+

diff --git a/ansible/roles/install_image_dependencies/tasks/main.yml b/ansible/roles/install_image_dependencies/tasks/main.yml
index 33d99af..f3a3151 100644 (file)
--- a/ansible/roles/install_image_dependencies/tasks/main.yml
+++ b/ansible/roles/install_image_dependencies/tasks/main.yml
@@ -13,11 +13,11 @@
 # limitations under the License.
 ---
   - name: OS pre-workarounds
-    include: "{{ target_os_family }}.yml"
+    include: "{{ ansible_os_family }}.yml"
 
   - name: Install core packages
     action: "{{ ansible_pkg_mgr }} name={{ item }} state=latest"
     register: pkg_mgr_results
     retries: "{{ pkg_mgr_retries }}"
     until: pkg_mgr_results|success
-    with_items: "{{ install_dependencies[target_os_family] }}"
+    with_items: "{{ install_dependencies[ansible_os_family] }}"

diff --git a/tests/ci/ansible_load_images.sh b/tests/ci/ansible_load_images.sh
index 4f62024..9a2d8ef 100755 (executable)
--- a/tests/ci/ansible_load_images.sh
+++ b/tests/ci/ansible_load_images.sh
@@ -24,9 +24,4 @@ ansible-playbook \
          -e img_modify_playbook='ubuntu_server_cloudimg_modify.yml' \
          -e target_os='Ubuntu' \
          -e YARD_IMG_ARCH='amd64' \
-         -e target_os_version='16.04' \
-         -e target_os_family='Debian' \
-         -e clone_dest='/usr/local/src' \
-         -e ubuntu_image='yardstick-trusty-server.raw' \
-         -e ubuntu_image_file='/tmp/workspace/yardstick/yardstick-trusty-server.raw' \
          -vvv -i inventory.ini load_images.yml