<html><head></head><body bgcolor="#FFFFFF"><div> Begin forwarded message: </div><blockquote type="cite"><div>From: Dave Cheney <<a href="mailto:dave@cheney.net">dave@cheney.net</a>> Date: 20 June 2012 19:59:54 AEST To: <a href="mailto:juju-dev@lists.ubuntu.com">juju-dev@lists.ubuntu.com</a> Subject: Machine.InstanceId error condition. </div></blockquote><div></div><blockquote type="cite"><div>Hello, I'm working on a branch to address the todo surrounding machine.InstanceId retuning an error if no Id was set. Given the frequency of this condition (at least 50% of the code paths expect the instance id to be _not_ set) I am considering this signature. id, set, err := machine.InstanceId() Where set is a Boolean, and error is only non nil when something error worthy happened. In other words, id should not be an empty string unless set is false or err is non nil. I've already prototyped the more obvious id, err := ... version, but found that I was duplicating a lot of logic dissecting the err value for the NoInstanceIdError sentinel. Thoughts Cheers Dave</div></blockquote></body></html>