)]}'
{"docker/base/set_configs.py":[{"author":{"_account_id":1390,"name":"Paul Bourke","email":"pauldbourke@proton.me","username":"pauldbourke"},"change_message_id":"f527ee33571c1a7b3a852c81266a5d48397eff51","unresolved":false,"context_lines":[{"line_number":338,"context_line":""},{"line_number":339,"context_line":"def handle_ownership(config):"},{"line_number":340,"context_line":"    if \u0027permissions\u0027 in config:"},{"line_number":341,"context_line":"        if \u0027ownership\u0027 in config:"},{"line_number":342,"context_line":"            LOG.warning(\u0027permissions is ignored in favor of ownership\u0027)"},{"line_number":343,"context_line":"        else:"},{"line_number":344,"context_line":"            LOG.info(\u0027permissions is deprecated, please use ownership instead\u0027)"},{"line_number":345,"context_line":"            config[\u0027ownership\u0027] \u003d config[\u0027permissions\u0027]"},{"line_number":346,"context_line":"    for ownership in config.get(\u0027ownership\u0027, list()):"},{"line_number":347,"context_line":"        path \u003d ownership.get(\u0027path\u0027)"},{"line_number":348,"context_line":"        owner \u003d ownership.get(\u0027owner\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_c8f7bbf5","line":345,"range":{"start_line":341,"start_character":8,"end_line":345,"end_character":55},"updated":"2017-06-21 13:04:46.000000000","message":"Not sure this conditional is needed, if \u0027permissions\u0027 is used we should warn regardless. Also could use oslo\u0027s built in deprecation warning feature: https://docs.openstack.org/developer/oslo.log/api/versionutils.html#oslo_log.versionutils.report_deprecated_feature","commit_id":"2cf2408df14c3bff1ee7326f9b47bfc2932487de"}]}