)]}'
{"manifests/api.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"fc5965ea5c69c63f7688d164c8f101025ba53ca9","unresolved":false,"context_lines":[{"line_number":81,"context_line":"  $enable_api_admin   \u003d false,"},{"line_number":82,"context_line":"  $api_base_uri       \u003d $::os_service_default,"},{"line_number":83,"context_line":"  $listen             \u003d $::os_service_default,"},{"line_number":84,"context_line":"  $workers            \u003d undef,"},{"line_number":85,"context_line":"  $threads            \u003d 1000,"},{"line_number":86,"context_line":"  $enable_host_header \u003d false,"},{"line_number":87,"context_line":"  $max_header_line    \u003d 16384,"}],"source_content_type":"text/x-puppet","patch_set":9,"id":"ba5201f7_d69c50c2","line":84,"range":{"start_line":84,"start_character":24,"end_line":84,"end_character":29},"updated":"2017-01-05 15:03:19.000000000","message":"if you\u0027re not going to provide a value, this should be $::os_service_default. We also have a default for workers $::os_workers that could be used if you wanted an actual default configuration value.  Given that you have a default value for $threads you probably just want to use $::os_service_default","commit_id":"ee9e0cf470912e6019d47ab54def2f33e41e1b07"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"fc5965ea5c69c63f7688d164c8f101025ba53ca9","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    \u0027service:api/max_header_line\u0027           : value \u003d\u003e $max_header_line;"},{"line_number":113,"context_line":"  }"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"  if $workers and is_integer($workers) {"},{"line_number":116,"context_line":"    designate_config {"},{"line_number":117,"context_line":"      \u0027service:api/workers\u0027                 : value \u003d\u003e $workers;"},{"line_number":118,"context_line":"    }"},{"line_number":119,"context_line":"  } else {"},{"line_number":120,"context_line":"    warning(\u0027Invalid value for workers parameter, it should be an integer.\u0027)"},{"line_number":121,"context_line":"  }"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"  if $auth_strategy \u003d\u003d \u0027keystone\u0027 {"},{"line_number":124,"context_line":"    include ::designate::keystone::authtoken"}],"source_content_type":"text/x-puppet","patch_set":9,"id":"ba5201f7_569860c0","line":121,"range":{"start_line":115,"start_character":1,"end_line":121,"end_character":3},"updated":"2017-01-05 15:03:19.000000000","message":"is_integer is actually deprecated so we don\u0027t want to do this. Additionally if you switch to $::os_service_default you can just include this setting in the above config block","commit_id":"ee9e0cf470912e6019d47ab54def2f33e41e1b07"}]}