)]}'
{"etc/tempest.conf.sample":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"e5229fd27739fa2286785c8bd714ff3ed280bddb","unresolved":false,"context_lines":[{"line_number":379,"context_line":"# test-name.  If a match occurs, the test is skipped.  You may repeat"},{"line_number":380,"context_line":"# as many skip_list entries as you like. e.g."},{"line_number":381,"context_line":"#  - skipping a single test"},{"line_number":382,"context_line":"# skip_list \u003d tempest.api.compute.admin.test_aggregates.AggregatesAdminTestJSON.test_aggregate_get_details_with_invalid_id"},{"line_number":383,"context_line":"#  - skip all test_aggreates tests"},{"line_number":384,"context_line":"# skip_list \u003d tempest.api.compute.admin.test_aggregates"},{"line_number":385,"context_line":"#  - skip all api tests"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAQn%2F%2FbBw%3D","line":382,"updated":"2013-07-31 13:10:42.000000000","message":"For me \"skip_list\" is a bit misleading since all entries together is one list.","commit_id":"b06fdc440a2bc2eb7cc2d47a9de59e0105219137"}],"tempest/config.py":[{"author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"change_message_id":"e0409b110977e6f867f899578ebf41acbc892caa","unresolved":false,"context_lines":[{"line_number":588,"context_line":"        conf.register_opt(opt, group\u003d\u0027service_available\u0027)"},{"line_number":589,"context_line":""},{"line_number":590,"context_line":"skip_group \u003d cfg.OptGroup(name\u003d\"skip\","},{"line_number":591,"context_line":"                          title\u003d\"Classes to skip\")"},{"line_number":592,"context_line":""},{"line_number":593,"context_line":"SkipGroup \u003d ["},{"line_number":594,"context_line":"    cfg.MultiStrOpt(\u0027skip_list\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FcIU%3D","line":591,"updated":"2013-07-31 09:58:08.000000000","message":"minor: isn\u0027t it misleading to say \u0027Classes\u0027 to skip while it could be used to skip test methods as well.","commit_id":"b06fdc440a2bc2eb7cc2d47a9de59e0105219137"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e95fa4d43567a4ab6fd342d5a5eae8325cb531f5","unresolved":false,"context_lines":[{"line_number":588,"context_line":"        conf.register_opt(opt, group\u003d\u0027service_available\u0027)"},{"line_number":589,"context_line":""},{"line_number":590,"context_line":"skip_group \u003d cfg.OptGroup(name\u003d\"skip\","},{"line_number":591,"context_line":"                          title\u003d\"Classes to skip\")"},{"line_number":592,"context_line":""},{"line_number":593,"context_line":"SkipGroup \u003d ["},{"line_number":594,"context_line":"    cfg.MultiStrOpt(\u0027skip_list\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FYug%3D","line":591,"in_reply_to":"AAAAQn%2F%2FcIU%3D","updated":"2013-07-31 23:59:07.000000000","message":"Yes, I\u0027ll correct that","commit_id":"b06fdc440a2bc2eb7cc2d47a9de59e0105219137"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"e5229fd27739fa2286785c8bd714ff3ed280bddb","unresolved":false,"context_lines":[{"line_number":591,"context_line":"                          title\u003d\"Classes to skip\")"},{"line_number":592,"context_line":""},{"line_number":593,"context_line":"SkipGroup \u003d ["},{"line_number":594,"context_line":"    cfg.MultiStrOpt(\u0027skip_list\u0027,"},{"line_number":595,"context_line":"                    default\u003dNone,"},{"line_number":596,"context_line":"                    help\u003d\"List of classes to skip\")"},{"line_number":597,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2Fa%2Fo%3D","line":594,"updated":"2013-07-31 13:10:42.000000000","message":"For me skip_list sounds like a ListOpt. But a MultiStrOpt makes sense, too. Maybe we could add both options and bring them together to one list?","commit_id":"b06fdc440a2bc2eb7cc2d47a9de59e0105219137"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e95fa4d43567a4ab6fd342d5a5eae8325cb531f5","unresolved":false,"context_lines":[{"line_number":591,"context_line":"                          title\u003d\"Classes to skip\")"},{"line_number":592,"context_line":""},{"line_number":593,"context_line":"SkipGroup \u003d ["},{"line_number":594,"context_line":"    cfg.MultiStrOpt(\u0027skip_list\u0027,"},{"line_number":595,"context_line":"                    default\u003dNone,"},{"line_number":596,"context_line":"                    help\u003d\"List of classes to skip\")"},{"line_number":597,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FYuk%3D","line":594,"in_reply_to":"AAAAQn%2F%2Fa%2Fo%3D","updated":"2013-07-31 23:59:07.000000000","message":"I think two options might be a bit unnecessarily complicated.  However I can see \"skip_list\" implies a list ... so maybe the best thing is to just have individual \"skip\" options","commit_id":"b06fdc440a2bc2eb7cc2d47a9de59e0105219137"}],"tempest/test.py":[{"author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"change_message_id":"e0409b110977e6f867f899578ebf41acbc892caa","unresolved":false,"context_lines":[{"line_number":125,"context_line":"            cls.useFixture(fixtures.MonkeyPatch(\u0027sys.stderr\u0027, stderr))"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"        if cls.config.skip.skip_list:"},{"line_number":128,"context_line":"            if [c for c in cls.config.skip.skip_list if c in repr(cls)]:"},{"line_number":129,"context_line":"                raise testtools.TestCase.skipException("},{"line_number":130,"context_line":"                    \"Skipping due to config skip_list match\")"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2Fbnk%3D","line":128,"updated":"2013-07-31 09:58:08.000000000","message":"minor: could stop at after finding the first one.","commit_id":"b06fdc440a2bc2eb7cc2d47a9de59e0105219137"}]}