)]}'
{"manifests/init.pp":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a35d01fab8d6d5947513d9ccfe5c882e0c59ad03","unresolved":false,"context_lines":[{"line_number":391,"context_line":"  }"},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"  cloudkitty_config {"},{"line_number":394,"context_line":"    \u0027fetcher_keystone/auth_section\u0027:     value \u003d\u003e $auth_section;"},{"line_number":395,"context_line":"    \u0027fetcher_keystone/keystone_version\u0027: value \u003d\u003e $keystone_version;"},{"line_number":396,"context_line":"  }"},{"line_number":397,"context_line":""},{"line_number":398,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"9f560f44_054851b1","line":395,"range":{"start_line":394,"start_character":0,"end_line":395,"end_character":68},"updated":"2020-09-29 09:47:28.000000000","message":"Can we make sure that deprecated parameters are removed ? I think this would be helpful for the deployment upgraded from older versions.\n\nYou can implement this by adding lines like the following examples.\n\n \u0027keystone_fetcher/auth_section\u0027: ensure \u003d\u003e absent;","commit_id":"ed3e99bdb51cb709ae32ce9a6c5c63d508e42a8c"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"9ed8748cc20ea22c1e3b35e0638275ff3198df93","unresolved":false,"context_lines":[{"line_number":391,"context_line":"  }"},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"  cloudkitty_config {"},{"line_number":394,"context_line":"    \u0027fetcher_keystone/auth_section\u0027:     value \u003d\u003e $auth_section;"},{"line_number":395,"context_line":"    \u0027fetcher_keystone/keystone_version\u0027: value \u003d\u003e $keystone_version;"},{"line_number":396,"context_line":"  }"},{"line_number":397,"context_line":""},{"line_number":398,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"9f560f44_09518b64","line":395,"range":{"start_line":394,"start_character":0,"end_line":395,"end_character":68},"in_reply_to":"9f560f44_054851b1","updated":"2020-09-29 10:51:09.000000000","message":"I would rather not do that. if one like to ensure the \u0027old\u0027 entries are absent, he can set cludkitty::purge_config to true. \n\nFurther, if we ensure the old values absent, it will prevent people from \u0027manually\u0027 setting the old values in case of a migration (eg, if you upgrade the puppet module but still have old cloudkitty workers).\n\nSince the old values do not hurt at all I think the module is much more flexible by not setting these to absent.\n\nI will not change that !","commit_id":"ed3e99bdb51cb709ae32ce9a6c5c63d508e42a8c"}]}