)]}'
{"manifests/worker.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"08e59744d1941fc14b1353dc7131e853771e022e","unresolved":false,"context_lines":[{"line_number":12,"context_line":"#   Whether Puppet should manage the package. Default is true."},{"line_number":13,"context_line":"#"},{"line_number":14,"context_line":"# [*worker_package_name*]"},{"line_number":15,"context_line":"#  (optional) Name of the package containing worker "},{"line_number":16,"context_line":"#  resources. Defaults to worker_package_name from"},{"line_number":17,"context_line":"#  designate::params"},{"line_number":18,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"1f485f77_e5cea8b2","line":15,"updated":"2017-11-15 21:58:20.000000000","message":"white space","commit_id":"52fdf8b416f839142aaec81231abe564fb9d7c81"},{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"9dfc43c2611121f8b61cbf3f9026884ce7eb7d4f","unresolved":false,"context_lines":[{"line_number":12,"context_line":"#   Whether Puppet should manage the package. Default is true."},{"line_number":13,"context_line":"#"},{"line_number":14,"context_line":"# [*worker_package_name*]"},{"line_number":15,"context_line":"#  (optional) Name of the package containing worker "},{"line_number":16,"context_line":"#  resources. Defaults to worker_package_name from"},{"line_number":17,"context_line":"#  designate::params"},{"line_number":18,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"1f485f77_279ddb09","line":15,"in_reply_to":"1f485f77_e5cea8b2","updated":"2017-11-16 07:44:22.000000000","message":"Done","commit_id":"52fdf8b416f839142aaec81231abe564fb9d7c81"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"08e59744d1941fc14b1353dc7131e853771e022e","unresolved":false,"context_lines":[{"line_number":71,"context_line":"  $worker_package_name  \u003d undef,"},{"line_number":72,"context_line":"  $enabled              \u003d true,"},{"line_number":73,"context_line":"  $service_ensure       \u003d \u0027running\u0027,"},{"line_number":74,"context_line":"  $workers              \u003d $::os_service_default,"},{"line_number":75,"context_line":"  $threads              \u003d $::os_service_default,"},{"line_number":76,"context_line":"  $threshold_percentage \u003d $::os_service_default,"},{"line_number":77,"context_line":"  $poll_timeout         \u003d $::os_service_default,"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"1f485f77_c5c72c96","line":74,"updated":"2017-11-15 21:58:20.000000000","message":"should we use ::os_workers ?","commit_id":"52fdf8b416f839142aaec81231abe564fb9d7c81"},{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"9dfc43c2611121f8b61cbf3f9026884ce7eb7d4f","unresolved":false,"context_lines":[{"line_number":71,"context_line":"  $worker_package_name  \u003d undef,"},{"line_number":72,"context_line":"  $enabled              \u003d true,"},{"line_number":73,"context_line":"  $service_ensure       \u003d \u0027running\u0027,"},{"line_number":74,"context_line":"  $workers              \u003d $::os_service_default,"},{"line_number":75,"context_line":"  $threads              \u003d $::os_service_default,"},{"line_number":76,"context_line":"  $threshold_percentage \u003d $::os_service_default,"},{"line_number":77,"context_line":"  $poll_timeout         \u003d $::os_service_default,"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"1f485f77_c78727b5","line":74,"in_reply_to":"1f485f77_c5c72c96","updated":"2017-11-16 07:44:22.000000000","message":"Same as the other patch, I can fix them all in one go later on.","commit_id":"52fdf8b416f839142aaec81231abe564fb9d7c81"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"3a84f4d7509a20432b9f7ab8ce5e03825fed9bd1","unresolved":false,"context_lines":[{"line_number":71,"context_line":"  $worker_package_name  \u003d undef,"},{"line_number":72,"context_line":"  $enabled              \u003d true,"},{"line_number":73,"context_line":"  $service_ensure       \u003d \u0027running\u0027,"},{"line_number":74,"context_line":"  $workers              \u003d $::os_service_default,"},{"line_number":75,"context_line":"  $threads              \u003d $::os_service_default,"},{"line_number":76,"context_line":"  $threshold_percentage \u003d $::os_service_default,"},{"line_number":77,"context_line":"  $poll_timeout         \u003d $::os_service_default,"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"ff82abbf_e913025b","line":74,"updated":"2017-11-27 16:44:03.000000000","message":"so usually for workers we use $::os_workers.  We probably want to follow up with the correct setting for threads/workers depending on what is optimal for designate","commit_id":"8120a19e9d15516f0cd23d1e77b19a7053733f67"}]}