Bug 1152550

Summary: tuned bootloader plugin somehow duplicates cmdline option arguments
Product: Red Hat Enterprise Linux 7 Reporter: Jeremy Eder <jeder>
Component: grubbyAssignee: Peter Jones <pjones>
Status: CLOSED ERRATA QA Contact: Release Test Team <release-test-team-automation>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 7.1CC: bhu, cdonnell, jeder, jscotka, jskarvad, lcapitulino, mbanas, ovasik, pjones, williams
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: grubby-8.28-14.el7 Doc Type: Bug Fix
Doc Text:
Cause: grubby was incorrectly treating some grub2 configuration commands with the quoting rules of another command, and then re-assembling the result with a different procedure. This also happened in reverse. Consequence: Quotes got messed up and some lines that are assignments got duplicated parts. Fix: grubby's parser was modified to handle these items separately. Result: Duplicated sections of assignment lines should no longer be generated.
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-11-19 12:55:21 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1240765    
Attachments:
Description Flags
grub.cfg none

Comment 8 Jeremy Eder 2015-07-16 11:51:54 UTC
*** Bug 1240771 has been marked as a duplicate of this bug. ***

Comment 9 Jaroslav Škarvada 2015-07-20 14:51:25 UTC
I tried running stress test on several machines, but it didn't reproduce the problem.

Next time you encounter it, please provide content of grub.cfg, content of tuned.conf (or the profile name you used if it wasn't customized), content of the *variables.conf (if utilized) and content of /etc/tuned/bootcmdline.

Comment 10 Luiz Capitulino 2015-07-20 15:07:20 UTC
What's the stress test you ran?

For me to reproduce this problem is just a matter of installing a new kernel. I've provided full grub.cfg in bug 1152550. I can provide the other files and debug this more later today.

Comment 11 Jaroslav Škarvada 2015-07-20 16:36:29 UTC
(In reply to Luiz Capitulino from comment #10)
> What's the stress test you ran?
>
It was switching profiles and changing the bootcmdline in a loop for couple of minutes.
 
> For me to reproduce this problem is just a matter of installing a new
> kernel.
>
It didn't trigger it for me. I will retry.

> I've provided full grub.cfg in bug 1152550. I can provide the other
> files and debug this more later today.
>
Ok, thanks.

Comment 12 Jaroslav Škarvada 2015-07-20 17:05:42 UTC
(In reply to Jaroslav Škarvada from comment #11)
> > For me to reproduce this problem is just a matter of installing a new
> > kernel.
> >
> It didn't trigger it for me. I will retry.
> 
I finally reproduced it.

Comment 14 Luiz Capitulino 2015-07-20 17:25:36 UTC
That's excellent. I can test a fixed tuned version as soon as you have patches.

Comment 15 Jaroslav Škarvada 2015-07-22 18:58:33 UTC
Created attachment 1054983 [details]
grub.cfg

This is not tuned bug, but bug in grubby - it is crippling the grub.cfg. It may cause other problems, thus it needs to be fixed in grubby.

Version:
grubby-8.28-12.el7.x86_64

Reproducer:

Run the following command on the grub.cfg from the attachment:

# /sbin/grubby --grub2 -c /boot/grub2/grub.cfg --add-kernel=/boot/vmlinuz-3.10.0-297.el7.x86_64 --copy-default --title 'Red Hat Enterprise Linux Server (3.10.0-297.el7.x86_64) 7.2 (Maipo)' '--args=root=/dev/mapper/rhel_hp--dl380pgen8--02--vm--10-root ' '--remove-kernel=TITLE=Red Hat Enterprise Linux Server (3.10.0-297.el7.x86_64) 7.2 (Maipo)' --make-default

Result:
The following from grub.cfg:
set tuned_params="isolcpus=1,3"

is changed by grubby to:
set tuned_params="isolcpus=1,3"=1,3"

Notice the suffix '=1,3"' which shouldn't be added by grubby.

Comment 24 Craig Donnelly 2015-10-02 09:39:02 UTC
Apologies,

Probably should have directed this to BZ#1212128

Comment 25 errata-xmlrpc 2015-11-19 12:55:21 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-2110.html