Bug 980352 - No failure message when system update status can't be saved
Summary: No failure message when system update status can't be saved
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: web UI
Version: 0.13
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 19.0
Assignee: Dan Callaghan
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On: 1014438
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-02 07:04 UTC by Nick Coghlan
Modified: 2018-02-06 00:41 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-11-25 07:18:14 UTC
Embargoed:


Attachments (Terms of Use)

Description Nick Coghlan 2013-07-02 07:04:25 UTC
Description of problem:

While attempting to mark a system as broken, it fails if the condition report is too long.

When I copied a snippet from the console log, it refused to save the update, but didn't tell me why - I had to guess.

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

Beaker 0.13.1

How reproducible:

Always

Steps to Reproduce:
1. Mark a system as Broken
2. Enter a long-ish Condition Report
3. Try to save the changes

Actual results:

Edit page is redisplayed with no error message

Expected results:

Error message is displayed, explaining the condition report is too long (and specifying both the length limit and the current length)

Additional info:

It may not hurt to allow for longer condition reports, too. The error message I was trying to paste in really wasn't *that* long:

From console logs:

HA-0      
(Bus       
2       
Dev       
14)       
PERC       
4e/Si       
Standard       
FW       
513O       
DRAM       
=256MB       
(SDRAM)      
Battery      
module       
is       
present       
on       
adapter      
Memory/battery       
problems       
were       
detected.      
The      
adapter       
has       
recovered,       
but       
cached       
data       
was       
lost.      
Press      
any       
key       
to       
continue.

Comment 1 Dan Callaghan 2013-08-21 23:48:07 UTC
I guess the condition report field is failing TG validation but the template for that form doesn't have the necessary bits to display validation errors on every field.

Comment 4 Dan Callaghan 2014-09-24 09:46:21 UTC
One day we will have a database that doesn't just silently truncate our data... In the meantime:

http://gerrit.beaker-project.org/3351

I also took the opportunity to increase the column length from 255 to 4000:

http://gerrit.beaker-project.org/3350

Comment 7 Dan Callaghan 2014-11-25 07:18:14 UTC
Beaker 19.0 has been released.


Note You need to log in before you can comment on or make changes to this bug.