value too long for type character varying(32)

Bug #892562 reported by Paul Larson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
LAVA Dashboard (deprecated)
Fix Released
High
Michael Hudson-Doyle
LAVA Validation Lab
Fix Released
High
Unassigned

Bug Description

Looks like all the daily runs from today failed to deserialize. Unfortunately, making sense of the deserialization error is still a bit of a pain. Looking at the model and the bundle though, my best guess is that it's the NamedAttribute name limitation of 32 characters. It looks like there was also a change that went in that added this:
"target.linaro-media-create-version": "linaro-media-create 2011.10 : qemu-arm version 0.15.50 (Debian\n0.15.50-2011.10-0ubuntu1~ppa10.10.1), Copyright (c) 2003-2008 Fabrice Bellard",

I've hot-patched this on the server for now, but I think this needs to be increased.

as a side note, I seem to recall some discussion a long time ago about whether it made sense to limit the number of characters, as there was some sort of alleged performance gain by not putting a limit on them. What ever happened to that?

Related branches

Revision history for this message
Paul Larson (pwlars) wrote :

The task for lava-lab is to upgrade dashboard once we have a fix for this, and re-install, or undo the hotfix I did for lava-dispatcher to make it quit sending this attribute

Changed in lava-dashboard:
importance: Undecided → High
milestone: none → 2011.12
Revision history for this message
Zygmunt Krynicki (zyga) wrote : Re: [Bug 892562] [NEW] value too long for type character varying(32)

How did you hot patch this?

Wysłane z iPhone'a

Dnia 19 lis 2011 o godz. 20:20 Paul Larson <email address hidden> napisał(a):

> Public bug reported:
>
> Looks like all the daily runs from today failed to deserialize. Unfortunately, making sense of the deserialization error is still a bit of a pain. Looking at the model and the bundle though, my best guess is that it's the NamedAttribute name limitation of 32 characters. It looks like there was also a change that went in that added this:
> "target.linaro-media-create-version": "linaro-media-create 2011.10 : qemu-arm version 0.15.50 (Debian\n0.15.50-2011.10-0ubuntu1~ppa10.10.1), Copyright (c) 2003-2008 Fabrice Bellard",
>
> I've hot-patched this on the server for now, but I think this needs to
> be increased.
>
> as a side note, I seem to recall some discussion a long time ago about
> whether it made sense to limit the number of characters, as there was
> some sort of alleged performance gain by not putting a limit on them.
> What ever happened to that?
>
> ** Affects: lava-dashboard
> Importance: High
> Status: New
>
> ** Affects: lava-lab
> Importance: Undecided
> Status: New
>
> ** Changed in: lava-dashboard
> Importance: Undecided => High
>
> ** Changed in: lava-dashboard
> Milestone: None => 2011.12
>
> ** Also affects: lava-lab
> Importance: Undecided
> Status: New
>
> --
> You received this bug notification because you are a member of Linaro
> Validation Team, which is subscribed to LAVA Dashboard.
> https://bugs.launchpad.net/bugs/892562
>
> Title:
> value too long for type character varying(32)
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/lava-dashboard/+bug/892562/+subscriptions

Fathi Boudra (fboudra)
Changed in lava-lab:
importance: Undecided → High
milestone: none → 2011.12
Revision history for this message
Paul Larson (pwlars) wrote :

@zyga: I just made the dispatcher not add this information for now

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

I'll take a look at this today.

Changed in lava-dashboard:
assignee: nobody → Michael Hudson-Doyle (mwhudson)
Changed in lava-dashboard:
status: New → Fix Committed
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

This was released with the 2011-12-06 flexible deployment

Changed in lava-lab:
status: New → Fix Released
Changed in lava-dashboard:
status: Fix Committed → Fix Released
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

The migration never ran so I'm reopening this in the lab.

Changed in lava-lab:
status: Fix Released → In Progress
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

We've run it now.

Changed in lava-lab:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.