)]}'
{"manifests/rgw/apache24.pp":[{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"52a08938f6c4b1b61fc6555b08e7f205883062c7","unresolved":false,"context_lines":[{"line_number":153,"context_line":"    mode   \u003d\u003e \u00270775\u0027,"},{"line_number":154,"context_line":"  }"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"  ceph::rgw{ \u0027radosgw.gateway\u0027:"},{"line_number":157,"context_line":"    pkg_radosgw        \u003d\u003e $pkg_radosgw,"},{"line_number":158,"context_line":"    user               \u003d\u003e $user,"},{"line_number":159,"context_line":"    rgw_dns_name       \u003d\u003e $vhost,"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"9a8ffd7b_18bc00ba","line":156,"updated":"2015-12-02 23:50:00.000000000","message":"I\u0027m not sure about this dependency, it might be cleaner to just leave it to the user to define both ceph::rgw and ceph::rgw::apache24 and pass it the correct parameters","commit_id":"5c2d3bf8e52ae41f8417fe82c8e4e6fe5da5f741"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"52a08938f6c4b1b61fc6555b08e7f205883062c7","unresolved":false,"context_lines":[{"line_number":162,"context_line":"    require            \u003d\u003e File[\u0027/var/run/ceph\u0027],"},{"line_number":163,"context_line":"  }"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"  package { \u0027radosgw-agent\u0027:"},{"line_number":166,"context_line":"    ensure  \u003d\u003e installed,"},{"line_number":167,"context_line":"  }"},{"line_number":168,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":3,"id":"9a8ffd7b_f8a1fca9","line":165,"updated":"2015-12-02 23:50:00.000000000","message":"I think this would fit better within ceph::rgw","commit_id":"5c2d3bf8e52ae41f8417fe82c8e4e6fe5da5f741"}]}