Code review comment for lp:~thumper/juju-core/ubuntu-profile-upgrade-step

Revision history for this message
Tim Penhey (thumper) wrote :

Reviewers: mp+212083_code.launchpad.net,

Message:
Please take a look.

Description:
Add upgrade job for ~ubuntu/.profile

The ubuntu user's .profile is updated to source the
~ubuntu/.juju-profile to get the proxy information.

The .juju-profile is written out by the machine agent
as it starts up if it is missing, but the .profile
needs to happen for any machines that were created by
older agents.

https://code.launchpad.net/~thumper/juju-core/ubuntu-profile-upgrade-step/+merge/212083

(do not edit description out of merge proposal)

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

Affected files (+134, -7 lines):
   A [revision details]
   A upgrades/dotprofile.go
   A upgrades/dotprofile_test.go
   M upgrades/export_test.go
   M upgrades/steps118.go

« Back to merge proposal