<div dir="ltr">Hi all,<div><br></div><div>Ian raised a good point in <a href="http://reviews.vapour.ws/r/885/">http://reviews.vapour.ws/r/885/</a> (caution advised, may cause eyebleed) about a change I made to errors: <a href="https://github.com/juju/errors/pull/17">https://github.com/juju/errors/pull/17</a></div><div><br></div><div>The NotAssigned error, which previously was only raised by state.Unit.AssignedMachineId, is now also raised by state.Volume.StorageInstance. I removed the error from the state package so we didn't have apiserver/common poking its head in there, and moved it over to the juju/errors package.</div><div><br></div><div>The issue Ian raised is that juju/errors should contain relatively generic error types, and we should keep domain-specific things elsewhere. Examples are NotAssigned, and NotProvisioned. We're thinking of moving these domain-specific error types into a new package, juju/juju/errors. What do you think about this?</div><div><br></div><div>Cheers,</div><div>Andrew</div></div>