Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix a bunch of places where the -machines flag was still attached to the apiserver #2118

Merged
merged 1 commit into from
Nov 2, 2014
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 1 addition & 5 deletions build/run-images/apiserver/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,4 @@
# See the License for the specific language governing permissions and
# limitations under the License.

# If the user doesn't specify a minion, assume we are running in a single node
# configuration and that we have a local minion.
KUBE_MINIONS="${KUBE_MINIONS:-$(hostname -f)}"

./apiserver -address=0.0.0.0 -etcd_servers="${ETCD_SERVERS}" --machines="${KUBE_MINIONS}"
./apiserver -address=0.0.0.0 -etcd_servers="${ETCD_SERVERS}"
4 changes: 2 additions & 2 deletions build/run-images/bootstrap/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -39,11 +39,11 @@ containers:
hostPort: 8080
containerPort: 8080
protocol: TCP
command: ["/kubernetes/apiserver", "-v=5", "-address=0.0.0.0", "-etcd_servers=http://127.0.0.1:4001", "-machines=${KUBELET_IP}"]
command: ["/kubernetes/apiserver", "-v=5", "-address=0.0.0.0", "-etcd_servers=http://127.0.0.1:4001"]
- name: controller-manager
image: kubernetes
imagePullPolicy: never
command: ["/kubernetes/controller-manager", "-v=5", "-master=127.0.0.1:8080"]
command: ["/kubernetes/controller-manager", "-v=5", "-master=127.0.0.1:8080", "-machines=${KUBELET_IP}"]
- name: proxy
image: kubernetes
imagePullPolicy: never
Expand Down
6 changes: 5 additions & 1 deletion build/run-images/controller-manager/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,4 +14,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.

./controller-manager -master="${API_SERVER}"
# If the user doesn't specify a minion, assume we are running in a single node
# configuration and that we have a local minion.
KUBE_MINIONS="${KUBE_MINIONS:-$(hostname -f)}"

./controller-manager -master="${API_SERVER}" --machines="${KUBE_MINIONS}"
4 changes: 2 additions & 2 deletions cluster/rackspace/cloud-config/master-cloud-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ coreos:
[Service]
EnvironmentFile=-/run/apiserver/minions.env
ExecStartPre=/usr/bin/ln -sf /opt/kubernetes/server/bin/apiserver /opt/bin/apiserver
ExecStart=/opt/bin/apiserver --address=127.0.0.1 --port=8080 --machines=${MINIONS} --etcd_servers=http://127.0.0.1:4001 --portal_net=PORTAL_NET --logtostderr=true
ExecStart=/opt/bin/apiserver --address=127.0.0.1 --port=8080 --etcd_servers=http://127.0.0.1:4001 --portal_net=PORTAL_NET --logtostderr=true
Restart=always
RestartSec=2
- name: master-apiserver-sighup.path
Expand Down Expand Up @@ -116,7 +116,7 @@ coreos:
Requires=master-apiserver.service
[Service]
ExecStartPre=/usr/bin/ln -sf /opt/kubernetes/server/bin/controller-manager /opt/bin/controller-manager
ExecStart=/opt/bin/controller-manager --master=127.0.0.1:8080 --logtostderr=true
ExecStart=/opt/bin/controller-manager --master=127.0.0.1:8080 --machines=${MINIONS} --logtostderr=true
Restart=always
RestartSec=2
- name: master-scheduler.service
Expand Down
3 changes: 0 additions & 3 deletions contrib/init/systemd/environ/apiserver
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,6 @@ KUBE_API_PORT="--port=8080"
# How the replication controller and scheduler find the apiserver
KUBE_MASTER="--master=127.0.0.1:8080"

# Comma seperated list of minions
KUBELET_ADDRESSES="--machines=127.0.0.1"

# Port minions listen on
KUBELET_PORT="--kubelet_port=10250"

Expand Down
3 changes: 3 additions & 0 deletions contrib/init/systemd/environ/controller-manager
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,8 @@

# defaults from config and apiserver should be adequate

# Comma seperated list of minions
KUBELET_ADDRESSES="--machines=127.0.0.1"

# Add you own!
KUBE_CONTROLLER_MANAGER_ARGS=""
1 change: 0 additions & 1 deletion contrib/init/systemd/kube-apiserver.service
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ ExecStart=/usr/bin/kube-apiserver \
${KUBE_ETCD_SERVERS} \
${KUBE_API_ADDRESS} \
${KUBE_API_PORT} \
${KUBELET_ADDRESSES} \
${KUBELET_PORT} \
${KUBE_ALLOW_PRIV} \
${KUBE_SERVICE_ADDRESSES} \
Expand Down
1 change: 1 addition & 0 deletions contrib/init/systemd/kube-controller-manager.service
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ User=kube
ExecStart=/usr/bin/kube-controller-manager \
${KUBE_LOGTOSTDERR} \
${KUBE_LOG_LEVEL} \
${KUBELET_ADDRESSES} \
${KUBE_MASTER} \
${KUBE_CONTROLLER_MANAGER_ARGS}
Restart=on-failure
Expand Down
2 changes: 1 addition & 1 deletion docs/getting-started-guides/coreos/configs/master.yml
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,6 @@ coreos:
--address=127.0.0.1 \
--port=8080 \
--etcd_servers=http://127.0.0.1:4001 \
--machines=192.168.12.10,192.168.12.11,192.168.12.12 \
--logtostderr=true
Restart=always
RestartSec=10
Expand Down Expand Up @@ -161,6 +160,7 @@ coreos:
[Service]
ExecStart=/opt/bin/controller-manager \
--master=127.0.0.1:8080 \
--machines=192.168.12.10,192.168.12.11,192.168.12.12 \
--logtostderr=true
Restart=always
RestartSec=10
Expand Down
2 changes: 1 addition & 1 deletion docs/getting-started-guides/coreos/configs/standalone.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ coreos:
--address=127.0.0.1 \
--port=8080 \
--etcd_servers=http://127.0.0.1:4001 \
--machines=127.0.0.1 \
--logtostderr=true
Restart=on-failure
RestartSec=1
Expand Down Expand Up @@ -86,6 +85,7 @@ coreos:
[Service]
ExecStart=/opt/bin/controller-manager \
--master=127.0.0.1:8080 \
--machines=127.0.0.1 \
--logtostderr=true
Restart=on-failure
RestartSec=1
Expand Down
1 change: 0 additions & 1 deletion docs/getting-started-guides/coreos/units/apiserver.service
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ ExecStart=/opt/bin/apiserver \
--address=127.0.0.1 \
--port=8080 \
--etcd_servers=http://127.0.0.1:4001 \
--machines=127.0.0.1 \
--portal_net=10.0.0.0/24 \
--logtostderr=true
Restart=on-failure
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ Documentation=https://github.com/GoogleCloudPlatform/kubernetes
[Service]
ExecStart=/opt/bin/controller-manager \
--master=127.0.0.1:8080 \
--machines=127.0.0.1 \
--etcd_servers=http://127.0.0.1:4001 \
--logtostderr=true
Restart=on-failure
Expand Down
15 changes: 12 additions & 3 deletions docs/getting-started-guides/fedora/fedora_manual_config.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,9 +102,6 @@ KUBE_API_PORT="--port=8080"
# How the replication controller and scheduler find the apiserver
KUBE_MASTER="--master=fed-master:8080"

# Comma seperated list of minions
KUBELET_ADDRESSES="--machines=fed-minion"

# Port minions listen on
KUBELET_PORT="--kubelet_port=10250"

Expand All @@ -115,6 +112,18 @@ KUBE_SERVICE_ADDRESSES="--portal_net=10.254.0.0/16"
KUBE_API_ARGS=""
```

* Edit /etc/kubernetes/controller-manager to appear as such:
```
###
# kubernetes system config
#
# The following values are used to configure the kubernetes-controller-manager
#

# Comma seperated list of minions
KUBELET_ADDRESSES="--machines=fed-minion"
```

* Start the appropriate services on master:

```
Expand Down
3 changes: 0 additions & 3 deletions docs/man/kube-apiserver.1.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,6 @@ The the kube-apiserver several options.
**-logtostderr**=
log to standard error instead of files. Default is false.

**-machines**=[]
List of machines to schedule onto, comma separated.

**-minion_cache_ttl**=30s
Duration of time to cache minion information. Default 30 seconds.

Expand Down
3 changes: 3 additions & 0 deletions docs/man/kube-controller-manager.1.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,9 @@ The kube-controller-manager has several options.
**-logtostderr**=false
log to standard error instead of files.

**-machines**=[]
List of machines to schedule onto, comma separated.

**-master**=""
The address of the Kubernetes API server.

Expand Down
4 changes: 0 additions & 4 deletions docs/man/man1/kube-apiserver.1
Original file line number Diff line number Diff line change
Expand Up @@ -69,10 +69,6 @@ The the kube\-apiserver several options.
\fB\-logtostderr\fP=
log to standard error instead of files. Default is false.

.PP
\fB\-machines\fP=[]
List of machines to schedule onto, comma separated.

.PP
\fB\-minion\_cache\_ttl\fP=30s
Duration of time to cache minion information. Default 30 seconds.
Expand Down
4 changes: 4 additions & 0 deletions docs/man/man1/kube-controller-manager.1
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,10 @@ The kube\-controller\-manager has several options.
\fB\-logtostderr\fP=false
log to standard error instead of files.

.PP
\fB\-machines\fP=[]
List of machines to schedule onto, comma separated.

.PP
\fB\-master\fP=""
The address of the Kubernetes API server.
Expand Down