From e45b2f7f0ca039cf496b4d06f04f9a9acc8f9d62 Mon Sep 17 00:00:00 2001 From: Nik Everett Date: Mon, 28 Sep 2015 03:33:05 +0200 Subject: [PATCH] [test] Jayatana is ignored Installs javatana in vivid, emulates its on-login actions when starting elasticsearch and verifies that elasticsearch turns off javatana. Relates to #13813 --- Vagrantfile | 12 ++++++++---- .../packaging/scripts/packaging_test_utils.bash | 10 ++++++++-- 2 files changed, 16 insertions(+), 6 deletions(-) diff --git a/Vagrantfile b/Vagrantfile index ab0e3224015..7c76e23df8e 100644 --- a/Vagrantfile +++ b/Vagrantfile @@ -32,7 +32,10 @@ Vagrant.configure(2) do |config| end config.vm.define "vivid" do |config| config.vm.box = "ubuntu/vivid64" - ubuntu_common config + ubuntu_common config, extra: <<-SHELL + # Install Jayatana so we can work around it being present. + [ -f /usr/share/java/jayatanaag.jar ] || install jayatana + SHELL end # Wheezy's backports don't contain Openjdk 8 and the backflips required to # get the sun jdk on there just aren't worth it. We have jessie for testing @@ -116,11 +119,11 @@ SOURCE_PROMPT end end -def ubuntu_common(config) - deb_common config, 'apt-add-repository -y ppa:openjdk-r/ppa > /dev/null 2>&1', 'openjdk-r-*' +def ubuntu_common(config, extra: '') + deb_common config, 'apt-add-repository -y ppa:openjdk-r/ppa > /dev/null 2>&1', 'openjdk-r-*', extra: extra end -def deb_common(config, add_openjdk_repository_command, openjdk_list) +def deb_common(config, add_openjdk_repository_command, openjdk_list, extra: '') # http://foo-o-rama.com/vagrant--stdin-is-not-a-tty--fix.html config.vm.provision "fix-no-tty", type: "shell" do |s| s.privileged = false @@ -137,6 +140,7 @@ def deb_common(config, add_openjdk_repository_command, openjdk_list) (echo "Importing java-8 ppa" && #{add_openjdk_repository_command} && apt-get update) + #{extra} SHELL ) end diff --git a/qa/vagrant/src/test/resources/packaging/scripts/packaging_test_utils.bash b/qa/vagrant/src/test/resources/packaging/scripts/packaging_test_utils.bash index 7f24b8a1061..c8c7b2df913 100644 --- a/qa/vagrant/src/test/resources/packaging/scripts/packaging_test_utils.bash +++ b/qa/vagrant/src/test/resources/packaging/scripts/packaging_test_utils.bash @@ -243,8 +243,14 @@ start_elasticsearch_service() { # su and the Elasticsearch init script work together to break bats. # sudo isolates bats enough from the init script so everything continues # to tick along - sudo -u elasticsearch /tmp/elasticsearch/bin/elasticsearch -d \ - -p /tmp/elasticsearch/elasticsearch.pid + sudo -u elasticsearch bash <