Skip to content

Commit

Permalink
vagrant: MASTER_IP is only needed in the Vagrantfile itself
Browse files Browse the repository at this point in the history
Remove all traces of MASTER_IP in all deployment scripts.

Change-Id: I53890672f9e38cfc2df53453a86090464cdd11ba
Signed-off-by: Roman Mohr <[email protected]>
  • Loading branch information
rmohr committed Dec 8, 2016
1 parent cec8678 commit 6f42c0b
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 10 deletions.
4 changes: 0 additions & 4 deletions Vagrantfile
Original file line number Diff line number Diff line change
Expand Up @@ -58,9 +58,6 @@ Vagrant.configure(2) do |config|
#!/bin/bash
set -xe
export WITH_LOCAL_NFS=true
export KUBERNETES_MASTER=true
export VM_IP=#{$master_ip}
export MASTER_IP=#{$master_ip}
export NETWORK_PROVIDER=#{$network_provider}
cd /vagrant/cluster
bash setup_kubernetes_master.sh
Expand All @@ -82,7 +79,6 @@ Vagrant.configure(2) do |config|
node.vm.provision "shell", inline: <<-SHELL
#!/bin/bash
set -xe
export MASTER_IP=#{$master_ip}
cd /vagrant/cluster
bash setup_kubernetes_node.sh
set +x
Expand Down
5 changes: 1 addition & 4 deletions cluster/setup_kubernetes_master.sh
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
#/bin/bash -xe

# Example environment variables (set by Vagrantfile)
# export KUBERNETES_MASTER=true
# export VM_IP=192.168.200.2
# export MASTER_IP=$VM_IP
export KUBERNETES_MASTER=true
bash ./setup_kubernetes_common.sh

# Cockpit with kubernetes plugin
Expand Down
2 changes: 0 additions & 2 deletions cluster/setup_kubernetes_node.sh
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
#/bin/bash -xe

# Example environment variables (set by Vagrantfile)
# export MASTER_IP=192.168.200.2
bash ./setup_kubernetes_common.sh

ADVERTISED_MASTER_IP=`dig +short master`
Expand Down

0 comments on commit 6f42c0b

Please sign in to comment.