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

F23 dnf root size #358

Closed

Conversation

jkonecny12
Copy link
Member

Anaconda must compute size for downloaded and installed packages when testing free space for installed root. Also use only 90% of disk capacity for required space compare.

Resolves: rhbz#1224048

@jkonecny12 jkonecny12 added master Please, use the `f39` label instead. f23-branch labels Sep 10, 2015
@vpodzime
Copy link
Contributor

Looks good to me otherwise.

@jkonecny12
Copy link
Member Author

Updated, thank you @vpodzime

@@ -58,6 +58,7 @@
DNF_CACHE_DIR = '/tmp/dnf.cache'
DNF_PLUGINCONF_DIR = '/tmp/dnf.pluginconf'
DNF_PACKAGE_CACHE_DIR_SUFFIX = 'dnf.package.cache'
INSTALL_ROOT_MPOINT = '/mnt/sysimage'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can't assume this anymore. Use iutil.getSysroot() instead.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch I'm going to update PR.

Anaconda must compute size for downloaded and installed packages when
testing free space for installed root.

Resolves: rhbz#1224048
We need to reduce the disk capacity for space required comparison
because fs takes some for metadata.

Related: rhbz#1224048
@jkonecny12
Copy link
Member Author

PR was updated according to @bcl comment. Thank you

@jkonecny12
Copy link
Member Author

Pushed. Thank you for help

@jkonecny12 jkonecny12 closed this Sep 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master Please, use the `f39` label instead.
3 participants