Skip to content

Commit 796e833

Browse files
authored
Cleanup vagrant setup (elastic#60697)
Though bats tests were recently removed, there remains a few unnecessary setups needed for those tests in our vagrant files, as well as CI setup. Additionally, we no longer rely on the vagrant images setting a JAVA_HOME, instead relying on DistroTestPlugin to pull the appropriate jdk when testing with no-jdk distributions.
1 parent 42460a1 commit 796e833

File tree

3 files changed

+3
-53
lines changed

3 files changed

+3
-53
lines changed

.ci/os.sh

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -31,12 +31,6 @@ cp -v .ci/init.gradle $HOME/.gradle/init.d
3131

3232
unset JAVA_HOME
3333

34-
if ! [ -e "/usr/bin/bats" ] ; then
35-
git clone https://github.com/sstephenson/bats /tmp/bats
36-
sudo /tmp/bats/install.sh /usr
37-
fi
38-
39-
4034
if [ -f "/etc/os-release" ] ; then
4135
cat /etc/os-release
4236
. /etc/os-release
@@ -54,16 +48,8 @@ else
5448
fi
5549

5650
sudo bash -c 'cat > /etc/sudoers.d/elasticsearch_vars' << SUDOERS_VARS
57-
Defaults env_keep += "ZIP"
58-
Defaults env_keep += "TAR"
59-
Defaults env_keep += "RPM"
60-
Defaults env_keep += "DEB"
61-
Defaults env_keep += "PACKAGING_ARCHIVES"
62-
Defaults env_keep += "PACKAGING_TESTS"
63-
Defaults env_keep += "BATS_UTILS"
64-
Defaults env_keep += "BATS_TESTS"
65-
Defaults env_keep += "SYSTEM_JAVA_HOME"
6651
Defaults env_keep += "JAVA_HOME"
52+
Defaults env_keep += "SYSTEM_JAVA_HOME"
6753
SUDOERS_VARS
6854
sudo chmod 0440 /etc/sudoers.d/elasticsearch_vars
6955

Vagrantfile

Lines changed: 1 addition & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -333,7 +333,7 @@ def sles_common(config, name)
333333
zypper ar http://download.opensuse.org/distribution/12.3/repo/oss/ oss
334334
zypper --non-interactive --gpg-auto-import-keys refresh
335335
zypper --non-interactive install git-core
336-
# choose to "ignore some dependencies" of expect, which has a problem with tcl...
336+
# choose to "ignore some dependencies" of expect, which has a problem with tcl...
337337
zypper --non-interactive install --force-resolution expect
338338
SHELL
339339
suse_common config, name, extra: extra
@@ -465,38 +465,13 @@ def sh_install_deps(config,
465465
466466
#{extra}
467467
468-
installed java || {
469-
echo "==> Java is not installed"
470-
return 1
471-
}
472-
cat \<\<JAVA > /etc/profile.d/java_home.sh
473-
if [ ! -z "\\\$JAVA_HOME" ]; then
474-
export SYSTEM_JAVA_HOME=\\\$JAVA_HOME
475-
unset JAVA_HOME
476-
fi
477-
JAVA
478468
ensure tar
479469
ensure curl
480470
ensure unzip
481471
ensure rsync
482472
ensure expect
483473
484-
installed bats || {
485-
# Bats lives in a git repository....
486-
ensure git
487-
echo "==> Installing bats"
488-
git clone https://github.com/sstephenson/bats /tmp/bats
489-
# Centos doesn't add /usr/local/bin to the path....
490-
/tmp/bats/install.sh /usr
491-
rm -rf /tmp/bats
492-
}
493-
494474
cat \<\<SUDOERS_VARS > /etc/sudoers.d/elasticsearch_vars
495-
Defaults env_keep += "BATS_UTILS"
496-
Defaults env_keep += "BATS_TESTS"
497-
Defaults env_keep += "BATS_PLUGINS"
498-
Defaults env_keep += "BATS_UPGRADE"
499-
Defaults env_keep += "PACKAGE_NAME"
500475
Defaults env_keep += "JAVA_HOME"
501476
Defaults env_keep += "SYSTEM_JAVA_HOME"
502477
SUDOERS_VARS
@@ -505,21 +480,9 @@ SUDOERS_VARS
505480
end
506481

507482
def windows_common(config, name)
508-
config.vm.provision 'markerfile', type: 'shell', inline: <<-SHELL
509-
$ErrorActionPreference = "Stop"
510-
New-Item C:/is_vagrant_vm -ItemType file -Force | Out-Null
511-
SHELL
512-
513483
config.vm.provision 'set prompt', type: 'shell', inline: <<-SHELL
514484
$ErrorActionPreference = "Stop"
515485
$ps_prompt = 'function Prompt { "#{name}:$($ExecutionContext.SessionState.Path.CurrentLocation)>" }'
516486
$ps_prompt | Out-File $PsHome/Microsoft.PowerShell_profile.ps1
517487
SHELL
518-
519-
config.vm.provision 'set env variables', type: 'shell', inline: <<-SHELL
520-
$ErrorActionPreference = "Stop"
521-
[Environment]::SetEnvironmentVariable("PACKAGING_ARCHIVES", "C:/project/build/packaging/archives", "Machine")
522-
[Environment]::SetEnvironmentVariable("PACKAGING_TESTS", "C:/project/build/packaging/tests", "Machine")
523-
[Environment]::SetEnvironmentVariable("JAVA_HOME", $null, "Machine")
524-
SHELL
525488
end

buildSrc/src/main/java/org/elasticsearch/gradle/test/DistroTestPlugin.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -265,6 +265,7 @@ private static List<Object> configureVM(Project project) {
265265
VagrantExtension vagrant = project.getExtensions().getByType(VagrantExtension.class);
266266
vagrant.setBox(box);
267267
vagrant.vmEnv("SYSTEM_JAVA_HOME", convertPath(project, vagrant, systemJdk, "", ""));
268+
vagrant.vmEnv("JAVA_HOME", ""); // make sure any default java on the system is ignored
268269
vagrant.vmEnv("PATH", convertPath(project, vagrant, gradleJdk, "/bin:$PATH", "\\bin;$Env:PATH"));
269270
// pass these along to get correct build scans
270271
if (System.getenv("JENKINS_URL") != null) {

0 commit comments

Comments
 (0)