)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"a73ca4d75274411715ddc69df5ab0114d6928314","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds a function to configure the settings for test cases. It"},{"line_number":10,"context_line":"currently sets the needed settings for the first federation scenario"},{"line_number":11,"context_line":"test (follow up patch). If needed, additional settings can be added."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I5f0d0b5eeee1d8f03b38a2eb4cdc2101d3dccaa1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"1a6eadb0_09af0fe4","line":11,"updated":"2016-12-19 16:45:30.000000000","message":"Finding follow on patches in gerrit after they merge can be tough. We can just link to is directly using the Change ID: \n\nI3b393a695c6d9f846efdaf302c1beea34e6bd54b","commit_id":"8193396dd092b7722951a6ccdf647d79815a75a2"}],"devstack/lib/federation.sh":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"a73ca4d75274411715ddc69df5ab0114d6928314","unresolved":false,"context_lines":[{"line_number":16,"context_line":"PROJECT_NAME\u003d${PROJECT_NAME:-federated_project}"},{"line_number":17,"context_line":"GROUP_NAME\u003d${GROUP_NAME:-federated_users}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# TODO(rodrigods): remove/update the settings based at testshib"},{"line_number":20,"context_line":"IDP_ID\u003d${IDP_ID:-testshib}"},{"line_number":21,"context_line":"IDP_USERNAME\u003d${IDP_USERNAME:-myself}"},{"line_number":22,"context_line":"IDP_PASSWORD\u003d${IDP_PASSWORD:-myself}"}],"source_content_type":"text/x-sh","patch_set":5,"id":"1a6eadb0_29da0b79","line":19,"updated":"2016-12-19 16:45:30.000000000","message":"Why do we need to remove or update this? Is it because we\u0027re using the added values below (lines 21 - 29)?","commit_id":"8193396dd092b7722951a6ccdf647d79815a75a2"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"40d31a46ed48403237d6b23264ec6fbbd5535757","unresolved":false,"context_lines":[{"line_number":16,"context_line":"PROJECT_NAME\u003d${PROJECT_NAME:-federated_project}"},{"line_number":17,"context_line":"GROUP_NAME\u003d${GROUP_NAME:-federated_users}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# TODO(rodrigods): remove/update the settings based at testshib"},{"line_number":20,"context_line":"IDP_ID\u003d${IDP_ID:-testshib}"},{"line_number":21,"context_line":"IDP_USERNAME\u003d${IDP_USERNAME:-myself}"},{"line_number":22,"context_line":"IDP_PASSWORD\u003d${IDP_PASSWORD:-myself}"}],"source_content_type":"text/x-sh","patch_set":5,"id":"1a6eadb0_ea9d38bc","line":19,"in_reply_to":"1a6eadb0_0f6d4f32","updated":"2016-12-20 11:57:18.000000000","message":"nice","commit_id":"8193396dd092b7722951a6ccdf647d79815a75a2"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"70a98e81d5420919a85c84c73c677bc57db6f7b6","unresolved":false,"context_lines":[{"line_number":16,"context_line":"PROJECT_NAME\u003d${PROJECT_NAME:-federated_project}"},{"line_number":17,"context_line":"GROUP_NAME\u003d${GROUP_NAME:-federated_users}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# TODO(rodrigods): remove/update the settings based at testshib"},{"line_number":20,"context_line":"IDP_ID\u003d${IDP_ID:-testshib}"},{"line_number":21,"context_line":"IDP_USERNAME\u003d${IDP_USERNAME:-myself}"},{"line_number":22,"context_line":"IDP_PASSWORD\u003d${IDP_PASSWORD:-myself}"}],"source_content_type":"text/x-sh","patch_set":5,"id":"1a6eadb0_8c032d76","line":19,"in_reply_to":"1a6eadb0_29da0b79","updated":"2016-12-19 17:03:01.000000000","message":"it is mainly to not rely on testshib. Ultimately, we want our plugin to deploy both SP and IdP (see [1]) - relying on an external public service can be dangerous, since if the service is down, means our tests are locked.\n\n[1] https://review.openstack.org/#/c/401421/","commit_id":"8193396dd092b7722951a6ccdf647d79815a75a2"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"93b1da9ccf8bae826054c14874713d1e2543bc61","unresolved":false,"context_lines":[{"line_number":16,"context_line":"PROJECT_NAME\u003d${PROJECT_NAME:-federated_project}"},{"line_number":17,"context_line":"GROUP_NAME\u003d${GROUP_NAME:-federated_users}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# TODO(rodrigods): remove/update the settings based at testshib"},{"line_number":20,"context_line":"IDP_ID\u003d${IDP_ID:-testshib}"},{"line_number":21,"context_line":"IDP_USERNAME\u003d${IDP_USERNAME:-myself}"},{"line_number":22,"context_line":"IDP_PASSWORD\u003d${IDP_PASSWORD:-myself}"}],"source_content_type":"text/x-sh","patch_set":5,"id":"1a6eadb0_0f6d4f32","line":19,"in_reply_to":"1a6eadb0_8c032d76","updated":"2016-12-19 17:26:17.000000000","message":"Aha - that makes sense.","commit_id":"8193396dd092b7722951a6ccdf647d79815a75a2"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"40d31a46ed48403237d6b23264ec6fbbd5535757","unresolved":false,"context_lines":[{"line_number":115,"context_line":"    iniset $TEMPEST_CONFIG fed_scenario mapping_group_domain_name $DOMAIN_NAME"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"    # Protocol settings"},{"line_number":118,"context_line":"    iniset $TEMPEST_CONFIG fed_scenario protocol_id $PROTOCOL_ID"},{"line_number":119,"context_line":"}"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"function uninstall_federation {"}],"source_content_type":"text/x-sh","patch_set":5,"id":"1a6eadb0_6a91289a","line":118,"updated":"2016-12-20 11:57:18.000000000","message":"all the configs match what is defined in https://review.openstack.org/#/c/324769/28/keystone_tempest_plugin/config.py","commit_id":"8193396dd092b7722951a6ccdf647d79815a75a2"}]}