)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"2cd234f6a6d35966f0d1adb04a4b4b6ad6611519","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For the moment, each tempest run loads all installed external"},{"line_number":10,"context_line":"plugins, reads them and loads list of tests."},{"line_number":11,"context_line":"But it is redundant, tempest user may want to use only some"},{"line_number":12,"context_line":"specific plugin or none of them for different reasons."},{"line_number":13,"context_line":"Since we cannot just get and disable entry points that are installed"},{"line_number":14,"context_line":"with packages we should be able to ignore them in tempest."},{"line_number":15,"context_line":"So, add config group called \"ext_plugin\" that will contain all"},{"line_number":16,"context_line":"options related to external plugins and define first option there"},{"line_number":17,"context_line":"called \"plugin_names_matcher\". This option expects any regexp"},{"line_number":18,"context_line":"that will be applied to tempest plugin names. By default it is"},{"line_number":19,"context_line":"empty string that means *all* plugins will be loaded."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Examples:"},{"line_number":22,"context_line":"1) All plugins"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bacf61ea_ea3e15b7","line":19,"range":{"start_line":11,"start_character":0,"end_line":19,"end_character":53},"updated":"2016-07-28 11:03:10.000000000","message":"I disagree with the commit message. \nThere is a way today to achieve what you want today i.e. by setting up a venv with no site packages, and installing in there only the plugins that you need. \nIf using a venv is not an option for any reason, I think this a potential solution. But I think the commit message should reflect that more clearly.","commit_id":"dd34d8de948796a8ba63edfa10e7da9fab89d4a1"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"3e7ee3fb1b5dc85450c3f2b9137a86b75ee8fe9e","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For the moment, each tempest run loads all installed external"},{"line_number":10,"context_line":"plugins, reads them and loads list of tests."},{"line_number":11,"context_line":"But it is redundant, tempest user may want to use only some"},{"line_number":12,"context_line":"specific plugin or none of them for different reasons."},{"line_number":13,"context_line":"Since we cannot just get and disable entry points that are installed"},{"line_number":14,"context_line":"with packages we should be able to ignore them in tempest."},{"line_number":15,"context_line":"So, add config group called \"ext_plugin\" that will contain all"},{"line_number":16,"context_line":"options related to external plugins and define first option there"},{"line_number":17,"context_line":"called \"plugin_names_matcher\". This option expects any regexp"},{"line_number":18,"context_line":"that will be applied to tempest plugin names. By default it is"},{"line_number":19,"context_line":"empty string that means *all* plugins will be loaded."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Examples:"},{"line_number":22,"context_line":"1) All plugins"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bacf61ea_05cf1a80","line":19,"range":{"start_line":11,"start_character":0,"end_line":19,"end_character":53},"in_reply_to":"bacf61ea_ea3e15b7","updated":"2016-07-28 11:23:23.000000000","message":"Even using venv, I should not be forced to remove packages for each of several such cases.","commit_id":"dd34d8de948796a8ba63edfa10e7da9fab89d4a1"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"2cd234f6a6d35966f0d1adb04a4b4b6ad6611519","unresolved":false,"context_lines":[{"line_number":19,"context_line":"empty string that means *all* plugins will be loaded."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Examples:"},{"line_number":22,"context_line":"1) All plugins"},{"line_number":23,"context_line":"[ext_plugins]"},{"line_number":24,"context_line":"plugin_names_matcher \u003d \u0027\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"2) Only plugins that contain word \u0027foo\u0027"},{"line_number":27,"context_line":"[ext_plugins]"},{"line_number":28,"context_line":"plugin_names_matcher \u003d foo #foo_tests, tests_for_foo"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"3) Only plugins that contain word \u0027foo\u0027 or \u0027bar\u0027"},{"line_number":31,"context_line":"[ext_plugins]"},{"line_number":32,"context_line":"plugin_names_matcher \u003d foo|bar # foo_tests, bar_tests"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Change-Id: I24927aed3c44230f43c4baccd68bf37742db9e85"},{"line_number":35,"context_line":"Closes-Bug: #1604097"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bacf61ea_eada9577","line":32,"range":{"start_line":22,"start_character":0,"end_line":32,"end_character":53},"updated":"2016-07-28 11:03:10.000000000","message":"Using an empty string to match all and a regex string to whitelist otherwise seems a bit confusing to me. Besides, since the match is on the name of the plugin, I think using a list would be more appropriate.","commit_id":"dd34d8de948796a8ba63edfa10e7da9fab89d4a1"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"3e7ee3fb1b5dc85450c3f2b9137a86b75ee8fe9e","unresolved":false,"context_lines":[{"line_number":19,"context_line":"empty string that means *all* plugins will be loaded."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Examples:"},{"line_number":22,"context_line":"1) All plugins"},{"line_number":23,"context_line":"[ext_plugins]"},{"line_number":24,"context_line":"plugin_names_matcher \u003d \u0027\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"2) Only plugins that contain word \u0027foo\u0027"},{"line_number":27,"context_line":"[ext_plugins]"},{"line_number":28,"context_line":"plugin_names_matcher \u003d foo #foo_tests, tests_for_foo"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"3) Only plugins that contain word \u0027foo\u0027 or \u0027bar\u0027"},{"line_number":31,"context_line":"[ext_plugins]"},{"line_number":32,"context_line":"plugin_names_matcher \u003d foo|bar # foo_tests, bar_tests"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Change-Id: I24927aed3c44230f43c4baccd68bf37742db9e85"},{"line_number":35,"context_line":"Closes-Bug: #1604097"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bacf61ea_650a56d4","line":32,"range":{"start_line":22,"start_character":0,"end_line":32,"end_character":53},"in_reply_to":"bacf61ea_eada9577","updated":"2016-07-28 11:23:23.000000000","message":"What is confusing in asking for a \"matcher\"? Why it should be list? List of what?","commit_id":"dd34d8de948796a8ba63edfa10e7da9fab89d4a1"}]}