Code review comment for lp:~axwalk/juju-core/delete-sudo-chown

Revision history for this message
Andrew Wilkins (axwalk) wrote :

Reviewers: mp+203665_code.launchpad.net,

Message:
Please take a look.

Description:
utils: remove sudo/chown helpers

We no longer need these, as we no longer require
sudo around juju for local. The one remaining user,
utils/ssh, has been updated to no longer use the
helpers.

https://code.launchpad.net/~axwalk/juju-core/delete-sudo-chown/+merge/203665

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/58160043/

Affected files (+3, -244 lines):
   A [revision details]
   M utils/ssh/clientkeys.go
   D utils/sudo.go
   D utils/sudo_test.go

« Back to merge proposal