)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"871c01df027d2a5c8f814f4d9531e25a190cb75e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6f4f59aa_44851f23","updated":"2022-08-29 03:32:11.000000000","message":"lgtm","commit_id":"d503ee5fc93c0b8303f8cb4de92d3b1f50c38237"}],"deployment/heat/heat-api-container-puppet.yaml":[{"author":{"_account_id":20733,"name":"Rajesh Tailor","email":"ratailor@redhat.com","username":"rajesht"},"change_message_id":"19d199952c494027d37da7a018148935c1b74300","unresolved":true,"context_lines":[{"line_number":69,"context_line":"  MonitoringSubscriptionHeatApi:"},{"line_number":70,"context_line":"    default: \u0027overcloud-heat-api\u0027"},{"line_number":71,"context_line":"    type: string"},{"line_number":72,"context_line":"  HeatApiPolicies:"},{"line_number":73,"context_line":"    description: |"},{"line_number":74,"context_line":"      A hash of policies to configure for Heat API."},{"line_number":75,"context_line":"      e.g. { heat-context_is_admin: { key: context_is_admin, value: \u0027role:admin\u0027 } }"},{"line_number":76,"context_line":"    default: {}"},{"line_number":77,"context_line":"    type: json"},{"line_number":78,"context_line":"  HeatStackDomainAdminPassword:"},{"line_number":79,"context_line":"    description: Password for heat_stack_domain_admin user."},{"line_number":80,"context_line":"    type: string"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"638d63e8_5d901e81","line":77,"range":{"start_line":72,"start_character":0,"end_line":77,"end_character":14},"updated":"2022-08-02 11:22:49.000000000","message":"IMO we should remove parameter declaration from here.","commit_id":"d503ee5fc93c0b8303f8cb4de92d3b1f50c38237"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"49bd5c003a33ff4ccf9c0a7235eb6f2eac160d58","unresolved":true,"context_lines":[{"line_number":69,"context_line":"  MonitoringSubscriptionHeatApi:"},{"line_number":70,"context_line":"    default: \u0027overcloud-heat-api\u0027"},{"line_number":71,"context_line":"    type: string"},{"line_number":72,"context_line":"  HeatApiPolicies:"},{"line_number":73,"context_line":"    description: |"},{"line_number":74,"context_line":"      A hash of policies to configure for Heat API."},{"line_number":75,"context_line":"      e.g. { heat-context_is_admin: { key: context_is_admin, value: \u0027role:admin\u0027 } }"},{"line_number":76,"context_line":"    default: {}"},{"line_number":77,"context_line":"    type: json"},{"line_number":78,"context_line":"  HeatStackDomainAdminPassword:"},{"line_number":79,"context_line":"    description: Password for heat_stack_domain_admin user."},{"line_number":80,"context_line":"    type: string"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6b5d69d8_5cd02327","line":77,"range":{"start_line":72,"start_character":0,"end_line":77,"end_character":14},"in_reply_to":"638d63e8_5d901e81","updated":"2022-08-02 11:50:02.000000000","message":"We can\u0027t remove this because it is used in service_config_settings .\n\nAlternatively we can move that definition to the base class and let it included by all of the services by resource reference. However that looks quite redundant and is not really useful unless we\u0027d expect users would split deployment of individual heat services to multiple stacks. So I\u0027m just keeping the service_config_settings definition in api atm.","commit_id":"d503ee5fc93c0b8303f8cb4de92d3b1f50c38237"}]}