Code review comment for lp:~thumper/juju-core/machine-agent-environment

LGTM, except you have conflicts. Silly ones, from the looks of it:
mostly imports, but also a case where a function gained or lost an error
return.

I was expecting to see changes to more providers' tests. Don't those
need to set the provider type in various places as well?

https://codereview.appspot.com/12005048/