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

[Jeremy Chang]

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"

[Zygmunt Krynicki (zyga)]

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)

[Dave Pigott (dpigott)]

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?