)]}'
{"docker/keystone/Dockerfile.j2":[{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"ef36dc66f647d5bd923ef196129676d7ecc7aab8","unresolved":false,"context_lines":[{"line_number":65,"context_line":"# Plugins"},{"line_number":66,"context_line":"{% import \u0027plugins_helper.j2\u0027 as plugins_helper %}"},{"line_number":67,"context_line":"{{ plugins_helper.add_label(\u0027keystone\u0027) }}"},{"line_number":68,"context_line":"{{ plugins_helper.include_plugins(\u0027keystone\u0027, {{ keystone_plugin }}) }}"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"RUN usermod -a -G kolla keystone \\"},{"line_number":71,"context_line":"    \u0026\u0026 chown -R keystone: /var/www/cgi-bin/keystone \\"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"dab17558_4bcdd985","line":68,"updated":"2016-05-12 09:07:44.000000000","message":"The call to add_label is already in the include_plugins macro and is not needed here.\n\nThe second argument to `include_plugins` must be the enabled_plugins variable:\n{{ plugins_helper.include_plugins(\u0027keystone\u0027, enabled_plugins) }}\n\nThere are ways to not have to pass the variable, but that involve the macro to have access to the whole context and is not less verbose in the end.","commit_id":"7716b7a38ae2f2a9f93e5c2d5252ab6b3e78ddb0"}]}