Bug 340922

Summary: Check update process for "menu.lst"
Product: [openSUSE] openSUSE 10.3 Reporter: Markus Elfring <Markus.Elfring>
Component: YaST2Assignee: Josef Reidinger <jreidinger>
Status: RESOLVED FIXED QA Contact: Jiri Srain <jsrain>
Severity: Normal    
Priority: P2 - High CC: jplack
Version: Final   
Target Milestone: ---   
Hardware: x86-64   
OS: openSUSE 10.3   
Whiteboard:
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: log output from the previous update on Friday
GRUB configuration after kernel update for the release "2.6.22.13-0.3".
log output from an update on Saturday
reordered GRUB configuration for my personal preferences
GRUB configuration after kernel update for the release "2.6.22.16-0.1"
log output from an update on Thursday
GRUB configuration after kernel update for the release "2.6.25.9-0.2"
log output from an update on Tuesday
reordered GRUB configuration for my personal preferences

Description Markus Elfring 2007-11-11 17:27:03 UTC
The YaST2 online update integrated the recent Linux kernel "2.6.22.12-0.1" in my system on Friday. I checked the file "/boot/grub/menu.lst" after this update.

I noticed that there were two menu entries with the same title "XEN -- openSUSE 10.3 - 2.6.22.12-0.1". So I changed this to distinguish the PAE enabled one.
The update process deleted the command "configfile" that I placed before the comment line "##YaST - activate" in my previous configuration.

http://lists.opensuse.org/opensuse-kernel/2007-11/msg00013.html
Comment 1 Cyril Hrubis 2007-11-22 18:00:03 UTC
Please attach y2logs. If you are in doubt follow:

http://en.opensuse.org/Bugs/YaST

Thanks!
Comment 2 Markus Elfring 2007-11-22 19:08:35 UTC
Created attachment 184459 [details]
log output from the previous update on Friday
Comment 5 Alexander Osthof 2007-12-06 15:00:03 UTC
Could you please attach versions of the menu.lst before and after ther update? Thanks.
Comment 6 Markus Elfring 2007-12-06 16:37:16 UTC
No, I'm sorry - I can not restore these versions.
Comment 7 Alexander Osthof 2007-12-06 17:09:33 UTC
So, if I understand you correctly, you had 2 xen entries before the kernel update. One for the "normal" xen kernel and one for a xen-pae one, right?

What if still don't understand is your following comment:

"The update process deleted the command "configfile" that I placed before the
comment line "##YaST - activate" in my previous configuration."

What exactly should this be good for?
Comment 8 Markus Elfring 2007-12-06 18:48:55 UTC
(In reply to comment #7 from Alexander Osthof)
> So, if I understand you correctly, you had 2 xen entries before the kernel
> update. One for the "normal" xen kernel and one for a xen-pae one, right?

Yes, this is a standard setting.


> What if still don't understand is your following comment:
> "The update process deleted the command "configfile" that I placed before the
> comment line "##YaST - activate" in my previous configuration."

Is the tool prepared to tolerate additional commands besides the ones that are marked with specific comments?


> What exactly should this be good for?

I want to put additional settings there. The YaST update should keep it.
Comment 9 Alexander Osthof 2007-12-07 16:42:54 UTC
(In reply to comment #8 from Markus Elfring)
> 
> Is the tool prepared to tolerate additional commands besides the ones that are
> marked with specific comments?

If the commands are supported, it tolerates and even uses them.

Implemented a distinction for various xen kernel names (flavors) and submitted a new package for 10.3, which will most likely appear with the next update.

Thus, closing this bug RESOLVED FIXED.

If related errors still occur, feel free to reopen.
Comment 10 Markus Elfring 2007-12-08 09:56:24 UTC
(In reply to comment #9 from Alexander Osthof)
> If the commands are supported, it tolerates and even uses them.

The line for inclusion of my additional configuration data was deleted again.


> Implemented a distinction for various xen kernel names (flavors) and submitted
> a new package for 10.3, which will most likely appear with the next update.

I do not notice a correction after the update for the release "2.6.22.13-0.3".
Comment 11 Markus Elfring 2007-12-08 10:01:37 UTC
Created attachment 186533 [details]
GRUB configuration after kernel update for the release "2.6.22.13-0.3".
Comment 12 Markus Elfring 2007-12-08 10:24:20 UTC
Created attachment 186534 [details]
log output from an update on Saturday
Comment 13 Markus Elfring 2007-12-08 18:07:40 UTC
Created attachment 186544 [details]
reordered GRUB configuration for my personal preferences
Comment 14 Anja Stock 2008-01-17 14:53:05 UTC
released -> RESOLVED FIXED
Comment 15 Markus Elfring 2008-01-31 20:46:10 UTC
Which software update will contain these fixes?
The current online update for Linux kernel 2.6.22.16-0.1 shows still open issues.
Comment 16 Markus Elfring 2008-01-31 21:08:18 UTC
Created attachment 192634 [details]
GRUB configuration after kernel update for the release "2.6.22.16-0.1"

1. a duplicate title

2. Is it correct to specify "module /initrd-2.6.22.16-0.1-xen" and "module /initrd-2.6.22.16-0.1-xenpae"?
  I would expect the key word "initrd" there as it is used at other places.

3. My include line was deleted again.

4. Why was the part "memtest86+" thrown away?
Comment 17 Markus Elfring 2008-01-31 21:32:38 UTC
Created attachment 192635 [details]
log output from an update on Thursday
Comment 18 Alexander Osthof 2008-02-01 10:14:32 UTC
Sorry, comment #14 was a bit misleading. This was because the fix for this bug went into the maintenance update for SLES10-SP1, but not yet for 10.3.

Please be patient, there will be an online update of perl-Bootloader for 10.3 soon.
Comment 20 Markus Elfring 2008-02-01 11:13:41 UTC
I hope that the correction will be published together with the fixes for the issues "CPU frequency scaling not working after upgrade to latest kernel (2.6.22.16-0.1-default)" (Bug #357598) and "cpufreq not supported" (Bug #357516).
It is surprising to get broken updates.
Comment 21 Alexander Osthof 2008-02-01 11:44:00 UTC
In reply to comment #19:

The first link is a reference to the original perl-Bootloader package from openSUSE 10.3, thus not fixed yet.

The second link is another implementation of a bootloader configurator, but we don't use it in our distro.

This is the link where the (SLES10-SP1 !) update is located:

http://download.novell.com/Download?buildid=ggD7kc_SyrM~
Comment 22 Markus Elfring 2008-02-11 10:00:14 UTC
How long will the publication of this fix be delayed for openSUSE users compared to enterprise customers?
Which position has it got in your processing queue?
Comment 23 Alexander Osthof 2008-02-11 12:23:21 UTC
My apologies, but I'm quite busy these days and resources are limited, too. But the perl-Bootloader update for openSUSE 10.3 is the next item in my queue, so I'm confident to release it soon.
Comment 24 Anja Stock 2008-03-25 07:45:02 UTC
released
Comment 25 Alexander Osthof 2008-05-14 08:04:56 UTC
I've prepared a new package of perl-Bootloader which should be available soon
as maintenance update for openSUSE 10.3.

Meanwhile, the new perl-Bootloader package (which will be released as
maintenance update) is available via openSUSE Build Service.

http://download.opensuse.org/repositories/home:/aosthof/openSUSE_10.3/

This package should fix this bug, thus closing RESOLVED FIXED.

If related errors still occur, feel free to reopen.
Comment 26 Markus Elfring 2008-07-09 17:16:07 UTC
This story continues for the package "perl-Bootloader 0.4.32.21-0.1" under openSUSE 11.x ...
Comment 27 Markus Elfring 2008-07-09 17:23:00 UTC
Created attachment 226814 [details]
GRUB configuration after kernel update for the release "2.6.25.9-0.2"

My additional "configfile" line was forgotten once more. It would also be nice if more menu entries will be kept.
Comment 28 Markus Elfring 2008-07-09 18:06:32 UTC
Created attachment 226820 [details]
log output from an update on Tuesday
Comment 29 Alexander Osthof 2008-07-14 13:31:17 UTC
Reassigning to new maintainer of perl-Bootloader.
Comment 30 Markus Elfring 2008-07-16 17:47:56 UTC
Created attachment 228220 [details]
reordered GRUB configuration for my personal preferences
Comment 31 Josef Reidinger 2008-07-22 11:35:02 UTC
I look at this bug and problem is, that from global options p-bootloader store only known and known is that which is shown in yast-bootloader. This bug need more work to consider how change editing global options.
Comment 32 Markus Elfring 2008-07-25 10:40:36 UTC
I wonder why it seems to be so hard to support global options (not per title settings) correctly.

Should a configuration line like "module /initrd-2.6.25.9-0.2-xen" also be adjusted?

Why does the software component "perl-Bootloader 0.4.63-2.1" require the release "perl = 5.10.0" by its dependencies?
Comment 33 Josef Reidinger 2008-07-25 10:52:19 UTC
(In reply to comment #32 from Markus Elfring)
> I wonder why it seems to be so hard to support global options (not per title
> settings) correctly.
> 
> Should a configuration line like "module /initrd-2.6.25.9-0.2-xen" also be
> adjusted?
> 
> Why does the software component "perl-Bootloader 0.4.63-2.1" require the
> release "perl = 5.10.0" by its dependencies?
> 

1) because menu.lst is parsed for known commands and configfile isn't as known (which is mistake, but whole system need improve due to additional negatives). and if I change something in global section, rewrite write only parsed options.

2) http://en.opensuse.org/SUSE_Build_Tutorial#Perl_Packages

I hope I answer all your question. I can provide patch for you, which stop removing configfile section, but final sollution need be more generic (this patch also breaks yast-bootloader in ncurses).
Comment 34 Markus Elfring 2008-07-25 11:29:21 UTC
(In reply to comment #33 from Josef Reidinger)
> 1) because menu.lst is parsed for known commands and configfile isn't as known

I find this acknowledgement very interesting. I wonder why my issue was closed even after it was known that it is not completely resolved so far.


> 2) http://en.opensuse.org/SUSE_Build_Tutorial#Perl_Packages

I would hope that the condition "perl >= 5.8.8" (instead of "perl = 5.10.0") would also be sufficient.


> I can provide patch for you, which stop removing configfile section,
> but final solution need be more generic (this patch also breaks
> yast-bootloader in ncurses).

I assume that more system administrators and advanced users are keen on a corresponding fix.
Comment 35 Josef Reidinger 2008-08-19 10:49:24 UTC
I fix it in svn and test it (and works). So now configfile survive any kernel update. Only problem is comments, which is removed. I try in future also survive all comments in global section.
Comment 36 Josef Reidinger 2008-09-12 12:52:49 UTC
0.4.70 released, if problem still happen please reopen with actual logs