Code review comment for ~goneri/cloud-init:freebsd_set_password

Revision history for this message
Gonéri Le Bouder (goneri) wrote :

It sounds like you want to jumpback to versin #1. I initially started with a tiny patch with a limited impacted. Follow-up to the iterations of the review process, we end-up with this slightly more invasive change. I personally think it's a better patch since:

- there is now only one way to set the user password
- it resolves the chpasswd issue
- it removes > 40 lines of code and several variables.

« Back to merge proposal