Bug 1328935

Summary:	Operating system description length accepts more than 255 characters
Product:	Red Hat Satellite	Reporter:	sthirugn <sthirugn>
Component:	WebUI	Assignee:	Daniel Lobato Garcia <dlobatog>
Status:	CLOSED ERRATA	QA Contact:	Renzo Nuccitelli <rnuccite>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	6.2.0	CC:	bbuckingham, bkearney, lpramuk, oshtaier, rnuccite
Target Milestone:	Unspecified	Keywords:	Regression, Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
URL:	http://projects.theforeman.org/issues/14783
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-02-21 16:54:17 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:

Description sthirugn@redhat.com 2016-04-20 16:06:59 UTC

Description of problem:
Operating system description length accepts more than 255 characters

Version-Release number of selected component (if applicable):
satellite-6.2.0-8.2.beta.el7sat.noarch

How reproducible:
Always

Steps to Reproduce:
1.Create an operating system with description of length 256 characters
2.
3.

Actual results:
Operating system created with no errors

Expected results:
Until last build, we allowed os description length upto a maximum of 255 characters only, now its broken.

Additional info:

Comment 4 Oleksandr Shtaier 2016-04-21 08:50:54 UTC

Raising the severity as operating system is unique entity in a way that Title of such entity is formed using not only name, but description too. We have similar increase of description length across other entities, but for them impact is minimal and do not transform table in a bad shape to make user experience worse.

Saying that we have a valid issue, no matter what reason get us there, so corresponding database field has to have sensible limit and we have to force it.

Comment 5 Daniel Lobato Garcia 2016-04-22 14:11:33 UTC

Created redmine issue http://projects.theforeman.org/issues/14783 from this bug

Comment 6 Bryan Kearney 2016-04-22 16:06:50 UTC

Upstream bug assigned to dlobatog

Comment 7 Bryan Kearney 2016-04-22 16:06:52 UTC

Upstream bug component is WebUI

Comment 8 Bryan Kearney 2016-06-13 10:15:31 UTC

Moving to POST since upstream bug http://projects.theforeman.org/issues/14783 has been closed

Comment 13 Renzo Nuccitelli 2016-11-29 11:07:05 UTC

I just ran the test and it passed on sat 6.3 snap 6. So I am moving this to verified.

Comment 14 Oleksandr Shtaier 2017-01-19 11:13:02 UTC

Resolved only for 6.3, but not for 6.2.z. Should we clone that defect?

Comment 15 Lukas Pramuk 2017-01-19 14:20:27 UTC

Oleksander,

What should suffice here is to set sat-6.2.z flag to '?'
But as you can see it gets changed to '+' shortly

Comment 16 Satellite Program 2018-02-21 16:54:17 UTC

Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA.
> 
> For information on the advisory, and where to find the updated files, follow the link below.
> 
> If the solution does not work for you, open a new bug report.
> 
> https://access.redhat.com/errata/RHSA-2018:0336