base_tests: remove unused clean_shutdown field
diff --git a/src/python/oftest/base_tests.py b/src/python/oftest/base_tests.py
index 38621af..6ae6fb8 100644
--- a/src/python/oftest/base_tests.py
+++ b/src/python/oftest/base_tests.py
@@ -29,8 +29,6 @@
             switch=config["switch_ip"],
             host=config["controller_host"],
             port=config["controller_port"])
-        # clean_shutdown should be set to False to force quit app
-        self.clean_shutdown = True
         self.controller.start()
         #@todo Add an option to wait for a pkt transaction to ensure version
         # compatibilty?
@@ -74,9 +72,7 @@
     def tearDown(self):
         logging.info("** END TEST CASE " + str(self))
         self.controller.shutdown()
-        #@todo Review if join should be done on clean_shutdown
-        if self.clean_shutdown:
-            self.controller.join()
+        self.controller.join()
         del self.controller
 
     def runTest(self):
@@ -125,7 +121,6 @@
     """
 
     def setUp(self):
-        self.clean_shutdown = True
         logging.info("** START DataPlaneOnly CASE " + str(self))
         self.dataplane = oftest.dataplane_instance
         self.dataplane.flush()