Merge pull request #747 from Sirozha1337/fix_jmeter_reader_not_closing

Fix JMeter reader not closing causing tank to hang
yandex · May 24, 2019 · bf64061 · bf64061
2 parents dfdb464 + ed9bef1
commit bf64061
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/yandextank/plugins/JMeter/plugin.py b/yandextank/plugins/JMeter/plugin.py
@@ -149,6 +149,7 @@ def is_test_finished(self):
             return -1
         elif aggregator.reader.jmeter_finished is True:
             if aggregator.reader.agg_finished:
+                self.reader.close()
                 return retcode
             else:
                 logger.info("Waiting for aggregator to finish")
@@ -164,6 +165,7 @@ def end_test(self, retcode):
         if self.process_stderr:
             self.process_stderr.close()
         self.core.add_artifact_file(self.jmeter_log)
+        self.reader.close()
         return retcode
 
     def __discover_jmeter_udp_port(self):