Code review comment for lp:~andreserl/maas/ipmi_usercreation_ilo_versions_trunk

Revision history for this message
Scott Moser (smoser) wrote :

Andres,
  Thanks for looking at this. I think the implementation seems generally correct, and I suspect you've tested.
  However, I'd like 2 things
a.) since this is non straight forward when you look at it, could you add some comments explaining why you're doing what you're doing?
b.) this really probably should have a bug associated with it (sorry for that nit pick, but in absence of comments, I'd go looking for a bug to read about what was going on).

review: Needs Fixing

« Back to merge proposal