)]}'
{"manifests/ssh.pp":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"8e736760ec8e2683534f508adb8a8f7a56896e51","unresolved":false,"context_lines":[{"line_number":16,"context_line":"#"},{"line_number":17,"context_line":"class cgit::ssh ("},{"line_number":18,"context_line":"  $user \u003d \u0027git\u0027,"},{"line_number":19,"context_line":"  $group \u003d \u0027git\u0027,"},{"line_number":20,"context_line":"  $manage_group \u003d true,"},{"line_number":21,"context_line":"  $home \u003d \u0027/var/lib/git\u0027,"},{"line_number":22,"context_line":"  $manage_home \u003d true,"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"ba3cc151_e6c49393","line":19,"updated":"2015-06-29 20:00:09.000000000","message":"Why do the user and group default to git and not cgit?","commit_id":"72fa37dade311dadd83d720f78d7e5ae7a1555f0"},{"author":{"_account_id":6133,"name":"yolanda.robla","email":"yroblamo@redhat.com","username":"yolanda.robla"},"change_message_id":"15b845c656f6473a8f37aefdff6fba659b3d4288","unresolved":false,"context_lines":[{"line_number":16,"context_line":"#"},{"line_number":17,"context_line":"class cgit::ssh ("},{"line_number":18,"context_line":"  $user \u003d \u0027git\u0027,"},{"line_number":19,"context_line":"  $group \u003d \u0027git\u0027,"},{"line_number":20,"context_line":"  $manage_group \u003d true,"},{"line_number":21,"context_line":"  $home \u003d \u0027/var/lib/git\u0027,"},{"line_number":22,"context_line":"  $manage_home \u003d true,"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"ba3cc151_a884785d","line":19,"in_reply_to":"ba3cc151_e6c49393","updated":"2015-07-09 06:21:29.000000000","message":"Well, the idea is to have an independent user to be used to do git clones by ssh. I didn\u0027t want to mess with cgit.\nAnd also, in system-config, you are specially ensuring that cgit authorized_keys is empty, that should be causing conflict if i add it here.","commit_id":"72fa37dade311dadd83d720f78d7e5ae7a1555f0"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"ee935edac6cd96d253efc7c8087e31ec5e353f86","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  $authorized_keys \u003d [],"},{"line_number":27,"context_line":") {"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"  if ($manage_home) and (! defined(File[$home])) {"},{"line_number":30,"context_line":"    file { $home:"},{"line_number":31,"context_line":"      ensure  \u003d\u003e directory,"},{"line_number":32,"context_line":"      owner   \u003d\u003e $user,"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"5ad58531_5c6fa992","line":29,"updated":"2015-05-11 21:37:02.000000000","message":"I would rather avoid using ! defined(File[]) because $manage_home being true should imply that File[$home] is not managed anywhere else. Same with the group below. Not going to -1 since this appears to be a common pattern.","commit_id":"72fa37dade311dadd83d720f78d7e5ae7a1555f0"}]}