Code review comment for lp:~fcorrea/landscape-client/add-otp-field

+1!

[5]

+ parser.add_option("--provisioning-otp", default="",

I think you can just omit the default="" parameter and you will get None if nothing is passed. I see it's there for the cloud otp option, but I'm not sure if there's a good reason for that.