)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"8da0fe835e913fcaf9d4126339291e7d955efae3","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Redstack guest agent fails if user not ubuntu"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added a chown command after useradd to change the ownership"},{"line_number":10,"context_line":"to ${GUEST_USERNAME}"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes_Bug: #1317450"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7adec928_f6096496","line":10,"updated":"2014-05-08 14:57:50.000000000","message":"IMO, you should also mention the group change from \u0027admin\u0027 to \u0027adm\u0027 here","commit_id":"aeb993d296bdee973a54d2d10dcd1daa9522dd35"},{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"a77d790c9f72ec8241251ab28147ee0670d5bb7d","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Also noted that the group being applied to GUEST_USERNAME"},{"line_number":12,"context_line":"was incorrect - should be \u0027adm\u0027"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes_Bug: #1317450"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia670d8b8c9e7f9378a98c9fbc7885e187ef1aaa0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7adec928_14752531","line":14,"updated":"2014-05-19 20:36:51.000000000","message":"Actual bug number appears to be: #1317540","commit_id":"3a2542bb546a741b32d4e67e54523699272480e3"}],"scripts/files/elements/ubuntu-guest/install.d/50-user":[{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"a77d790c9f72ec8241251ab28147ee0670d5bb7d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if ! id -u ${GUEST_USERNAME} \u003e/dev/null 2\u003e\u00261; then"},{"line_number":10,"context_line":"    echo \"Adding ${GUEST_USERNAME} user\""},{"line_number":11,"context_line":"    useradd -G adm -m ${GUEST_USERNAME} -s /bin/bash"},{"line_number":12,"context_line":"    chown ${GUEST_USERNAME}:${GUEST_USERNAME} /home/${GUEST_USERNAME}"},{"line_number":13,"context_line":"    passwd ${GUEST_USERNAME} \u003c\u003c_EOF_"},{"line_number":14,"context_line":"${GUEST_USERNAME}"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7adec928_542fad3e","line":11,"updated":"2014-05-19 20:36:51.000000000","message":"The \"adm\" group only adds access to /var/log files, which I don\u0027t believe we need. We probably want to either keep using \"admin\" here, or replace it with the \"sudo\" group.\n\nSee https://bugs.launchpad.net/ubuntu/+source/policykit-1/+bug/893842 -- the \u0027admin\u0027 group seems to be deprecated in favor of the \u0027sudo\u0027 group.","commit_id":"3a2542bb546a741b32d4e67e54523699272480e3"}]}