Code review comment for lp:~dpigott/lava-dispatcher/add-bootloader-support

Revision history for this message
Andy Doan (doanac) wrote :

On 11/20/2012 10:58 AM, Dave Pigott wrote:
> Ok. So what you're saying is that to support this, we need to accept a selection in the JSON file, and then use the read-hwpack option to determine if it's valid. If nothing is specified we use the first in the list for the matching device type.
>
> That's probably enough to push it into backlog, to be honest. If my change is unsupportable, then let's just throw it away,

Fathi and I discussed on IRC. I think what you have now is good. I don't
think we need to do validation of the bootloader parameter. we'll put
the responsibility on the user for now. In the, likely near, future we
can do Fathi's suggesion and validated (or automatically choose) based
on this l-m-c command.

« Back to merge proposal