)]}'
{"manifests/profile/pacemaker/ceilometer/api.pp":[{"author":{"_account_id":8042,"name":"Jiří Stránský","email":"jistr@redhat.com","username":"jistr"},"change_message_id":"256021f9036da9c7a5438d297c6f7fb202f9a8a3","unresolved":false,"context_lines":[{"line_number":39,"context_line":"  if $step \u003e\u003d 5 and downcase($::hostname) \u003d\u003d $pacemaker_master {"},{"line_number":40,"context_line":"    class { \u0027::tripleo::profile::base::ceilometer::api\u0027:"},{"line_number":41,"context_line":"      step    \u003d\u003e $step,"},{"line_number":42,"context_line":"    }"},{"line_number":43,"context_line":"  }"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":6,"id":"1ac06dbe_b20ed038","line":42,"updated":"2016-08-23 12:48:58.000000000","message":"We probably don\u0027t need to pass the step explicitly because it should default to the same hiera query as here.\n\nThis might not work well, because httpd already starts up at step 5, so we should probably write the WSGI configs on all nodes earlier (step 4?) to prevent race between config writing vs. service start on non-pacemaker nodes.\n\nAlso i wonder if we still need to keep maintaining the pacemaker manifests for OpenStack services now that we stopped using them everywhere AFAIK. This is probably a more general question for the whole tripleo group than a specific one to this review.","commit_id":"8b5fbc0664cba5f5ac70ff56cb7bdadc32928843"},{"author":{"_account_id":6924,"name":"Pradeep Kilambi","email":"pkilambi@redhat.com","username":"pkilambi"},"change_message_id":"269917b5092b6edbce919af9b394fadc793ab320","unresolved":false,"context_lines":[{"line_number":39,"context_line":"  if $step \u003e\u003d 5 and downcase($::hostname) \u003d\u003d $pacemaker_master {"},{"line_number":40,"context_line":"    class { \u0027::tripleo::profile::base::ceilometer::api\u0027:"},{"line_number":41,"context_line":"      step    \u003d\u003e $step,"},{"line_number":42,"context_line":"    }"},{"line_number":43,"context_line":"  }"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":6,"id":"1ac06dbe_7d0b892a","line":42,"in_reply_to":"1ac06dbe_b20ed038","updated":"2016-08-23 13:03:15.000000000","message":"Yea i dont think this will stay here long, with systemd kicking in, i assume this stuff will probably be removed eventually. I just added this here as we still maintain pcmk profiles for now.","commit_id":"8b5fbc0664cba5f5ac70ff56cb7bdadc32928843"}]}