Merge lp:~barry/update-manager/132311-bootspace into lp:update-manager

Proposed by Barry Warsaw
Status: Merged
Merge reported by: Michael Vogt
Merged at revision: not available
Proposed branch: lp:~barry/update-manager/132311-bootspace
Merge into: lp:update-manager
Diff against target: 27 lines (+8/-1)
2 files modified
DistUpgrade/DistUpgradeCache.py (+1/-1)
debian/changelog (+7/-0)
To merge this branch: bzr merge lp:~barry/update-manager/132311-bootspace
Reviewer Review Type Date Requested Status
Ubuntu Core Development Team Pending
Review via email: mp+22712@code.launchpad.net

Description of the change

Bumps the /boot space assigned to new kernels by about 500K/kernel. Based on my inspection, the old value undercounts by about 260K/kernel. I think this is about the best we can do to address the /boot space problem in bug 132311.

To post a comment you must log in.
Revision history for this message
Michael Vogt (mvo) wrote :

How about we add a full meg? Bump to 19 or even 20 mb just to be sure that we are not too low?

Revision history for this message
Barry Warsaw (barry) wrote :

I thought about using 19 but it seemed excessive. Maybe it wasn't! :) I think 19 would be plenty for now.

Revision history for this message
Michael Vogt (mvo) wrote :

Thanks, I merged it via diff apply now, I can not merge directly because its in a loom format that my bzr does not (yet?) understand :)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'DistUpgrade/DistUpgradeCache.py'
--- DistUpgrade/DistUpgradeCache.py 2010-04-01 20:18:49 +0000
+++ DistUpgrade/DistUpgradeCache.py 2010-04-02 16:53:16 +0000
@@ -49,7 +49,7 @@
49 pass49 pass
5050
51# the initrd/vmlinuz/abi space required in /boot for each kernel51# the initrd/vmlinuz/abi space required in /boot for each kernel
52KERNEL_INITRD_SIZE = 18*1024*102452KERNEL_INITRD_SIZE = 18.5 * 1024 * 1024
5353
54class FreeSpaceRequired(object):54class FreeSpaceRequired(object):
55 """ FreeSpaceRequired object:55 """ FreeSpaceRequired object:
5656
=== modified file 'debian/changelog'
--- debian/changelog 2010-03-31 19:49:55 +0000
+++ debian/changelog 2010-04-02 16:53:16 +0000
@@ -1,3 +1,10 @@
1update-manager (1:0.133.8ubuntu1) lucid; urgency=low
2
3 * Bump up the amount of /boot space calculated per kernel. The current
4 value appears to undercount by about 260K/kernel. (LP: #132311)
5
6 -- Barry Warsaw <barry@canonical.com> Fri, 02 Apr 2010 12:45:11 -0400
7
1update-manager (1:0.133.8) UNRELEASED; urgency=low8update-manager (1:0.133.8) UNRELEASED; urgency=low
29
3 * DistUpgrade/DistUpgradeController.py, DistUpgrade/DistUpgradeCache.py:10 * DistUpgrade/DistUpgradeController.py, DistUpgrade/DistUpgradeCache.py:

Subscribers

People subscribed via source and target branches

to status/vote changes: