Bug 701236

Summary: Setting "Kickstart Label" for a cloned KS is failing in case of "SPACE" at the end.
Product: Red Hat Satellite 5 Reporter: Dimitar Yordanov <dyordano>
Component: WebUIAssignee: Tomas Lestach <tlestach>
Status: CLOSED NOTABUG QA Contact: Red Hat Satellite QA List <satqe-list>
Severity: low Docs Contact:
Priority: low    
Version: 540   
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-05-02 10:16:08 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Dimitar Yordanov 2011-05-02 08:33:06 UTC 
Description of problem:
Setting "Kickstart Label"  for a cloned KS is failing in case of "SPACE" at the end.

Version-Release number of selected component (if applicable):
Satellite 5.4

How reproducible:
Always

Steps to Reproduce:
1.System->Kickstart->Profiles->some-KS -> clone kickstart
2.Kickstart Label : aaabbb(space)

  
Actual results:
Your selected kickstart label is invalid, it must be at least 6 characters and only contain the following characters : A-Z, a-z, 0-9, _, - 

Expected results:
Success

Additional info:
Comment 1 Tomas Lestach 2011-05-02 10:16:08 UTC 
Dimitar, what you describe here, is the correct behavior.
Like the error message you got displayed on the WebUI: space is not among the allowed charactes: "A-Z, a-z, 0-9, _, -".
Closing as NOTABUG.
Comment 2 Dimitar Yordanov 2011-05-02 11:52:00 UTC 
What I think is that the space at the end should be truncated the KS cloned.

aaabbb(SPACE) == aaabbb
Comment 3 Tomas Lestach 2011-05-02 12:18:53 UTC 
User input shall not be modified, if not stated.
RHN Satellite doesn't try to be smarter than the user.
Comment 4 Dimitar Yordanov 2011-05-02 13:18:48 UTC 
I just do not think that any user intends to have KS names : aaabbb and aaabbbb(SPACE).
But when  happen inevitably to leave one space at the end and gets an error because of that, this could be annoying.