)]}'
{"manifests/compute.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"1c25226e80e05be2e285906d781b6ce2c7e6ae76","unresolved":false,"context_lines":[{"line_number":229,"context_line":"  $instance_usage_audit_period                 \u003d \u0027month\u0027,"},{"line_number":230,"context_line":"  $force_raw_images                            \u003d true,"},{"line_number":231,"context_line":"  $reserved_host_memory                        \u003d \u0027512\u0027,"},{"line_number":232,"context_line":"  $reserved_host_disk                          \u003d \u00270\u0027,"},{"line_number":233,"context_line":"  $heal_instance_info_cache_interval           \u003d \u002760\u0027,"},{"line_number":234,"context_line":"  $config_drive_format                         \u003d $::os_service_default,"},{"line_number":235,"context_line":"  $resize_confirm_window                       \u003d $::os_service_default,"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"9f560f44_cd456d71","line":232,"updated":"2020-09-21 16:52:08.000000000","message":"We should use $::os_service_default rather than specifying the same default as in nova. I\u0027m also torn on if it his should be _disk or _disk_mb. I guess if we\u0027re going on the existing patern with reserved_host_memory, this name is fine. It would be better to use $::os_service_default which will remove it from the config and result in the 0 value by default.","commit_id":"f827ea9785f5beb185176f1cbc6682e7b70e2c9d"}]}