)]}'
{"manifests/roles/admin.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"a31fb9665d97c5a13b65bdcd0e5302c3d9a7a80b","unresolved":false,"context_lines":[{"line_number":139,"context_line":"    Keystone_user_role[\"${admin}@${admin_tenant}\"] -\u003e File\u003c| tag \u003d\u003d \u0027openrc\u0027 |\u003e"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"    if $target_admin_domain {"},{"line_number":142,"context_line":"      keystone_user_role { \"${admin}@::${target_admin_domain}\":"},{"line_number":143,"context_line":"        ensure      \u003d\u003e present,"},{"line_number":144,"context_line":"        user_domain \u003d\u003e $admin_user_domain,"},{"line_number":145,"context_line":"        roles       \u003d\u003e $admin_roles,"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"7aa08908_cbfc2a0f","line":142,"range":{"start_line":142,"start_character":41,"end_line":142,"end_character":60},"updated":"2016-06-15 10:27:31.000000000","message":"why not user_domain?\n\nwill lines 99-102 be failed, in case when we want exactly default domain (which already created)?","commit_id":"48ca6b0c26c62eb9b155cf1d4c6e67ba1b22d395"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"ca63a83ff09d39a9b332dc108fd37874da532014","unresolved":false,"context_lines":[{"line_number":139,"context_line":"    Keystone_user_role[\"${admin}@${admin_tenant}\"] -\u003e File\u003c| tag \u003d\u003d \u0027openrc\u0027 |\u003e"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"    if $target_admin_domain {"},{"line_number":142,"context_line":"      keystone_user_role { \"${admin}@::${target_admin_domain}\":"},{"line_number":143,"context_line":"        ensure      \u003d\u003e present,"},{"line_number":144,"context_line":"        user_domain \u003d\u003e $admin_user_domain,"},{"line_number":145,"context_line":"        roles       \u003d\u003e $admin_roles,"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"7aa08908_6e6704b8","line":142,"range":{"start_line":142,"start_character":41,"end_line":142,"end_character":60},"in_reply_to":"7aa08908_cbfc2a0f","updated":"2016-06-15 10:53:15.000000000","message":"Hum ... cause I\u0027m not sure we want to unconditionally associate the admin user with admin right on the user domain.  I may be missing the use case here.  Adding a variable ensure that the original behavior is kept.\n\nI don\u0027t get the second point.","commit_id":"48ca6b0c26c62eb9b155cf1d4c6e67ba1b22d395"}]}