Bug 15374 - btrfs / and /home on sda7 and sda8 correctly forces grub2 but is unable to install it
Summary: btrfs / and /home on sda7 and sda8 correctly forces grub2 but is unable to in...
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: Installer (show other bugs)
Version: Cauldron
Hardware: i586 Linux
Priority: release_blocker major
Target Milestone: ---
Assignee: Thierry Vignaud
QA Contact:
URL:
Whiteboard:
Keywords:
: 12362 (view as bug list)
Depends on:
Blocks: 14069 15846
  Show dependency treegraph
 
Reported: 2015-02-26 22:30 CET by claire robinson
Modified: 2015-05-11 11:08 CEST (History)
4 users (show)

See Also:
Source RPM:
CVE:
Status comment:


Attachments
report.bug.xz - had to compress it (163.55 KB, application/x-xz)
2015-02-26 22:32 CET, claire robinson
Details

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)

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