)]}'
{"manifests/mon.pp":[{"author":{"_account_id":13344,"name":"Oleksiy Molchanov","email":"omolchanov@mirantis.com","username":"cvieri.lviv"},"change_message_id":"bce8a37fee7f233d6159262bc0665ec361a8cad6","unresolved":false,"context_lines":[{"line_number":154,"context_line":"        --mkfs \\"},{"line_number":155,"context_line":"        --id ${id} \\"},{"line_number":156,"context_line":"        --keyring ${keyring_path} ; then"},{"line_number":157,"context_line":"    touch \\$mon_data/done \\$mon_data/${init} \\$mon_data/keyring"},{"line_number":158,"context_line":"  else"},{"line_number":159,"context_line":"    rm -fr \\$mon_data"},{"line_number":160,"context_line":"  fi"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"7a5de9d1_231612a8","line":157,"updated":"2016-02-02 15:05:02.000000000","message":"\u003e\u003e\u003e Where?  I\u0027m not seeing where the service is being enabled on boot\n\nHere","commit_id":"5e2078de0bfb90afddbc34c5494fe2aeeff669f3"},{"author":{"_account_id":13344,"name":"Oleksiy Molchanov","email":"omolchanov@mirantis.com","username":"cvieri.lviv"},"change_message_id":"a55fdd01ec8f1d033d5cd7d03f759210aafa86d0","unresolved":false,"context_lines":[{"line_number":171,"context_line":"      -\u003e"},{"line_number":172,"context_line":"      service { $mon_service:"},{"line_number":173,"context_line":"        ensure \u003d\u003e running,"},{"line_number":174,"context_line":"        enable \u003d\u003e true,"},{"line_number":175,"context_line":"      }"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"      if $authentication_type \u003d\u003d \u0027cephx\u0027 {"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"7a5de9d1_97c62b48","line":174,"updated":"2016-02-02 13:09:43.000000000","message":"It won\u0027t work, we have already created files - markers for enabling on boot.","commit_id":"5e2078de0bfb90afddbc34c5494fe2aeeff669f3"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"42dcf7b9fa3b151bb803c5d775d6584d3a1544f6","unresolved":false,"context_lines":[{"line_number":171,"context_line":"      -\u003e"},{"line_number":172,"context_line":"      service { $mon_service:"},{"line_number":173,"context_line":"        ensure \u003d\u003e running,"},{"line_number":174,"context_line":"        enable \u003d\u003e true,"},{"line_number":175,"context_line":"      }"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"      if $authentication_type \u003d\u003d \u0027cephx\u0027 {"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"7a5de9d1_23e132d1","line":174,"in_reply_to":"7a5de9d1_97c62b48","updated":"2016-02-02 15:03:25.000000000","message":"Where?  I\u0027m not seeing where the service is being enabled on boot","commit_id":"5e2078de0bfb90afddbc34c5494fe2aeeff669f3"},{"author":{"_account_id":13344,"name":"Oleksiy Molchanov","email":"omolchanov@mirantis.com","username":"cvieri.lviv"},"change_message_id":"a55fdd01ec8f1d033d5cd7d03f759210aafa86d0","unresolved":false,"context_lines":[{"line_number":191,"context_line":"    } else {"},{"line_number":192,"context_line":"      service { $mon_service:"},{"line_number":193,"context_line":"        ensure \u003d\u003e stopped,"},{"line_number":194,"context_line":"        enable \u003d\u003e false,"},{"line_number":195,"context_line":"      }"},{"line_number":196,"context_line":"      -\u003e"},{"line_number":197,"context_line":"      exec { \"remove-mon-${id}\":"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"7a5de9d1_57a9937a","line":194,"updated":"2016-02-02 13:09:43.000000000","message":"It won\u0027t work, anyway we are executing \"remove-mon-${id}\" below that deletes markers for starting monitors on boot.","commit_id":"5e2078de0bfb90afddbc34c5494fe2aeeff669f3"}],"manifests/rgw.pp":[{"author":{"_account_id":13344,"name":"Oleksiy Molchanov","email":"omolchanov@mirantis.com","username":"cvieri.lviv"},"change_message_id":"a55fdd01ec8f1d033d5cd7d03f759210aafa86d0","unresolved":false,"context_lines":[{"line_number":58,"context_line":"# [*service_enable*]"},{"line_number":59,"context_line":"#  (optional) Controls if the service should auto start on boot."},{"line_number":60,"context_line":"#  Default is true."},{"line_number":61,"context_line":"#"},{"line_number":62,"context_line":"# [*syslog*] Whether or not to log to syslog."},{"line_number":63,"context_line":"#   Optional. Default is true."},{"line_number":64,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"7a5de9d1_fc2f46a5","line":61,"updated":"2016-02-02 13:09:43.000000000","message":"I have already a patch for this \n\nhttps://review.openstack.org/#/c/273048/","commit_id":"5e2078de0bfb90afddbc34c5494fe2aeeff669f3"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"42dcf7b9fa3b151bb803c5d775d6584d3a1544f6","unresolved":false,"context_lines":[{"line_number":58,"context_line":"# [*service_enable*]"},{"line_number":59,"context_line":"#  (optional) Controls if the service should auto start on boot."},{"line_number":60,"context_line":"#  Default is true."},{"line_number":61,"context_line":"#"},{"line_number":62,"context_line":"# [*syslog*] Whether or not to log to syslog."},{"line_number":63,"context_line":"#   Optional. Default is true."},{"line_number":64,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"7a5de9d1_e301da6b","line":61,"in_reply_to":"7a5de9d1_fc2f46a5","updated":"2016-02-02 15:03:25.000000000","message":"Ok i\u0027ll rebase on top of that to get the enable flags","commit_id":"5e2078de0bfb90afddbc34c5494fe2aeeff669f3"}]}