)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"2ff0accdc83e5f466b63fdb19d64eea6108a5fa2","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2017-04-17 18:11:30 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Identity trust rbac tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add identity trust rbac tests corresponding to the policy actions"},{"line_number":10,"context_line":"in [0]."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"7ffa3b31_645c257e","line":8,"updated":"2017-04-18 14:51:57.000000000","message":"nit: Maybe also include in here that we\u0027re following OpenStack naming conventions and changing tenant -\u003e project.","commit_id":"fd1db988dc92e07be99d975665e5928ca5090978"}],"patrole_tempest_plugin/tests/api/identity/v3/test_trusts_rbac.py":[{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"ce91128251a9d6fdb07e836287c0b9482ceb987c","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    def skip_checks(cls):"},{"line_number":32,"context_line":"        super(IdentityTrustV3RbacTest, cls).skip_checks()"},{"line_number":33,"context_line":"        if not CONF.identity_feature_enabled.trust:"},{"line_number":34,"context_line":"            raise cls.skipException(\"Trusts aren\u0027t enabled\")"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    @classmethod"},{"line_number":37,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7ffa3b31_93045d7d","line":34,"updated":"2017-04-14 19:22:58.000000000","message":"This is inconsistent with our other error messages.","commit_id":"38255d38a697ef4a2fc09c4ccd446d6fae3e226f"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"f142f407bfba1ee13ef1b1bf1af0ea92c6561098","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    def skip_checks(cls):"},{"line_number":32,"context_line":"        super(IdentityTrustV3RbacTest, cls).skip_checks()"},{"line_number":33,"context_line":"        if not CONF.identity_feature_enabled.trust:"},{"line_number":34,"context_line":"            raise cls.skipException(\"Trusts aren\u0027t enabled\")"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    @classmethod"},{"line_number":37,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7ffa3b31_f62a6fe5","line":34,"in_reply_to":"7ffa3b31_93045d7d","updated":"2017-04-14 20:26:55.000000000","message":"Done","commit_id":"38255d38a697ef4a2fc09c4ccd446d6fae3e226f"},{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"ce91128251a9d6fdb07e836287c0b9482ceb987c","unresolved":false,"context_lines":[{"line_number":43,"context_line":"        # because the user_id in \"user_id:%(trust.trustor_user_id)s\" (the"},{"line_number":44,"context_line":"        # policy rule for creating a trust) corresponds to the primary user_id"},{"line_number":45,"context_line":"        # not the alt user_id."},{"line_number":46,"context_line":"        cls.alt_trustor_user_id \u003d cls.os_alt.auth_provider.credentials.user_id"},{"line_number":47,"context_line":"        cls.trustee_user_id \u003d cls.setup_test_user()[\u0027id\u0027]"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        # A role is guaranteed to exist (namely the admin role), because"}],"source_content_type":"text/x-python","patch_set":6,"id":"7ffa3b31_733e712a","line":46,"range":{"start_line":46,"start_character":16,"end_line":46,"end_character":23},"updated":"2017-04-14 19:22:58.000000000","message":"The use of trustor here is a little confusing to me; it implies that this also has trust permissions. Just \"alt_user\" or perhaps something like \"unauth_user\".","commit_id":"38255d38a697ef4a2fc09c4ccd446d6fae3e226f"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"f142f407bfba1ee13ef1b1bf1af0ea92c6561098","unresolved":false,"context_lines":[{"line_number":43,"context_line":"        # because the user_id in \"user_id:%(trust.trustor_user_id)s\" (the"},{"line_number":44,"context_line":"        # policy rule for creating a trust) corresponds to the primary user_id"},{"line_number":45,"context_line":"        # not the alt user_id."},{"line_number":46,"context_line":"        cls.alt_trustor_user_id \u003d cls.os_alt.auth_provider.credentials.user_id"},{"line_number":47,"context_line":"        cls.trustee_user_id \u003d cls.setup_test_user()[\u0027id\u0027]"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        # A role is guaranteed to exist (namely the admin role), because"}],"source_content_type":"text/x-python","patch_set":6,"id":"7ffa3b31_d62f73f4","line":46,"range":{"start_line":46,"start_character":16,"end_line":46,"end_character":23},"in_reply_to":"7ffa3b31_733e712a","updated":"2017-04-14 20:26:55.000000000","message":"Done","commit_id":"38255d38a697ef4a2fc09c4ccd446d6fae3e226f"}]}