Bug 1402881

Summary: Mandatory options in kickstart advanced options accepted even if empty
Product: Red Hat Satellite 5 Reporter: Radovan Drazny <rdrazny>
Component: WebUIAssignee: Grant Gainey <ggainey>
Status: CLOSED DUPLICATE QA Contact: Red Hat Satellite QA List <satqe-list>
Severity: low Docs Contact:
Priority: unspecified    
Version: 570CC: tlestach
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-02-15 20:00: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:
Bug Depends On:    
Bug Blocks: 1358815    

Description Radovan Drazny 2016-12-08 14:28:42 UTC
Description of problem:
In advanced options for a kickstart profile, there are some mandatory options marked with asterisk. If user unchecks mandatory option checkbox and attempts to update the kickstart, Satellite will return error about missing mandatory option. If user checks the mandatory option, but leaves text field empty, kickstart file is updated, even if there is no parameter for the options. 


Version-Release number of selected component (if applicable):
spacewalk-java-2.3.8-157.el6sat

How reproducible:
always

Steps to Reproduce:
1. Have a kickstart profile created.
2. Select a profile in Kickstart profiles, go to Kickstart Details -> Advanced Options. 
3. Verify that "auth" option is checked, and delete content of its text field. 
4. Press Update Kickstart button. 

Actual results:
Satellite happily accepts the update and the kickstart file now contains "auth" option without any parameters.

Expected results:
Validating that parameters for each and every (mandatory) option are correct would be almost impossible, of course. Still, there should be at least some kind of warning that option is enabled but without any parameters, and that the resulting kickstart file may not work as expected because of this.

Additional info:

Comment 2 Grant Gainey 2017-02-15 20:00:18 UTC

*** This bug has been marked as a duplicate of bug 1357824 ***