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

Revision history for this message
Dave Pigott (dpigott) wrote :

And actually, since it is indeed a free format string, then if somebody decides to call u_boot "neddy-the-bootloader" we have no way of knowing what sort of bootloader it really is. I guess what I'm advocating is that we should standardise on a naming convention in the hwpacks. Otherwise we are asking for a lot of trouble.

Another key point is that if the dispatcher made a decision about the bootloader, then that goes against someone explicitly trying to specify and test a particular bootloader when we get to that point.

« Back to merge proposal