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

Increase disk size for kubernetes master. #8051

Merged
merged 1 commit into from May 11, 2015

Conversation

Projects
None yet
4 participants
@fgrzadkowski
Copy link
Member

commented May 11, 2015

This improves request latency (#4521).

@wojtek-t

@@ -21,6 +21,8 @@ ZONE=${KUBE_GCE_ZONE:-us-central1-b}
MASTER_SIZE=${MASTER_SIZE:-n1-standard-1}
MINION_SIZE=${MINION_SIZE:-n1-standard-1}
NUM_MINIONS=${NUM_MINIONS:-4}
MASTER_DISK_TYPE=pd-standard
MASTER_DISK_SIZE=${MINION_DISK_SIZE:-100GB}

This comment has been minimized.

Copy link
@wojtek-t

wojtek-t May 11, 2015

Member

s/MINION/MASTER/

This comment has been minimized.

Copy link
@fgrzadkowski

fgrzadkowski May 11, 2015

Author Member

Done.

@@ -21,6 +21,8 @@ ZONE=${KUBE_GCE_ZONE:-us-central1-b}
MASTER_SIZE=${MASTER_SIZE:-g1-small}
MINION_SIZE=${MINION_SIZE:-g1-small}
NUM_MINIONS=${NUM_MINIONS:-2}
MASTER_DISK_TYPE=pd-standard
MASTER_DISK_SIZE=${MINION_DISK_SIZE:-100GB}

This comment has been minimized.

Copy link
@wojtek-t

wojtek-t May 11, 2015

Member

s/MINION/MASTER/

This comment has been minimized.

Copy link
@fgrzadkowski

fgrzadkowski May 11, 2015

Author Member

Done.

Increase disk size for kubernetes master.
Signed-off-by: Filip Grzadkowski <filipg@google.com>

@fgrzadkowski fgrzadkowski force-pushed the fgrzadkowski:big_disk branch from a0f6f18 to 26d1430 May 11, 2015

@wojtek-t wojtek-t added the lgtm label May 11, 2015

@wojtek-t

This comment has been minimized.

Copy link
Member

commented May 11, 2015

LGTM

wojtek-t added a commit that referenced this pull request May 11, 2015

Merge pull request #8051 from fgrzadkowski/big_disk
Increase disk size for kubernetes master.

@wojtek-t wojtek-t merged commit a98770e into kubernetes:master May 11, 2015

3 of 4 checks passed

Shippable Builds failed on Shippable
Details
cla/google All necessary CLAs are signed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage decreased (-0.15%) to 48.65%
Details
@bprashanth

This comment has been minimized.

Copy link
Member

commented May 15, 2015

@fgrzadkowski for my own education, how did this help? IIUC etcd stores everything in memory. I suppose it helped snapshotting, but that should just be transient?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.