Bug 1387256
Summary: | RFE: allow customizing options to kickstart %packages | ||||||
---|---|---|---|---|---|---|---|
Product: | [Retired] Beaker | Reporter: | Jeff Bastian <jbastian> | ||||
Component: | general | Assignee: | Dan Callaghan <dcallagh> | ||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | tools-bugs <tools-bugs> | ||||
Severity: | low | Docs Contact: | |||||
Priority: | low | ||||||
Version: | 23 | CC: | dcallagh, dowang, mjia, mkolar, rjoost | ||||
Target Milestone: | 24.0 | Keywords: | FutureFeature, Patch | ||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | If docs needed, set a value | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2017-02-21 18:51:25 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: | |||||||
Attachments: |
|
Description
Jeff Bastian
2016-10-20 13:12:53 UTC
If I'm reading the Jinja2 template correctly, I think a one-line patch to add a "pkgoptions" variable to the kickstart metadata should do the trick: diff --git a/Server/bkr/server/kickstarts/default b/Server/bkr/server/kickstarts/default index e937945e92d4..c4d72d87f930 100644 --- a/Server/bkr/server/kickstarts/default +++ b/Server/bkr/server/kickstarts/default @@ -70,7 +70,7 @@ unsupported_hardware {% endif %} {% if has_rpmostree is not defined %} -%packages --ignoremissing +%packages {{ pkgoptions|default('--ignoremissing') }} {%- if not recipe and packages is undefined %} --default {% else %} Example usage: <recipe ks_meta="pkgoptions='--excludedocs --ignoremissing'" ....> Created attachment 1212631 [details]
pkgoptions patch
This patch adds the pkgoptions kickstart metadata variable plus an integration test and documentation.
I'm having difficulty setting up a development environment in order to actually run the integration tests, however, so the patch is just a proof of concept at this time. If somebody with a development environment already set up can run a sanity test on my patch, I would appreciate it.
I got my devel environment fixed up with help from Dan Callaghan, Matt Jia, and Nick Coghlan -- thanks all! -- and verified the patch is working with an added test in the kickstart integration tests. The patch is in gerrit: https://gerrit.beaker-project.org/5346 BTW Jeff why did you need this? Which option do you want to pass down to %packages? Just curious. Michal Kolar (CC'ed on this bug) asked for help on IRC for using the --instLangs option with the %packages section. Beaker 24.0 has been released. |