[
https://issues.apache.org/jira/browse/AURORA-779?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14157002#comment-14157002
]
Bill Farner commented on AURORA-779:
------------------------------------
{quote}
the user should get a clean, comprehensible error message
{quote}
I don't think anybody is arguing against that, but this approach lumps expected
and unexpected errors into the same bin, and assumes we will always have the
context to figure out how the error happened. For example, i just bumped into
this because i had a syntax error in python code while changing something in
the client. This surfaced as:
{quote}
Internal error executing command: 'str' object has no attribute 'err_msg'
{quote}
At least in that case we're left with something searchable (ignoring any
{{getattr}}-like tricks). However, what if the error were something like this?
{quote}
Internal error executing command: unsupported operand type(s) for +: 'dict' and
'int'
{quote}
I just don't see how this catch-all helps the user, it seems to just admit that
we don't have a handle on where exceptions come from.
> v2 client has worse error message than v1
> -----------------------------------------
>
> Key: AURORA-779
> URL: https://issues.apache.org/jira/browse/AURORA-779
> Project: Aurora
> Issue Type: Bug
> Components: Client, Usability
> Reporter: Kevin Sweeney
>
> Copied from a user report at Twitter:
> I had a typo in an aurora config, where I was resolving a mustache variable
> outside of a string:
> {code}
> instances = {{profile.instances}}
> {code}
> where I should have been doing
> {code}
> instances = "{{profile.instances}}"
> {code}
> The error message on the v2 client was:
> {quote}
> Error executing command: Error loading configuration: name 'profile' is not
> defined
> {quote}
> The v1 client gave me a line number:
> {quote}
> File "router.aurora", line 90, in <module>
> instances = {{profile.instances}},
> NameError: name 'profile' is not defined
> {quote}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)