)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a861880b8c9df569ff1a8f16620e022c6331d873","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Divyansh \u003cdivyansh.acharya7@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-08-07 19:23:43 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add a --load-list argument to tempest run that will accept a non-regex whitelist."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In tempest run"},{"line_number":10,"context_line":"\u0027-l/--list-tests\u0027 argument generates a non-regex list of all the tests in a seperate line:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9f436f4f_c490dc98","line":7,"updated":"2017-08-08 07:30:12.000000000","message":"nit, we don\u0027t need a trailing period. And the first should be limited to 50 chars if you can. And subsequent lines should be wrapped at 72 characters.\n\nPlease see the wiki page[1].\n\n[1] https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"f0e4490e2288dcc75d7abd7f0565cfb5cac49640"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a861880b8c9df569ff1a8f16620e022c6331d873","unresolved":false,"context_lines":[{"line_number":9,"context_line":"In tempest run"},{"line_number":10,"context_line":"\u0027-l/--list-tests\u0027 argument generates a non-regex list of all the tests in a seperate line:"},{"line_number":11,"context_line":"Added the argument \u0027--load-list\u0027, to tempest run, that accepts a file of non-regex whitelists like testr supports"},{"line_number":12,"context_line":"Added unit tests to test --whitelist-file, --blacklist-file and updated documentation for --load-list"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I75488871ac797d6bec4e2834c9d1b0c7885c891b"},{"line_number":15,"context_line":"Closes-Bug: #1700136"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9f436f4f_a48dd87c","line":12,"updated":"2017-08-08 07:30:12.000000000","message":"I think adding unit test for white/blacklist-file is not for this patch responsibility. So, I think it should be separated. And, why not add load-list unit tests in this patch?","commit_id":"f0e4490e2288dcc75d7abd7f0565cfb5cac49640"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"479f144d762f74fd89e2ae29e1c13fc17136eec9","unresolved":false,"context_lines":[{"line_number":9,"context_line":"In tempest run"},{"line_number":10,"context_line":"\u0027-l/--list-tests\u0027 argument generates a non-regex list of all the tests in a seperate line:"},{"line_number":11,"context_line":"Added the argument \u0027--load-list\u0027, to tempest run, that accepts a file of non-regex whitelists like testr supports"},{"line_number":12,"context_line":"Added unit tests to test --whitelist-file, --blacklist-file and updated documentation for --load-list"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I75488871ac797d6bec4e2834c9d1b0c7885c891b"},{"line_number":15,"context_line":"Closes-Bug: #1700136"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9f436f4f_67448d75","line":12,"in_reply_to":"9f436f4f_a48dd87c","updated":"2017-08-17 21:31:06.000000000","message":"yeah, nice point. This comment is not applied to the latest one yet.","commit_id":"f0e4490e2288dcc75d7abd7f0565cfb5cac49640"}],"tempest/cmd/run.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"016771a1e036c1c3a014e86e6ab82e30ca26745c","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"You can also use the **--load-list** option that lets you pass a filepath to"},{"line_number":51,"context_line":"tempest run with the file format being in a non-regex format, similar to the"},{"line_number":52,"context_line":"tests generated by the **--list-tests** option. Load list inputs the file"},{"line_number":53,"context_line":"as a whitelist so it simply OR all the tests in the file."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"Test Execution"},{"line_number":56,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-python","patch_set":7,"id":"9f436f4f_020ecf31","line":53,"range":{"start_line":52,"start_character":48,"end_line":53,"end_character":57},"updated":"2017-08-17 21:35:43.000000000","message":"How about\n\n You can specify target tests by removing unnecessary tests\n from a list file which is generated from **--list-tests** option.","commit_id":"9f816b513b2245cb6c4ba57aa8bf463773962e86"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"221894c92c45609cb3445966e47ced7734346081","unresolved":false,"context_lines":[{"line_number":272,"context_line":"                                        \u0027contains a separate regex exclude on \u0027"},{"line_number":273,"context_line":"                                        \u0027each newline\u0027)"},{"line_number":274,"context_line":"        list_selector.add_argument(\u0027--load-list\u0027, \u0027--load_list\u0027,"},{"line_number":275,"context_line":"                                   help\u003d\u0027Path to a non-regex whitelist file, \u0027"},{"line_number":276,"context_line":"                                        \u0027this file contains a seperate test \u0027"},{"line_number":277,"context_line":"                                        \u0027on each newline\u0027)"},{"line_number":278,"context_line":"        # list only args"},{"line_number":279,"context_line":"        parser.add_argument(\u0027--list-tests\u0027, \u0027-l\u0027, action\u003d\u0027store_true\u0027,"},{"line_number":280,"context_line":"                            help\u003d\u0027List tests\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"7f287b81_3e20aaf7","line":277,"range":{"start_line":275,"start_character":41,"end_line":277,"end_character":58},"updated":"2017-09-03 10:56:32.000000000","message":"It would probably be nice to specify that the file can be produced via the --list-tests command, as I think in any other case the regex is probably just more convenient.","commit_id":"63d0d1a92d47ece20bf2c203f0361bd97652f2c9"},{"author":{"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"},"change_message_id":"6ebdfb15e4e191e73ef4813d3759f16c325eb491","unresolved":false,"context_lines":[{"line_number":272,"context_line":"                                        \u0027contains a separate regex exclude on \u0027"},{"line_number":273,"context_line":"                                        \u0027each newline\u0027)"},{"line_number":274,"context_line":"        list_selector.add_argument(\u0027--load-list\u0027, \u0027--load_list\u0027,"},{"line_number":275,"context_line":"                                   help\u003d\u0027Path to a non-regex whitelist file, \u0027"},{"line_number":276,"context_line":"                                        \u0027this file contains a seperate test \u0027"},{"line_number":277,"context_line":"                                        \u0027on each newline\u0027)"},{"line_number":278,"context_line":"        # list only args"},{"line_number":279,"context_line":"        parser.add_argument(\u0027--list-tests\u0027, \u0027-l\u0027, action\u003d\u0027store_true\u0027,"},{"line_number":280,"context_line":"                            help\u003d\u0027List tests\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"7f287b81_7bf29a71","line":277,"range":{"start_line":275,"start_character":41,"end_line":277,"end_character":58},"in_reply_to":"7f287b81_3e20aaf7","updated":"2017-09-05 14:12:02.000000000","message":"Divi was a summer intern that work at AT\u0026T that is back at school, so I am going to give some suggestion for Andrea\u0027s request here.  \n\nPath to a non-regex whitelist file, this file contains a separate test on each newline. This command supports files created by the tempest run --list-tests command.","commit_id":"63d0d1a92d47ece20bf2c203f0361bd97652f2c9"},{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"3402ede3f2f82783c4b7101a9e7e2aa55721a466","unresolved":false,"context_lines":[{"line_number":272,"context_line":"                                        \u0027contains a separate regex exclude on \u0027"},{"line_number":273,"context_line":"                                        \u0027each newline\u0027)"},{"line_number":274,"context_line":"        list_selector.add_argument(\u0027--load-list\u0027, \u0027--load_list\u0027,"},{"line_number":275,"context_line":"                                   help\u003d\u0027Path to a non-regex whitelist file, \u0027"},{"line_number":276,"context_line":"                                        \u0027this file contains a seperate test \u0027"},{"line_number":277,"context_line":"                                        \u0027on each newline\u0027)"},{"line_number":278,"context_line":"        # list only args"},{"line_number":279,"context_line":"        parser.add_argument(\u0027--list-tests\u0027, \u0027-l\u0027, action\u003d\u0027store_true\u0027,"},{"line_number":280,"context_line":"                            help\u003d\u0027List tests\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"7f287b81_504d7070","line":277,"range":{"start_line":275,"start_character":41,"end_line":277,"end_character":58},"in_reply_to":"7f287b81_7bf29a71","updated":"2017-09-05 19:21:37.000000000","message":"Done","commit_id":"63d0d1a92d47ece20bf2c203f0361bd97652f2c9"}]}