)]}'
{"deployment/glance/glance-api-container-puppet.yaml":[{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"c5d5cfe52a4dbc29e54ba14a3fc07efabe57d58d","unresolved":true,"context_lines":[{"line_number":309,"context_line":"  ContainerGlanceApiConfigImage:"},{"line_number":310,"context_line":"    description: The container image to use for the glance_api config_volume"},{"line_number":311,"context_line":"    type: string"},{"line_number":312,"context_line":"  GlanceCronDbPurgeMinute:"},{"line_number":313,"context_line":"    type: string"},{"line_number":314,"context_line":"    description: \u003e"},{"line_number":315,"context_line":"        Cron to purge db entries marked as deleted and older than $age - Minute"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2213d7f4_906350c3","line":312,"range":{"start_line":312,"start_character":2,"end_line":312,"end_character":25},"updated":"2021-06-18 03:56:41.000000000","message":"Though this is  a backport, does having these per service add much value other than making the templates big? We should probably understand the other implications of adding new interfaces in the future.","commit_id":"733f3ecbe4e4faa6d9d2c6fa613c82d36b69ecd2"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c1970143d955e8ce64df619871b338d16ae54fd6","unresolved":true,"context_lines":[{"line_number":309,"context_line":"  ContainerGlanceApiConfigImage:"},{"line_number":310,"context_line":"    description: The container image to use for the glance_api config_volume"},{"line_number":311,"context_line":"    type: string"},{"line_number":312,"context_line":"  GlanceCronDbPurgeMinute:"},{"line_number":313,"context_line":"    type: string"},{"line_number":314,"context_line":"    description: \u003e"},{"line_number":315,"context_line":"        Cron to purge db entries marked as deleted and older than $age - Minute"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"539141de_8f6f9195","line":312,"range":{"start_line":312,"start_character":2,"end_line":312,"end_character":25},"in_reply_to":"2213d7f4_906350c3","updated":"2021-06-18 04:02:48.000000000","message":"This follows the existing parameter interfaces we have for cron configuration but I agree that it\u0027s too big considering its value. I feel like we should drop these parameters and just let users use puppet parameters assuming it\u0027s not very common to customize these configuration. (For example Destination parameter controls name of the log file which is not likely to be changed.)","commit_id":"733f3ecbe4e4faa6d9d2c6fa613c82d36b69ecd2"}]}