)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27078,"name":"Manik Bindlish","email":"manikbindlish19@gmail.com","username":"manik"},"change_message_id":"80b18119f5e9ff877f94752b6d89345e4ccd3f4d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support immutable user source in test_users_negative"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If the keystone user source is immutable, such as an LDAP active directory"},{"line_number":10,"context_line":"implementation, tempest tests that try to create or delete a user will fail."},{"line_number":11,"context_line":"Instead of failing, we would like them to skip. This change uses a"},{"line_number":12,"context_line":"testtools decorator to avoid unnecessary modifications and allow those tests"},{"line_number":13,"context_line":"to skip."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Depends-On: Idc09d6272386f026a899787c2151745916a7228d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_4ea665c1","line":12,"range":{"start_line":9,"start_character":0,"end_line":12,"end_character":76},"updated":"2018-08-02 04:09:06.000000000","message":"It should be as per standard of openstack git commit message.\nLength of line is exceeding than actual required\nPlease refer: https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"05f39219ada617e26f1fbe656819bcdac74cbf02"}],"tempest/api/identity/admin/v3/test_users_negative.py":[{"author":{"_account_id":27078,"name":"Manik Bindlish","email":"manikbindlish19@gmail.com","username":"manik"},"change_message_id":"80b18119f5e9ff877f94752b6d89345e4ccd3f4d","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class UsersNegativeTest(base.BaseIdentityV3AdminTest):"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    @testtools.skipIf(CONF.auth.immutable_user_source,"},{"line_number":30,"context_line":"                      \u0027Environment has an immutable user source and does \u0027"},{"line_number":31,"context_line":"                      \u0027not support creating users.\u0027)"},{"line_number":32,"context_line":"    @decorators.attr(type\u003d[\u0027negative\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_0e8ced36","line":29,"range":{"start_line":29,"start_character":32,"end_line":29,"end_character":54},"updated":"2018-08-02 04:09:06.000000000","message":"I have not found anything like \"immutable_user_source\" in config.py\nAm I understanding it wrong?\nIf it is a new variable, have you defined that in config too?","commit_id":"05f39219ada617e26f1fbe656819bcdac74cbf02"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"2006fe0ea104c7562ceb35b51acbbaf9817d4ecd","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class UsersNegativeTest(base.BaseIdentityV3AdminTest):"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    @testtools.skipIf(CONF.auth.immutable_user_source,"},{"line_number":30,"context_line":"                      \u0027Environment has an immutable user source and does \u0027"},{"line_number":31,"context_line":"                      \u0027not support creating users.\u0027)"},{"line_number":32,"context_line":"    @decorators.attr(type\u003d[\u0027negative\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_ee79712f","line":29,"range":{"start_line":29,"start_character":32,"end_line":29,"end_character":54},"in_reply_to":"5f7c97a3_0e8ced36","updated":"2018-08-02 04:11:58.000000000","message":"It\u0027s implemented in dependent patch set here: https://review.openstack.org/#/c/585536/\n\nWhich is linked to in commit message with depends-on, Zuul does the rest to make the dependency work. That being said it is better to layer the patches than to use depends-on as this isn\u0027t a cross-project dependency.","commit_id":"05f39219ada617e26f1fbe656819bcdac74cbf02"},{"author":{"_account_id":27078,"name":"Manik Bindlish","email":"manikbindlish19@gmail.com","username":"manik"},"change_message_id":"9f636237420a8ac1fc302dce97e190441d5e1b73","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class UsersNegativeTest(base.BaseIdentityV3AdminTest):"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    @testtools.skipIf(CONF.auth.immutable_user_source,"},{"line_number":30,"context_line":"                      \u0027Environment has an immutable user source and does \u0027"},{"line_number":31,"context_line":"                      \u0027not support creating users.\u0027)"},{"line_number":32,"context_line":"    @decorators.attr(type\u003d[\u0027negative\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_4e7e65e2","line":29,"range":{"start_line":29,"start_character":32,"end_line":29,"end_character":54},"in_reply_to":"5f7c97a3_ee79712f","updated":"2018-08-02 04:34:38.000000000","message":"Thanks, I just missed the dependent patch","commit_id":"05f39219ada617e26f1fbe656819bcdac74cbf02"}]}