From 9d7efd13936af264743cc06825c713dc19dd1848 Mon Sep 17 00:00:00 2001 From: Alexander Fisher Date: Fri, 1 Nov 2019 14:00:56 +0000 Subject: [PATCH] Add debian 10 support Packages for `buster` were released with ProxySQL 2.0.7 Fixes #114 --- .sync.yml | 1 + .travis.yml | 8 ++++++++ metadata.json | 3 ++- spec/acceptance/class_spec.rb | 3 ++- spec/classes/proxysql_spec.rb | 2 +- 5 files changed, 14 insertions(+), 3 deletions(-) diff --git a/.sync.yml b/.sync.yml index fa768be6..0081435d 100644 --- a/.sync.yml +++ b/.sync.yml @@ -9,3 +9,4 @@ - set: centos7-64 - set: debian8-64 - set: debian9-64 + - set: debian10-64 diff --git a/.travis.yml b/.travis.yml index b996a84d..e4c17e9d 100644 --- a/.travis.yml +++ b/.travis.yml @@ -79,6 +79,14 @@ matrix: bundler_args: --without development release env: PUPPET_INSTALL_TYPE=agent BEAKER_IS_PE=no BEAKER_PUPPET_COLLECTION=puppet6 BEAKER_debug=true BEAKER_setfile=debian9-64 BEAKER_HYPERVISOR=docker CHECK=beaker services: docker + - rvm: 2.5.3 + bundler_args: --without development release + env: PUPPET_INSTALL_TYPE=agent BEAKER_IS_PE=no BEAKER_PUPPET_COLLECTION=puppet5 BEAKER_debug=true BEAKER_setfile=debian10-64 BEAKER_HYPERVISOR=docker CHECK=beaker + services: docker + - rvm: 2.5.3 + bundler_args: --without development release + env: PUPPET_INSTALL_TYPE=agent BEAKER_IS_PE=no BEAKER_PUPPET_COLLECTION=puppet6 BEAKER_debug=true BEAKER_setfile=debian10-64 BEAKER_HYPERVISOR=docker CHECK=beaker + services: docker branches: only: - master diff --git a/metadata.json b/metadata.json index aa4b200f..238961c4 100644 --- a/metadata.json +++ b/metadata.json @@ -64,7 +64,8 @@ "operatingsystem": "Debian", "operatingsystemrelease": [ "8", - "9" + "9", + "10" ] }, { diff --git a/spec/acceptance/class_spec.rb b/spec/acceptance/class_spec.rb index 78def8bd..8ad977f3 100644 --- a/spec/acceptance/class_spec.rb +++ b/spec/acceptance/class_spec.rb @@ -1,7 +1,8 @@ require 'spec_helper_acceptance' describe 'proxysql class' do - unless fact('os.release.major') == '18.04' # There are no proxysql 1.4 packages for bionic + unless fact('os.release.major') == '18.04' || + (fact('os.name') == 'Debian' && fact('os.release.major') == '10') # There are no proxysql 1.4 packages for these OSes context 'version 1.4' do it 'works idempotently with no errors' do pp = <<-EOS diff --git a/spec/classes/proxysql_spec.rb b/spec/classes/proxysql_spec.rb index c403d857..6086d76e 100644 --- a/spec/classes/proxysql_spec.rb +++ b/spec/classes/proxysql_spec.rb @@ -80,7 +80,7 @@ unless (facts[:osfamily] == 'RedHat' && facts[:operatingsystemmajrelease] == '7') || (facts[:operatingsystem] == 'Ubuntu' && facts[:operatingsystemmajrelease] == '18.04') || - (facts[:operatingsystem] == 'Debian' && facts[:operatingsystemmajrelease] == '9') + (facts[:operatingsystem] == 'Debian' && facts[:operatingsystemmajrelease] =~ %r{^(9|10)$}) it { is_expected.to contain_service('proxysql').with_hasstatus(true) } it { is_expected.to contain_service('proxysql').with_hasrestart(true) } end