)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"83f60d817ad6d6ff3d10aa21274c26741412e76b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Leo Henken \u003clh236s@att.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-07-26 15:26:09 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Skip tests in UsersV3TestJSON with an immutable user source "},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds a skip_checks condition to UsersV3TestJSON to skip if immutable "},{"line_number":10,"context_line":"user source is true. resource_setup() in this class attempts to create"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7faddb67_f22a312a","line":7,"updated":"2019-07-29 05:22:53.000000000","message":"nit: This contains a trailing space.","commit_id":"0f348f79c8e8470b9607bccbdf3620dda594f1b2"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"83f60d817ad6d6ff3d10aa21274c26741412e76b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Skip tests in UsersV3TestJSON with an immutable user source "},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds a skip_checks condition to UsersV3TestJSON to skip if immutable "},{"line_number":10,"context_line":"user source is true. resource_setup() in this class attempts to create"},{"line_number":11,"context_line":"users, which will fail if the environment uses an LDAP directory for "},{"line_number":12,"context_line":"example."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7faddb67_92217d50","line":9,"updated":"2019-07-29 05:22:53.000000000","message":"ditto","commit_id":"0f348f79c8e8470b9607bccbdf3620dda594f1b2"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"83f60d817ad6d6ff3d10aa21274c26741412e76b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds a skip_checks condition to UsersV3TestJSON to skip if immutable "},{"line_number":10,"context_line":"user source is true. resource_setup() in this class attempts to create"},{"line_number":11,"context_line":"users, which will fail if the environment uses an LDAP directory for "},{"line_number":12,"context_line":"example."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I42c3b7a17bfb73079b1d05951636359324bed207"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7faddb67_b21cb913","line":11,"updated":"2019-07-29 05:22:53.000000000","message":"ditto","commit_id":"0f348f79c8e8470b9607bccbdf3620dda594f1b2"}],"tempest/api/identity/admin/v3/test_list_users.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a55947cb49ca796f3531c66365be75a5256e1d9c","unresolved":false,"context_lines":[{"line_number":34,"context_line":"                         map(lambda x: x[key], body))"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    @classmethod"},{"line_number":37,"context_line":"    def skip_checks(cls):"},{"line_number":38,"context_line":"        super(UsersV3TestJSON, cls).skip_checks()"},{"line_number":39,"context_line":"        if CONF.identity_feature_enabled.immutable_user_source:"},{"line_number":40,"context_line":"            raise cls.skipException(\u0027Skipped because environment has an \u0027"},{"line_number":41,"context_line":"                                    \u0027immutable user source and solely \u0027"},{"line_number":42,"context_line":"                                    \u0027provides read-only access to users.\u0027)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    @classmethod"},{"line_number":45,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_c6b17e91","line":42,"range":{"start_line":37,"start_character":0,"end_line":42,"end_character":74},"updated":"2019-07-27 11:47:39.000000000","message":"I was thinking on this that this is read-only tests test_list_users and should not be skipped if your user resource is immutable. \n\nBut we cannot do strong verification of list users until we have some users in system.","commit_id":"0f348f79c8e8470b9607bccbdf3620dda594f1b2"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"83f60d817ad6d6ff3d10aa21274c26741412e76b","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        if CONF.identity_feature_enabled.immutable_user_source:"},{"line_number":40,"context_line":"            raise cls.skipException(\u0027Skipped because environment has an \u0027"},{"line_number":41,"context_line":"                                    \u0027immutable user source and solely \u0027"},{"line_number":42,"context_line":"                                    \u0027provides read-only access to users.\u0027)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    @classmethod"},{"line_number":45,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_521b051a","line":42,"updated":"2019-07-29 05:22:53.000000000","message":"Yeah, agree. We can probably test that with adding configs for test users. But I think it makes tempest more complicated and brings few benefit..","commit_id":"0f348f79c8e8470b9607bccbdf3620dda594f1b2"}]}