Confusing error message when calling DashboardAPI.make_stream(..., name=None)

Bug #771182 reported by Jeremy Chang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
LAVA Dashboard (deprecated)
Fix Released
High
Dave Pigott

Bug Description

The command I tried is
"lc-tool make-stream --dashboard-url http://validation.linaro.org/launch-control/ /anonymous/jeremy/"

The error log returns as
"XML-RPC error 409: Stream with the specified pathname already exists"

Related branches

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

The actual issue is caused by lc-tool passing None as the default argument to "name". On the server side this caused IntegrityError (as name cannot be NULL). The client is then given a rather confusing error message (normally IntegrityError only occurs when we have a duplicate stream path).

To solve this we should make lc-tool provide the correct default (empty string) and l-c to handle this situation better (perhaps by converting None to an empty string)

Changed in launch-control:
status: New → Confirmed
importance: Undecided → High
summary: - Stream create can not work if missing --name "something" argument
+ Confusing error message when calling DashboardAPI.make_stream(...,
+ name=None)
Changed in launch-control-tool:
status: New → Confirmed
importance: Undecided → High
Paul Larson (pwlars)
Changed in launch-control:
assignee: nobody → Dave Pigott (dpigott)
Changed in launch-control-tool:
assignee: nobody → Dave Pigott (dpigott)
Zygmunt Krynicki (zyga)
Changed in launch-control-tool:
milestone: none → 0.5
Changed in launch-control:
milestone: none → 0.5
Revision history for this message
Dave Pigott (dpigott) wrote :

Surely as long as make_stream on the server side handles name being None by substituting it with empty string, then we don't need to modify the client?

Zygmunt Krynicki (zyga)
Changed in lava-dashboard:
status: Confirmed → In Progress
milestone: 0.5 → 11.07
Zygmunt Krynicki (zyga)
Changed in lava-dashboard:
status: In Progress → Fix Committed
Zygmunt Krynicki (zyga)
Changed in lava-dashboard-tool:
milestone: 0.5 → 2011.06
status: Confirmed → Fix Committed
Paul Larson (pwlars)
Changed in lava-dashboard:
status: Fix Committed → Fix Released
Changed in lava-dashboard-tool:
status: Fix Committed → 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.