Code review comment for lp:~tycho-s/maas/add-missing-node-fields

Revision history for this message
Tycho Andersen (tycho-s) wrote :

On Wed, Nov 27, 2013 at 11:01:54PM -0000, Julian Edwards wrote:
> Review: Disapprove
>
> Indeed - it's why they're not writable by the API or the UI, the are supposed
> to come from hardware inventory. If that process is not working, we should
> fix it, not paper over it.
>
> However, I'll change my vote if you can justify it :)

If I create a node via `maas-cli maas nodes new', there is currently
no way to set the hwinfo that I can see. I spoke with jtv, and he
seemed to think this was an oversight. That said, is there any reason
/not/ to support this? What happens if I add some RAM to a box?

« Back to merge proposal