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 filesystem sizes #232

Merged
merged 2 commits into from
Nov 3, 2017

Conversation

carbonin
Copy link
Member

This PR increases the / filesystem from 4.5GiB to 10GiB and the /var filesystem from 2GiB to 12GiB

The root filesystem change is needed so that upgrades can be performed by using bin/update as described in #219. In this case the new gems being installed in /opt/rubies/ruby-2.3.1/lib/ruby/gems were filling up the space on the root filesystem. It should be noted that after upgrading the appliance, we can run bundle clean --force to remove old versions of updated gems and reclaim some space.

The /var filesystem change is needed so that we can use embedded ansible. Both the rpm and docker versions require more space. The rpm version has an explicit check which requires at least 10GiB of free space on the /var partition. We currently ignore that check here, but it isn't checking without reason, so we would like to stop doing that.
Additionally the container version of embedded ansible (AWX) which is being developed in ManageIQ/manageiq#16205 will need more space in /var for the docker daemon.

Adding WIP until my test build finishes.

This is needed for upgrades when new gems are installed

Fixes ManageIQ#219
This is needed for various forms of embedded ansible

When using the rpms (Ansible Tower) the installer checks that
there is at least 10GiB free on the /var filesystem.

When using the docker containers (AWX), docker keeps the images
and runs the containers on the /var filesystem.
@miq-bot
Copy link
Member

miq-bot commented Oct 17, 2017

Checked commits carbonin/manageiq-appliance-build@913e2db~...911759d with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@carbonin
Copy link
Member Author

This is what the appliance filesystems look like now.

screenshot from 2017-10-17 18-07-00

Removing WIP.

@carbonin carbonin removed the wip label Oct 17, 2017
@carbonin carbonin changed the title [WIP] Increase filesystem sizes Increase filesystem sizes Oct 17, 2017
@gtanzillo gtanzillo added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 3, 2017
@gtanzillo gtanzillo merged commit 5da3b7b into ManageIQ:master Nov 3, 2017
simaishi pushed a commit that referenced this pull request Nov 29, 2017
Increase filesystem sizes
(cherry picked from commit 5da3b7b)
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 0614eda1576dddad4a1bc1fbabfa8119af42066b
Author: Gregg Tanzillo <[email protected]>
Date:   Fri Nov 3 11:33:33 2017 -0400

    Merge pull request #232 from carbonin/increase_filesystem_sizes
    
    Increase filesystem sizes
    (cherry picked from commit 5da3b7b0e52964f048c5bde74529e61fb16ba79a)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants