)]}'
{"deployment/keystone/keystone-container-puppet.yaml":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"abc82fefac98162389a378c9055ccd013588fc9c","unresolved":false,"context_lines":[{"line_number":193,"context_line":"    description: \u003e"},{"line_number":194,"context_line":"      Cron to purge expired or soft-deleted trusts - Ensure"},{"line_number":195,"context_line":"    default: \u0027present\u0027"},{"line_number":196,"context_line":"  KeystoneCronTrustFlushMinute:"},{"line_number":197,"context_line":"    type: comma_delimited_list"},{"line_number":198,"context_line":"    description: \u003e"},{"line_number":199,"context_line":"      Cron to purge expired or soft-deleted trusts - Minute"},{"line_number":200,"context_line":"    default: \u00271\u0027"},{"line_number":201,"context_line":"  KeystoneCronTrustFlushHour:"},{"line_number":202,"context_line":"    type: comma_delimited_list"},{"line_number":203,"context_line":"    description: \u003e"},{"line_number":204,"context_line":"      Cron to purge expired or soft-deleted trusts - Hour"},{"line_number":205,"context_line":"    default: \u0027*\u0027"},{"line_number":206,"context_line":"  KeystoneCronTrustFlushMonthday:"},{"line_number":207,"context_line":"    type: comma_delimited_list"},{"line_number":208,"context_line":"    description: \u003e"},{"line_number":209,"context_line":"      Cron to purge expired or soft-deleted trusts - Month Day"},{"line_number":210,"context_line":"    default: \u0027*\u0027"},{"line_number":211,"context_line":"  KeystoneCronTrustFlushMonth:"},{"line_number":212,"context_line":"    type: comma_delimited_list"},{"line_number":213,"context_line":"    description: \u003e"},{"line_number":214,"context_line":"      Cron to purge expired or soft-deleted trusts - Month"},{"line_number":215,"context_line":"    default: \u0027*\u0027"},{"line_number":216,"context_line":"  KeystoneCronTrustFlushWeekday:"},{"line_number":217,"context_line":"    type: comma_delimited_list"},{"line_number":218,"context_line":"    description: \u003e"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_7cc17f2e","line":215,"range":{"start_line":196,"start_character":0,"end_line":215,"end_character":16},"updated":"2020-07-31 14:58:03.000000000","message":"Instead of requiring the user to set 5 different options what if we just had a:\n\n  KeystoneTrustCronSetting:\n    type: string\n    default: \u00271 * * * * *\u0027\n\nIs it safe to assume cron is well-known enough for us to reduce the amount of parameters we\u0027re using here?","commit_id":"ee9f2a3ef84cedfa23bf211b9dc9559b4c56965a"},{"author":{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},"change_message_id":"27ee4440b9982ec081aa3c5f2d0d375417789e40","unresolved":false,"context_lines":[{"line_number":193,"context_line":"    description: \u003e"},{"line_number":194,"context_line":"      Cron to purge expired or soft-deleted trusts - Ensure"},{"line_number":195,"context_line":"    default: \u0027present\u0027"},{"line_number":196,"context_line":"  KeystoneCronTrustFlushMinute:"},{"line_number":197,"context_line":"    type: comma_delimited_list"},{"line_number":198,"context_line":"    description: \u003e"},{"line_number":199,"context_line":"      Cron to purge expired or soft-deleted trusts - Minute"},{"line_number":200,"context_line":"    default: \u00271\u0027"},{"line_number":201,"context_line":"  KeystoneCronTrustFlushHour:"},{"line_number":202,"context_line":"    type: comma_delimited_list"},{"line_number":203,"context_line":"    description: \u003e"},{"line_number":204,"context_line":"      Cron to purge expired or soft-deleted trusts - Hour"},{"line_number":205,"context_line":"    default: \u0027*\u0027"},{"line_number":206,"context_line":"  KeystoneCronTrustFlushMonthday:"},{"line_number":207,"context_line":"    type: comma_delimited_list"},{"line_number":208,"context_line":"    description: \u003e"},{"line_number":209,"context_line":"      Cron to purge expired or soft-deleted trusts - Month Day"},{"line_number":210,"context_line":"    default: \u0027*\u0027"},{"line_number":211,"context_line":"  KeystoneCronTrustFlushMonth:"},{"line_number":212,"context_line":"    type: comma_delimited_list"},{"line_number":213,"context_line":"    description: \u003e"},{"line_number":214,"context_line":"      Cron to purge expired or soft-deleted trusts - Month"},{"line_number":215,"context_line":"    default: \u0027*\u0027"},{"line_number":216,"context_line":"  KeystoneCronTrustFlushWeekday:"},{"line_number":217,"context_line":"    type: comma_delimited_list"},{"line_number":218,"context_line":"    description: \u003e"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f560f44_e6c31456","line":215,"range":{"start_line":196,"start_character":0,"end_line":215,"end_character":16},"in_reply_to":"9f560f44_7cc17f2e","updated":"2020-08-04 16:18:02.000000000","message":"The variables for each time slot is how Puppet\u0027s \u0027cron\u0027 resource handles it (and Ansible\u0027s \u0027cron\u0027 module, for that matter). Without having to write a compatibility layer to translate from a full string to individual variables we will probably want to stick with this.","commit_id":"ee9f2a3ef84cedfa23bf211b9dc9559b4c56965a"}]}