Bug 768399

Summary: Options Format File is not applied to command line prompts
Product: [Retired] Katello Reporter: Bryan Kearney <bkearney>
Component: InstallerAssignee: Lukas Zapletal <lzap>
Status: CLOSED UPSTREAM QA Contact: Katello QA List <katello-qa-list>
Severity: low Docs Contact:
Priority: unspecified    
Version: 1.0CC: tlestach
Target Milestone: ---Keywords: Patch, Triaged
Target Release: 1.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-07-17 13:09:38 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
proposed patch none

Description Bryan Kearney 2011-12-16 15:03:03 UTC 
I added this line to the file in /usr/share/katello/install/options-format-file.

# deployment types
deployment                  true   [headpin|katello]


It will catch errors in the default answer file. For example, if I set deploymnet=Fred. However, if leave the default file unchanged and I do this:

katello-confgure --deployment fred

I do not get an error. I would have expected an error to be raised.
Comment 1 Tomas Lestach 2012-01-09 12:22:26 UTC 
Created attachment 551548 [details]
proposed patch
Comment 2 Tomas Lestach 2012-01-09 12:22:53 UTC 
I assume, you've meant to use (headpin|katello) instead of [headpin|katello].
To work the reg exp as expected I'd recommend to use:

deployment                  true   ^(headpin|katello)$


However, there's a misbehavior. I'm attaching a patch, that fixes the default value check.
Comment 3 Lukas Zapletal 2012-04-02 13:47:12 UTC 
Pushing upstream, thanks Tomas!

1f7b670 768399 - deployment configure values are checked
Comment 4 Bryan Kearney 2012-07-17 13:09:38 UTC 
Moving all community bugs with fixes to closed.