Bug 952635 - colon in quoted package require breaks kickstart
Summary: colon in quoted package require breaks kickstart
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: lab controller
Version: 0.11
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: 0.16.2
Assignee: Dan Callaghan
QA Contact: tools-bugs
URL:
Whiteboard: Kickstart
Depends On:
Blocks: 798718
TreeView+ depends on / blocked
 
Reported: 2013-04-16 10:52 UTC by Ales Zelinka
Modified: 2018-02-06 00:41 UTC (History)
13 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-04-28 23:00:28 UTC
Embargoed:

Attachments (Terms of Use)

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.
Note You need to log in before you can comment on or make changes to this bug.