Bug 15374

Summary:	btrfs / and /home on sda7 and sda8 correctly forces grub2 but is unable to install it
Product:	Mageia	Reporter:	claire robinson <eeeemail>
Component:	Installer	Assignee:	Thierry Vignaud <thierry.vignaud>
Status:	RESOLVED FIXED	QA Contact:
Severity:	major
Priority:	release_blocker	CC:	ennael1, rverschelde, tarakbumba, tmb
Version:	Cauldron
Target Milestone:	---
Hardware:	i586
OS:	Linux
Whiteboard:
Source RPM:		CVE:
Status comment:
Bug Depends on:
Bug Blocks:	14069, 15846
Attachments:	report.bug.xz - had to compress it

Description claire robinson 2015-02-26 22:30:59 CET

5RC first build (Feb 25th) testing DVD 32

Checked btrfs support as it should now default to grub2 if btrfs root partition with no separate boot, which is does nicely.

Installed in dual boot with mga4, so partitions sda7 and sda8 are used for testing installs.

The installer is unable to install grub2 to the / partition on sda7. It also missed the mga4 chainload it normally adds.

grabbed a report.bug which I'll attach.

Line 19878

* error: You cannot install the bootloader on a btrfs partition
	...propagated at /usr/lib/libDrakX/any.pm line 264.

This is the error which was displayed in the installer.

Reproducible: 

Steps to Reproduce:

Comment 1 claire robinson 2015-02-26 22:32:47 CET

Created attachment 5953 [details]
report.bug.xz - had to compress it

Comment 2 claire robinson 2015-02-26 22:34:12 CET

Setting as release blocker as this prevents installation completing and this is named as one of mga5 features.

Priority: Normal => release_blocker
CC: (none) => ennael1, tmb
Blocks: (none) => 14069

Comment 3 Thomas Backlund 2015-02-26 22:36:17 CET

that feature was dropped thankfully, but we should enforce separate /boot

Comment 4 claire robinson 2015-02-26 22:41:46 CET

it's mentioned in release notes. If btrfs is chosen as root and no separate boot then it forces grub2. That bit works. It won't actually install it though.

It's perhaps badly worded there as it says btrfs by default, which we dont want yet do we

Comment 5 Atilla ÖNTAŞ 2015-03-18 08:09:07 CET

Doesn't this previous #12362 bug?

CC: (none) => tarakbumba

Comment 6 Rémi Verschelde 2015-03-20 00:44:45 CET

*** Bug 12362 has been marked as a duplicate of this bug. ***

Comment 7 Rémi Verschelde 2015-03-20 00:45:55 CET

What is the status on this bug? Is it still valid in Mageia 5 RC?
Are there any leads for a possible resolution?

Rémi Verschelde 2015-03-29 22:04:58 CEST

CC: (none) => remi

Comment 8 Thierry Vignaud 2015-03-31 15:16:58 CEST

Well, thomas, is grub2 supposed to be working on btrfs?
If yes, we just need to drop btrfs from bootloader::install() :
http://gitweb.mageia.org/software/drakx/tree/perl-install/bootloader.pm#n2047

aka reverting part of commit 900590657e:
http://gitweb.mageia.org/software/drakx/diff/perl-install/bootloader.pm?id=900590657e

Also it looks like some people reported having success with grub2 & /boot as xfs, so could drop the fs check from install() and just list xfs along btrfs here:http://gitweb.mageia.org/software/drakx/diff/perl-install/bootloader.pm?id=fb5ed803
  && (/grub2/ || !member ($boot_part->{fs_type}, qw(btrfs xfs))
(but for mga6 :-)

Comment 9 Anne Nicolas 2015-04-05 23:19:17 CEST

Thomas, any input on that one ?

Comment 10 Mageia Robot 2015-04-06 19:00:00 CEST

commit 8ffb546708149e582a2fb92efbff948de88ed800
Author: Thomas Backlund <tmb@...>
Date:   Mon Apr 6 19:58:47 2015 +0259

    allow btrfs for / without separate /boot with grub2 (mga#15374)
---
 Commit Link:
   http://gitweb.mageia.org/software/drakx/commit/?id=8ffb546708149e582a2fb92efbff948de88ed800

Comment 11 Thierry Vignaud 2015-04-07 10:40:20 CEST

Closing

Status: NEW => RESOLVED
Resolution: (none) => FIXED

Thierry Vignaud 2015-05-11 11:06:51 CEST

Blocks: (none) => 15846

Comment 12 Thierry Vignaud 2015-05-11 11:08:07 CEST

This obviously don't work (see bug #15846)