Bug 1162513

Summary: PReP Boot partition is not added when provisioning ppc64le distros with custom partitions
Product: [Retired] Beaker Reporter: Zorro Lang <zlang>
Component: generalAssignee: Dan Callaghan <dcallagh>
Status: CLOSED CURRENTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: high Docs Contact:
Priority: high    
Version: 0.18CC: aigao, asaha, dcallagh, dowang, eguan, mbanas
Target Milestone: 19.1Keywords: Patch, TestBlocker
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-12-17 08:40:18 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:

Comment 1 Zorro Lang 2014-11-11 08:46:00 UTC
Created attachment 956180 [details]
job's xml

Comment 2 Zorro Lang 2014-11-11 08:46:23 UTC
Created attachment 956181 [details]
console.log

Comment 3 Zorro Lang 2014-11-11 08:47:01 UTC
Created attachment 956182 [details]
anaconda.log

Comment 4 Zorro Lang 2014-11-11 08:47:22 UTC
Created attachment 956183 [details]
storage.log

Comment 5 Zorro Lang 2014-11-11 08:47:52 UTC
Created attachment 956184 [details]
sys.log

Comment 6 Eryu Guan 2014-11-11 09:01:30 UTC
This would blocks most local filesystem tests on ppc64le in beaker, set testblocker keyword.

Comment 7 Nick Coghlan 2014-11-11 09:12:33 UTC
Beaker uses Anaconda to create the partitions. The partitions section of the generated kickstart looks normal:

===============
part /boot --size 200 --recommended --asprimary
part / --size 1024 --grow
part swap --recommended
 
part /mnt/ltp-aiodio --size=5120 --fstype xfs
===============

If autopart recipes (which are the default) are working, while explicit partitions are failing, then the issue is more likely to be on the Anaconda side.

Comment 8 Martin Banas 2014-11-11 09:37:56 UTC
Hi Nick,
PrepBoot partition is missing in kickstart

Comment 9 Nick Coghlan 2014-11-11 10:01:25 UTC
Ah, now I see the bug - there's a hardcoded check for ppc and ppc64 that adds the "PPC PReP Boot" partition in the rhts_part snippet:

    part None --fstype 'PPC PReP Boot' --size 8

That check currently isn't triggering for ppc64le.

As a possible workaround, Beaker may let you add a partition like:

    <partition fs="PPC PReP Boot" name="None" size="8" type="part"/>

(I'm not sure that will actually work, but it's worth trying)

Comment 10 Dan Callaghan 2014-11-11 22:39:52 UTC
(In reply to Nick Coghlan from comment #9)
> As a possible workaround, Beaker may let you add a partition like:
> 
>     <partition fs="PPC PReP Boot" name="None" size="8" type="part"/>

Unfortunately I think this will give you an 8GB PReP Boot partition...

Comment 11 Nick Coghlan 2014-11-12 01:41:55 UTC
With 19.0 already in acceptance testing, it makes more sense to target 19.1 for this fix (that should still be available later this month).

If the oversized prep boot partition workaround isn't adequate in the meantime, please let us know and we can get the updated rhts_partitions snippet deployed early via Beaker's snippet customisation support.

Comment 12 Nick Coghlan 2014-11-17 05:17:30 UTC
Bug 1164660 is an RFE against RHEL 7 requesting the ability to tell Anaconda to create all required partitions, while still being able to define custom partitions.

Comment 13 Dan Callaghan 2014-11-19 00:21:57 UTC
http://gerrit.beaker-project.org/3492

Also fixed an oversight where --ondisk= was not being passed for the PReP Boot partition.

Comment 21 Dan Callaghan 2014-12-17 08:40:18 UTC
Beaker 19.1 is released.