From 20c4cf2625cf3d9796af17070bb172680223b552 Mon Sep 17 00:00:00 2001 From: Jesse Pretorius Date: Wed, 10 Oct 2018 10:13:15 +0100 Subject: [PATCH] MNAIO: Add supported distro check and remove trusty support Ubuntu Trusty is too old to support much of what this tooling does, so we remove support for it and add a task to verify that a supported distribution version is being used. Change-Id: I0247be130dd4adf1d1d50518e6ec5c617a3a8f7c --- multi-node-aio/playbooks/setup-host.yml | 10 ++++++++++ multi-node-aio/playbooks/vars/ubuntu.yml | 4 ---- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/multi-node-aio/playbooks/setup-host.yml b/multi-node-aio/playbooks/setup-host.yml index d16ba4c6..ad0cd165 100644 --- a/multi-node-aio/playbooks/setup-host.yml +++ b/multi-node-aio/playbooks/setup-host.yml @@ -20,6 +20,16 @@ tags: - setup-host tasks: + - name: Check for a supported Operating System + assert: + that: + - (ansible_distribution == 'Ubuntu' and ansible_distribution_release == 'xenial') or + (ansible_distribution == 'Ubuntu' and ansible_distribution_release == 'bionic') + msg: >- + The only supported host platforms for this tooling are Ubuntu 16.04 LTS (Xenial) + and Ubuntu 18.04 LTS (Bionic). Patches to add support for other distributions are + most welcome. + - name: Gather variables for each operating system include_vars: "{{ item }}" with_first_found: diff --git a/multi-node-aio/playbooks/vars/ubuntu.yml b/multi-node-aio/playbooks/vars/ubuntu.yml index b107ed26..d4c9ee57 100644 --- a/multi-node-aio/playbooks/vars/ubuntu.yml +++ b/multi-node-aio/playbooks/vars/ubuntu.yml @@ -17,10 +17,6 @@ mnaio_host_required_distro_packages: - ubuntu-cloud-keyring mnaio_host_package_repos: - - repo: "deb http://ubuntu-cloud.archive.canonical.com/ubuntu {{ ansible_lsb.codename }}-updates/mitaka main" - state: present - filename: "uca" - condition: "{{ ansible_lsb.codename == 'trusty' }}" - repo: "deb http://ubuntu-cloud.archive.canonical.com/ubuntu {{ ansible_lsb.codename }}-updates/queens main" state: present filename: "uca"