Skip to content
This repository has been archived by the owner on Jan 4, 2022. It is now read-only.

Re-add EnlargeStoragePool invocation #283

Merged
merged 1 commit into from
Jul 24, 2018

Conversation

schu
Copy link
Contributor

@schu schu commented Jul 9, 2018

While working on 924bb3b I did not
re-add the EnlargeStoragePool invocation and first time users
encountered disk space issues more often again ever since.

Also, add a note to the README and troubleshooting document to tell
users what to do.

Partly covers #281

@schu
Copy link
Contributor Author

schu commented Jul 9, 2018

I should add a comment regarding: #281 (comment)

@schu
Copy link
Contributor Author

schu commented Jul 9, 2018

Doesn't work on my system currently:

$ sudo ./kube-spawn up --kubernetes-version v1.10.5
Using config file "/etc/kube-spawn/config.yaml"
Copying files for cluster ...
Generating configuration files from templates ...
Cluster c1 created
making iptables FORWARD chain defaults to ACCEPT...
Failed to start cluster: not enough space on disk for 3 nodes, Each node needs about 1679 MB, so in total you'll need about 5037 MB available.

I do have enough free space I think:

/dev/loop0                15G  2.5G   13G  17% /var/lib/machines
/var/lib/machines.raw on /var/lib/machines type btrfs (rw,relatime,space_cache,subvolid=5,subvol=/)

@dongsupark any idea what could be wrong here?

@schu schu force-pushed the schu/readd-enlarge-storage-pool branch from 9e1d745 to 41dbaa4 Compare July 9, 2018 13:55
While working on 924bb3b I did not
re-add the `EnlargeStoragePool` invocation and first time users
encountered disk space issues more often again ever since.

Also, add a note to the README and troubleshooting document to tell
users what to do.

Resolves #281
@schu schu force-pushed the schu/readd-enlarge-storage-pool branch from 41dbaa4 to 3d0ce37 Compare July 9, 2018 13:57
@dongsupark
Copy link
Member

@schu I guess, there's still a corner case not being caught in GetPoolSize().
I cannot reproduce it right away. Maybe I would need some time to debug it. Will try later.
Anyway that does not seem to be directly related to this PR. This PR looks good to me.

@dongsupark dongsupark changed the title [WIP] Re-add EnlargeStoragePool invocation Re-add EnlargeStoragePool invocation Jul 24, 2018
@dongsupark
Copy link
Member

Will merge this PR, and add another PR to write more documents.

@dongsupark dongsupark merged commit 6f9e973 into master Jul 24, 2018
@dongsupark dongsupark deleted the schu/readd-enlarge-storage-pool branch July 24, 2018 16:02
dongsupark pushed a commit that referenced this pull request Jul 24, 2018
The PR #283 added some
missing code for fixing free space issues. Though in some corner
cases, we would still need to repair the volume manually.

Partly addresses #281
dongsupark pushed a commit that referenced this pull request Aug 30, 2018
The PR #283 added some
missing code for fixing free space issues. Though in some corner
cases, we would still need to repair the volume manually.

Partly addresses #281
dongsupark pushed a commit that referenced this pull request Aug 30, 2018
The PR #283 added some
missing code for fixing free space issues. Though in some corner
cases, we would still need to repair the volume manually.

Partly addresses #281
dongsupark pushed a commit that referenced this pull request Aug 30, 2018
The PR #283 added some
missing code for fixing free space issues. Though in some corner
cases, we would still need to repair the volume manually.

Partly addresses #281
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants