Bug 1082163

Summary: Content Views: Long descriptions for view cause mostly-silent failure to publish
Product: Red Hat Satellite Reporter: Corey Welton <cwelton>
Component: WebUIAssignee: Walden Raines <walden>
Status: CLOSED CURRENTRELEASE QA Contact: Katello QA List <katello-qa-list>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.2CC: mmccune, walden
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-07-02 14:04:41 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 Flags
error.log from candlepin none

Description Corey Welton 2014-03-28 21:06:57 UTC
Description of problem:
When a user enters a long description for a content view (presumably more than 255 characters?), subsequent attempts to promote error out.  This does not seem to be readily apparent in UI except for a tiny red smidge in the progress bar and clicking through to see that "something" caused an error.  Traceback in candlepin.log


Version-Release number of selected component (if applicable):
Satellite-6.0.3-RHEL-6-20140328.1

How reproducible:


Steps to Reproduce:
1.  Create  view called "this is a test" with the following description:

this is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a testthis is a test
2.  Noting that view is created with no failure, add content to view and attempt to promote it.
3.  View results

Actual results:
An incomplete red status bar; clicking through "Publishing and promoting to 1 environment." notes an error but doesn't really provide details.


Expected results:
Validation in the UI to keep user from creating a content view longer than allowable db field length.


Additional info:
See forthcoming example from candlepin.log

Comment 1 Corey Welton 2014-03-28 21:08:43 UTC
Created attachment 880028 [details]
error.log from candlepin

Note: I referred to the above file twice as "candlepin.log".  Rather, it is the error log.

Comment 3 Walden Raines 2014-04-11 18:08:35 UTC
PR: https://github.com/Katello/katello/pull/3973

Comment 4 Walden Raines 2014-04-14 17:58:39 UTC
commit facafed473d7d7900cedbe6f211f63f370921182
Merge: 53902fb 99d8144
Author: Walden Raines <walden>
Date:   Mon Apr 14 13:57:51 2014 -0400

    Merge pull request #3973 from waldenraines/bug-5160
    
    Fixes #5160, BZ1082163 - limit content view descriptions to 255 chars.

commit 99d814414cfd4bd218eb46a22b59b7f02206f2f7
Author: Walden Raines <walden>
Date:   Fri Apr 11 14:05:59 2014 -0400

    Fixes #5160, BZ1082163 - limit content view descriptions to 255 chars.

Comment 6 Corey Welton 2014-05-05 19:21:42 UTC
Verified in Satellite/Satellite-6.0.3-RHEL-6-20140430.4, UI tells user that descriptions > 255char are disallowed.

Comment 7 Bryan Kearney 2014-07-02 14:04:41 UTC
This was delivered with 6.0.3, which is the Satellite 6 Beta.