Bug 796381

Summary: Unable to upload a CLI alert script using the Upload mechanism on the Repository detail page
Product: [Other] RHQ Project Reporter: Lukas Krejci <lkrejci>
Component: Core UIAssignee: Lukas Krejci <lkrejci>
Status: CLOSED WONTFIX QA Contact: Mike Foley <mfoley>
Severity: unspecified Docs Contact:
Priority: medium    
Version: 4.3CC: hrupp, snegrea
Target Milestone: ---   
Target Release: JON 3.0.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 794768 Environment:
Last Closed: 2012-02-27 22:40:19 UTC Type: ---
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: 794768, 840650    
Bug Blocks:    

Description Lukas Krejci 2012-02-22 19:27:50 UTC 
+++ This bug was initially created as a clone of Bug #794768 +++

Description of problem:
$SUBJECT

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


How reproducible:
always

Steps to Reproduce:
1. Create a repository
2. Scroll down, upload a js script
3. select type of "Serverside CLI Script"
4. Click "Create Package" button
  
Actual results:
exception complaining that a sha256 hash is not an OSGi version string (which it isn't)

Expected results:
upload should work

Additional info:
This exception is most likely caused by the validation the CLI package type performs on the version string.
Not sure what the solution should be here though. Using sha256 as a version string makes a lot of sense (as we've seen lately) but at the same time the validation (of CLI package type versions )that is in place right now does a good deal of user visible version validation. Maybe we could just change the validator to check the display version instead of the version field?

--- Additional comment from lkrejci on 2012-02-17 10:12:54 EST ---

Stefan, what is your opinion on this?

--- Additional comment from lkrejci on 2012-02-17 10:17:29 EST ---

The uploading using the CLI alert notification definition popup while defining an alert definition still works though...
Comment 1 Lukas Krejci 2012-02-22 19:29:00 UTC 
Note that this is a documented feature in JON and a regression from JON 3.0.0.
Comment 2 Mike Foley 2012-02-27 15:47:22 UTC 
triage to JON 3.1 mfoley,crouch
Comment 3 Mike Foley 2012-02-27 17:13:09 UTC 
triage 2/27/2012 mfoley, asantos, crouch, loleary