)]}'
{"tempest/stress/actions/unit_test.py":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"9e59dac813da90a38347432663a6c1c925fb9303","unresolved":false,"context_lines":[{"line_number":33,"context_line":"            raise Exception(res.errors)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def run(self):"},{"line_number":36,"context_line":"        if self.class_setup_in_run:"},{"line_number":37,"context_line":"            self.klass.setUpClass()"},{"line_number":38,"context_line":"            self.run_core()"},{"line_number":39,"context_line":"            self.klass.tearDownClass()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2Fj6M%3D","line":36,"updated":"2013-07-29 06:48:07.000000000","message":"Maybe we find a way how to avoid this. I would like to have a stress test run focused on the real action.","commit_id":"2271ba997ffdd3884e532d075c3db46c82842a8c"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"c51d473e3a111cf54d2a5b55cf7a3d0439b57e5e","unresolved":false,"context_lines":[{"line_number":33,"context_line":"            raise Exception(res.errors)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def run(self):"},{"line_number":36,"context_line":"        if self.class_setup_in_run:"},{"line_number":37,"context_line":"            self.klass.setUpClass()"},{"line_number":38,"context_line":"            self.run_core()"},{"line_number":39,"context_line":"            self.klass.tearDownClass()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FTsM%3D","line":36,"in_reply_to":"AAAAQn%2F%2Fj6M%3D","updated":"2013-08-02 06:07:12.000000000","message":"You can turn it off.\nand just run the setUp, tearDown, (and another post cleanups) in the main loop.","commit_id":"2271ba997ffdd3884e532d075c3db46c82842a8c"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"a8ae0727ed2cffa84d82aaa1b2934f907ebf463b","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"class UnitTest(stressaction.StressAction):"},{"line_number":34,"context_line":"    \"\"\"This is a special action for running existing unittests,"},{"line_number":35,"context_line":"       In the action description ``kwargs`` filed you need to specify the"},{"line_number":36,"context_line":"       ``test_method`` which the fully qualified name of a unittest test"},{"line_number":37,"context_line":"       method."}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQ3%2F%2F5N8%3D","line":34,"updated":"2013-08-13 11:07:56.000000000","message":"\"existing unittests as stress tests.\"","commit_id":"c74f1b70b14113d2a94647ee3171d22fcb58b764"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"a8ae0727ed2cffa84d82aaa1b2934f907ebf463b","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"class UnitTest(stressaction.StressAction):"},{"line_number":34,"context_line":"    \"\"\"This is a special action for running existing unittests,"},{"line_number":35,"context_line":"       In the action description ``kwargs`` filed you need to specify the"},{"line_number":36,"context_line":"       ``test_method`` which the fully qualified name of a unittest test"},{"line_number":37,"context_line":"       method."},{"line_number":38,"context_line":"       Depending on what kind of stress you would to cause, you can specify"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQ3%2F%2F5NE%3D","line":35,"updated":"2013-08-13 11:07:56.000000000","message":"I\u0027d replace 35-40 with the following:\n\nYou need to pass ``test_method`` and ``class_setup_per`` using ``kwargs`` in the JSON descriptor; ``test_method`` should be the fully qualified name of a unittest, ``class_setup_per`` should be one from:","commit_id":"c74f1b70b14113d2a94647ee3171d22fcb58b764"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"a8ae0727ed2cffa84d82aaa1b2934f907ebf463b","unresolved":false,"context_lines":[{"line_number":39,"context_line":"       when to run the setUpClass and tearDownClass by the ``class_setup_per``"},{"line_number":40,"context_line":"       option in the ``kwargs`` as well."},{"line_number":41,"context_line":"           ``application``: once in the stress job lifetime"},{"line_number":42,"context_line":"           ``process``: one in the worker process lifetime"},{"line_number":43,"context_line":"           ``action``: on each action"},{"line_number":44,"context_line":"       Not all combination working in every case."},{"line_number":45,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQ3%2F%2F5QU%3D","line":42,"updated":"2013-08-13 11:07:56.000000000","message":"I think this should read \"once in the thread process lifetime\"","commit_id":"c74f1b70b14113d2a94647ee3171d22fcb58b764"}],"tempest/stress/etc/sample-unit-test.json":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"9e59dac813da90a38347432663a6c1c925fb9303","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[{\"action\": \"tempest.stress.actions.unit_test.UnitTest\","},{"line_number":2,"context_line":"  \"threads\": 8,"},{"line_number":3,"context_line":"  \"use_admin\": false,"},{"line_number":4,"context_line":"  \"use_isolated_tenants\": false,"}],"source_content_type":"application/json","patch_set":1,"id":"AAAAQn%2F%2Fj7Y%3D","line":1,"updated":"2013-07-29 06:48:07.000000000","message":"Other way around: we specify here the unit test class / function as action and add a properties test_type \u003d stress | unit? I think it makes sense that run_stress natively supports unit tests without a \"workaround\" stress action.","commit_id":"2271ba997ffdd3884e532d075c3db46c82842a8c"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"c51d473e3a111cf54d2a5b55cf7a3d0439b57e5e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[{\"action\": \"tempest.stress.actions.unit_test.UnitTest\","},{"line_number":2,"context_line":"  \"threads\": 8,"},{"line_number":3,"context_line":"  \"use_admin\": false,"},{"line_number":4,"context_line":"  \"use_isolated_tenants\": false,"}],"source_content_type":"application/json","patch_set":1,"id":"AAAAQn%2F%2FTqE%3D","line":1,"in_reply_to":"AAAAQn%2F%2Fj7Y%3D","updated":"2013-08-02 06:07:12.000000000","message":"Lets\u0027 call it extension.\n\nI am not sure we want to extend the core with real unittest runner capabilities.","commit_id":"2271ba997ffdd3884e532d075c3db46c82842a8c"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"0665466738469430e9020a921f3fcc9e36988d1e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"  \"use_isolated_tenants\": false,"},{"line_number":5,"context_line":"  \"kwargs\": {\"test_class\": \"tempest.cli.simple_read_only.test_glance.SimpleReadOnlyGlanceClientTest\","},{"line_number":6,"context_line":"             \"test_method\": \"test_glance_fake_action\","},{"line_number":7,"context_line":"             \"class_setup_in_run\": true}"},{"line_number":8,"context_line":"  }"},{"line_number":9,"context_line":"]"}],"source_content_type":"application/json","patch_set":1,"id":"AAAAQn%2F%2FkZM%3D","line":7,"updated":"2013-07-28 20:49:07.000000000","message":"For scenario tests the sanity check logic in the test.py,  prevents to reuse the class in the simple way.\n\nMore tricky class loading could help, but it costs more, and I do not know about a tricky test case where you both use the mangers and an interesting setUpClass.\n\nSo,  the forced class reload is not real need now, If it will be, the reload will get an additional option.","commit_id":"2271ba997ffdd3884e532d075c3db46c82842a8c"}]}