)]}'
{"manifests/mon.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"1832317c6e3c2d548a5cd3d5d1d843e1b981c08f","unresolved":false,"context_lines":[{"line_number":89,"context_line":"      $init \u003d \u0027upstart\u0027"},{"line_number":90,"context_line":"      Service {"},{"line_number":91,"context_line":"        name     \u003d\u003e \"ceph-mon-${id}\","},{"line_number":92,"context_line":"        provider \u003d\u003e $::ceph::params::service_provider,"},{"line_number":93,"context_line":"        start    \u003d\u003e \"start ceph-mon id\u003d${id}\","},{"line_number":94,"context_line":"        stop     \u003d\u003e \"stop ceph-mon id\u003d${id}\","},{"line_number":95,"context_line":"        status   \u003d\u003e \"status ceph-mon id\u003d${id}\","}],"source_content_type":"text/x-puppet","patch_set":1,"id":"7f287b81_ab4ff768","side":"PARENT","line":92,"updated":"2017-08-31 05:06:27.000000000","message":"it sounds like we have the same thing in rgw class. Could you please look at it as well, so our code remains consistent and we can also remove this variable from params.pp.\n\nThanks","commit_id":"a564389aeb8a47f5cc5f3a155d30117d42dd1b11"},{"author":{"_account_id":1004,"name":"Mohammed Naser","email":"mnaser@vexxhost.com","username":"mnaser"},"change_message_id":"53cfbd7f09d2d1d17531ac08d44ee8e60cb9b416","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    if $::service_provider \u003d\u003d \u0027upstart\u0027 {"},{"line_number":89,"context_line":"      $init \u003d \u0027upstart\u0027"},{"line_number":90,"context_line":"      Service {"},{"line_number":91,"context_line":"        name     \u003d\u003e \"ceph-mon\","},{"line_number":92,"context_line":"        provider \u003d\u003e $::service_provider,"},{"line_number":93,"context_line":"        start    \u003d\u003e \"start ceph-mon id\u003d${id}\","},{"line_number":94,"context_line":"        stop     \u003d\u003e \"stop ceph-mon id\u003d${id}\","}],"source_content_type":"text/x-puppet","patch_set":3,"id":"7f287b81_68a2064e","line":91,"range":{"start_line":91,"start_character":20,"end_line":91,"end_character":30},"updated":"2017-09-01 18:32:55.000000000","message":"This needs to be single quoted to pass Puppet lint checks :)","commit_id":"67d68ebbff07b3e2967ea9fcc979294a0304b1e8"},{"author":{"_account_id":1004,"name":"Mohammed Naser","email":"mnaser@vexxhost.com","username":"mnaser"},"change_message_id":"53cfbd7f09d2d1d17531ac08d44ee8e60cb9b416","unresolved":false,"context_lines":[{"line_number":100,"context_line":"      $init \u003d \u0027systemd\u0027"},{"line_number":101,"context_line":"      Service {"},{"line_number":102,"context_line":"        enable \u003d\u003e $mon_enable,"},{"line_number":103,"context_line":"        name   \u003d\u003e \"ceph-mon@${id}\","},{"line_number":104,"context_line":"      }"},{"line_number":105,"context_line":"    }"},{"line_number":106,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":3,"id":"7f287b81_c8bc5a6e","line":103,"updated":"2017-09-01 18:32:55.000000000","message":"Could you switch this back up to where it was just to avoid the extra line change that doesn\u0027t really affect what happens?","commit_id":"67d68ebbff07b3e2967ea9fcc979294a0304b1e8"},{"author":{"_account_id":1004,"name":"Mohammed Naser","email":"mnaser@vexxhost.com","username":"mnaser"},"change_message_id":"53cfbd7f09d2d1d17531ac08d44ee8e60cb9b416","unresolved":false,"context_lines":[{"line_number":226,"context_line":""},{"line_number":227,"context_line":"    } elsif $ensure \u003d\u003d absent {"},{"line_number":228,"context_line":"      service { $mon_service:"},{"line_number":229,"context_line":"        ensure \u003d\u003e stopped,"},{"line_number":230,"context_line":"        name   \u003d\u003e $service_name,"},{"line_number":231,"context_line":"      }"},{"line_number":232,"context_line":"      -\u003e exec { \"remove-mon-${id}\":"},{"line_number":233,"context_line":"        command   \u003d\u003e \"/bin/true # comment to satisfy puppet syntax requirements"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"7f287b81_726554e1","line":230,"range":{"start_line":229,"start_character":0,"end_line":230,"end_character":32},"updated":"2017-09-01 18:32:55.000000000","message":"Looks like this $service_name variable comes from no where.  Also, I wonder how this ever worked before, because with `systemd` the service resource doesn\u0027t even match the service name.","commit_id":"67d68ebbff07b3e2967ea9fcc979294a0304b1e8"}],"manifests/rgw.pp":[{"author":{"_account_id":1004,"name":"Mohammed Naser","email":"mnaser@vexxhost.com","username":"mnaser"},"change_message_id":"53cfbd7f09d2d1d17531ac08d44ee8e60cb9b416","unresolved":false,"context_lines":[{"line_number":178,"context_line":"    }"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"    Service {"},{"line_number":181,"context_line":"      name     \u003d\u003e \"radosgw\","},{"line_number":182,"context_line":"      start    \u003d\u003e \"start radosgw id\u003d${name}\","},{"line_number":183,"context_line":"      stop     \u003d\u003e \"stop radosgw id\u003d${name}\","},{"line_number":184,"context_line":"      status   \u003d\u003e \"status radosgw id\u003d${name}\","}],"source_content_type":"text/x-puppet","patch_set":3,"id":"7f287b81_2872bee4","line":181,"range":{"start_line":181,"start_character":18,"end_line":181,"end_character":27},"updated":"2017-09-01 18:32:55.000000000","message":"Double quotes here again! :)","commit_id":"67d68ebbff07b3e2967ea9fcc979294a0304b1e8"}]}