Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 234486 - pygrub tries to read menu.lst before grub.conf, tries to start non-existent kernel
Summary: pygrub tries to read menu.lst before grub.conf, tries to start non-existent k...
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: xen
Version: 5.0
Hardware: All
OS: Linux
Target Milestone: ---
: ---
Assignee: Daniel Berrange
QA Contact:
Depends On:
Blocks: 449772
TreeView+ depends on / blocked
Reported: 2007-03-29 16:23 UTC by Nils Philippsen
Modified: 2009-01-20 21:13 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2009-01-20 21:13:21 UTC
Target Upstream Version:

Attachments (Terms of Use)
patch to let pygrub check for grub.conf before checking for menu.lst (deleted)
2007-03-29 16:23 UTC, Nils Philippsen
no flags Details | Diff
Actual patch posted for inclusion (deleted)
2008-07-10 11:20 UTC, Daniel Berrange
no flags Details | Diff

System ID Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2009:0118 normal SHIPPED_LIVE xen bug fix and enhancement update 2009-01-20 16:04:49 UTC

Description Nils Philippsen 2007-03-29 16:23:32 UTC
Description of problem:

After updating a virtual machine I couldn't restart it with "xm create", it
searched for a (now non-existent) old version of the kernel and initrd. I found
that something or someone wrote a menu.lst that wasn't a symlink to grub.conf,
i.e. its contents didn't change when the kernel was updated. pygrub read that
menu.lst instead of the (correct) grub.conf and failed to start the guest.

I've attached a patch to pygrub that lets it check grub.conf before menu.lst
which should fix this problem.

Version-Release number of selected component (if applicable):


Comment 1 Nils Philippsen 2007-03-29 16:23:32 UTC
Created attachment 151210 [details]
patch to let pygrub check for grub.conf before checking for menu.lst

Comment 2 Jeremy Katz 2007-03-29 17:45:25 UTC
And then you could have the problem the other way.  This is the unfortunate
problem with the difference in grub config naming between distros.

Comment 3 Nils Philippsen 2007-03-30 08:00:55 UTC
If we use grub.conf in one place and prefer menu.lst in another, we're not
consistent with ourselves. Don't you think pygrub should (if in doubt) use what
we use elsewhere? The way it is, we support other distros better than we support
RHEL as a guest, which is more than unfortunate. AFAIK other distros don't have
both a grub.conf and a menu.lst, so the patch aboive shouldn't cause problems
with them.

The least we should do (if we are going out of our way to support other distros
better than our own) is warn people about the circumstance that there are both a
grub.conf and a menu.lst file and which one we used.

Comment 4 Daniel Berrange 2008-07-09 11:21:52 UTC
The real 'grub' package in RHEL-5 has a patch grub-0.93-configfile.patch which
removes its use of menu.lst in favour of grub.conf. 

pygrub needs to behave in the same way, and not look at menu.lst so we are at
least consistent within RHEL-5.

Comment 5 RHEL Product and Program Management 2008-07-09 11:39:15 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux maintenance release.  Product Management has requested
further review of this request by Red Hat Engineering, for potential
inclusion in a Red Hat Enterprise Linux Update release for currently deployed
products.  This request is not yet committed for inclusion in an Update

Comment 7 Daniel Berrange 2008-07-10 11:20:57 UTC
Created attachment 311468 [details]
Actual patch posted for inclusion

Comment 9 Daniel Berrange 2008-07-21 10:48:09 UTC
Built into xen-3.0.3-67.el5

Comment 12 errata-xmlrpc 2009-01-20 21:13:21 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

Note You need to log in before you can comment on or make changes to this bug.