)]}'
{"lib/keystone":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"69cf62f1366d15db845ea6e2f7ee1f8b580c8755","unresolved":false,"context_lines":[{"line_number":334,"context_line":"    local admin_user"},{"line_number":335,"context_line":"    admin_user\u003d$(openstack user show \"admin\" -f value -c id)"},{"line_number":336,"context_line":"    local admin_role\u003d\"admin\""},{"line_number":337,"context_line":"    local member_role\u003d\"member\""},{"line_number":338,"context_line":""},{"line_number":339,"context_line":"    get_or_add_user_domain_role $admin_role $admin_user default"},{"line_number":340,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5f7c97a3_cbff1e32","line":337,"updated":"2018-06-22 02:43:04.000000000","message":"should we add the sanity check like we do for the admin role as well?  follow-on would be fine","commit_id":"a7d0c6fa2c443b2b4b5f4680faff09c6b2bd00d2"},{"author":{"_account_id":11589,"name":"Harry Rybacki","email":"hrybacki@redhat.com","username":"hrybacki"},"change_message_id":"dd0f24dd3b067a0bf16b42d2365e82d46b61ac57","unresolved":false,"context_lines":[{"line_number":335,"context_line":"    admin_user\u003d$(openstack user show \"admin\" -f value -c id)"},{"line_number":336,"context_line":"    local admin_role\u003d\"admin\""},{"line_number":337,"context_line":"    local member_role\u003d\"member\""},{"line_number":338,"context_line":""},{"line_number":339,"context_line":"    get_or_add_user_domain_role $admin_role $admin_user default"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"    # Create service project/role"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5f7c97a3_807080a6","line":338,"updated":"2018-06-18 18:10:00.000000000","message":"Do we want to add `local reader_role\u003d\"reader\"` for future use cases?","commit_id":"a7d0c6fa2c443b2b4b5f4680faff09c6b2bd00d2"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"3d2419e68de1c29fc28be8bf728e2d6bf2b206ed","unresolved":false,"context_lines":[{"line_number":335,"context_line":"    admin_user\u003d$(openstack user show \"admin\" -f value -c id)"},{"line_number":336,"context_line":"    local admin_role\u003d\"admin\""},{"line_number":337,"context_line":"    local member_role\u003d\"member\""},{"line_number":338,"context_line":""},{"line_number":339,"context_line":"    get_or_add_user_domain_role $admin_role $admin_user default"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"    # Create service project/role"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5f7c97a3_b7792dfc","line":338,"in_reply_to":"5f7c97a3_807080a6","updated":"2018-06-19 13:59:41.000000000","message":"I imagine we could, but I\u0027d be inclined to say that we should wait to do that until we know we\u0027re going to use it somewhere. Otherwise it\u0027s just unused code.","commit_id":"a7d0c6fa2c443b2b4b5f4680faff09c6b2bd00d2"},{"author":{"_account_id":11589,"name":"Harry Rybacki","email":"hrybacki@redhat.com","username":"hrybacki"},"change_message_id":"c8ed2b29042571a8602d553412b0e340c1a2b40b","unresolved":false,"context_lines":[{"line_number":335,"context_line":"    admin_user\u003d$(openstack user show \"admin\" -f value -c id)"},{"line_number":336,"context_line":"    local admin_role\u003d\"admin\""},{"line_number":337,"context_line":"    local member_role\u003d\"member\""},{"line_number":338,"context_line":""},{"line_number":339,"context_line":"    get_or_add_user_domain_role $admin_role $admin_user default"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"    # Create service project/role"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5f7c97a3_376f5dfb","line":338,"in_reply_to":"5f7c97a3_b7792dfc","updated":"2018-06-19 14:05:01.000000000","message":"Ack ack, thanks Lance.","commit_id":"a7d0c6fa2c443b2b4b5f4680faff09c6b2bd00d2"}]}