Fixes for vagrant setup

[rafael]

Desc

• These fixes vagrant setup dependencies after changes in Make zookeeper version adjustable (ZK_VERSION env) #4734 and Bump default Zookeper version to 3.4.14 #4778.

[dkhenry]

👍

[guidoiaquinti]

in vagrant we install openjdk-7-jdk while in Docker we use openjdk-8-jdk.

This is another example for #4780

[rafael]

Yeah. This is a limitation of the ubuntu version we are using in the Vagrant setup. We need to do some upgrades. openjdk-8 is not available. I think we should do that as part of a different PR.

[guidoiaquinti]

Sure, I just mentioned to keep track of it

…

On Wed, 3 Apr 2019 at 17:29, Rafael Chacon ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In vagrant-scripts/vitess/build.sh <#4782 (comment)>: > @@ -14,6 +14,10 @@ printf "\nBuilding Vitess...\n" sudo chown "$(whoami)":"$(whoami)" /vagrant sudo chown "$(whoami)":"$(whoami)" /vagrant/src cd "$VITESS_WORKSPACE" + +# open-jdk version that we are using in the VM needs this flag, otherwise we will fail to build ZK +export JAVA_TOOL_OPTIONS="-Dhttps.protocols=TLSv1.2" Yeah. This is a limitation of the ubuntu version we are using in the Vagrant setup. We need to do some upgrades. openjdk-8 is not available. I think we should do that as part of a different PR. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#4782 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AD2dmWy7BiI1OpW955Lk_bjHPzl3Mz4Oks5vdMjegaJpZM4cZNPg> .