Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 952635

Summary: colon in quoted package require breaks kickstart
Product: [Retired] Beaker Reporter: Ales Zelinka <azelinka>
Component: lab controllerAssignee: Dan Callaghan <dcallagh>
Status: CLOSED CURRENTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: high Docs Contact:
Priority: high    
Version: 0.11CC: aigao, asaha, dapospis, dcallagh, jingwang, lzachar, mkyral, psklenar, psplicha, qwan, rmancy, tools-bugs, xjia
Target Milestone: 0.16.2   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: Kickstart
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-04-28 23:00:28 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: 798718    

Comment 1 Amit Saha 2013-04-16 11:02:07 UTC
Marking it as private, since it has some internal RH URLs

Comment 3 Dan Callaghan 2013-04-22 01:45:19 UTC
The colon problem comes from the fact that the packages ks_meta variable uses colon as a separator:
http://git.beaker-project.org/cgit/beaker/tree/Server/bkr/server/model.py#n4946

But as far as I can tell Anaconda doesn't actually allow arbitrary RPM requirements in %packages, it has to be actual package names. So this isn't going to work properly anyway.

The real solution would be a harness which installs task requirements instead of putting them into kickstart %packages...

Comment 5 Dan Callaghan 2014-03-31 04:11:39 UTC
On Gerrit: http://gerrit.beaker-project.org/2991

This patch fixes the kickstart generation so that task requirements are passed through as is, even when they contain colons.

Note that this does not change the fact that Anaconda does not accept arbitrary requires in %packages, only package names, so things like perl(Archive::Tar) will still not work, although they will now appear in the kickstart correctly.

Comment 11 Dan Callaghan 2014-04-28 23:00:28 UTC
Beaker 0.16.2 has been released.