make the bin scripts called with main()

This will make everything easier to manage and callable by other python
scripts/modules if you need to depend on the module. Others can then
just call the main() function instead of executing the script.

Closes-Bug: #1261823

Change-Id: I5e94ee19bd4f7a69d7ca5efc6288d402520cd5bf

bin/trove-api

@@ -41,7 +41,7 @@ from trove.db import get_db_api
 CONF = cfg.CONF
 
 
-if __name__ == '__main__':
+def main():
     cfg.parse_args(sys.argv)
     logging.setup(None)
 
@@ -55,3 +55,7 @@ if __name__ == '__main__':
         import traceback
         print(traceback.format_exc())
         sys.exit("ERROR: %s" % error)
+
+
+if __name__ == '__main__':
+    main()

bin/trove-api-os-database

@@ -57,7 +57,7 @@ FLAGS = flags.FLAGS
 FLAGS.register_opts(trove_opts)
 
 
-if __name__ == '__main__':
+def main():
     utils.default_flagfile()
     flags.FLAGS(sys.argv)
     logging.setup()
@@ -66,3 +66,7 @@ if __name__ == '__main__':
     server = service.WSGIService(SERVICE_NAME, loader=loader)
     service.serve(server)
     service.wait()
+
+
+if __name__ == '__main__':
+    main()

bin/trove-conductor

@@ -37,7 +37,7 @@ from trove.openstack.common import service as openstack_service
 CONF = cfg.CONF
 
 
-def main():
+def launch_services():
     get_db_api().configure_db(CONF)
     manager = 'trove.conductor.manager.Manager'
     topic = CONF.conductor_queue
@@ -47,7 +47,7 @@ def main():
     launcher.wait()
 
 
-if __name__ == '__main__':
+def main():
     cfg.parse_args(sys.argv)
     logging.setup(None)
 
@@ -56,7 +56,11 @@ if __name__ == '__main__':
     if not debug_utils.enabled():
         eventlet.monkey_patch(thread=True)
     try:
-        main()
+        launch_services()
     except RuntimeError as error:
         print(traceback.format_exc())
         sys.exit("ERROR: %s" % error)
+
+
+if __name__ == '__main__':
+    main()

bin/trove-guestagent

@@ -43,7 +43,8 @@ from trove.db import get_db_api
 CONF = cfg.CONF
 CONF.register_opts([openstack_cfg.StrOpt('guest_id')])
 
-if __name__ == '__main__':
+
+def main():
     cfg.parse_args(sys.argv)
     from trove.guestagent import dbaas
     logging.setup(None)
@@ -62,3 +63,7 @@ if __name__ == '__main__':
         import traceback
         print(traceback.format_exc())
         sys.exit("ERROR: %s" % error)
+
+
+if __name__ == '__main__':
+    main()

bin/trove-manage

@@ -95,7 +95,7 @@ class Commands(object):
             return utils.MethodInspector(getattr(self, command_name))
 
 
-if __name__ == '__main__':
+def main():
 
     def actions(subparser):
         parser = subparser.add_parser('db_sync')
@@ -133,3 +133,7 @@ if __name__ == '__main__':
     except Exception:
         print(_("Command failed, please check log for more info"))
         raise
+
+
+if __name__ == '__main__':
+    main()

bin/trove-mgmt-taskmanager

@@ -45,7 +45,8 @@ from trove.db import get_db_api
 CONF = cfg.CONF
 CONF.register_opts([openstack_cfg.StrOpt('taskmanager_manager')])
 
-if __name__ == '__main__':
+
+def main():
     cfg.parse_args(sys.argv)
     logging.setup(None)
 
@@ -59,3 +60,7 @@ if __name__ == '__main__':
         import traceback
         print(traceback.format_exc())
         sys.exit("ERROR: %s" % error)
+
+
+if __name__ == '__main__':
+    main()

bin/trove-taskmanager

@@ -45,7 +45,8 @@ if os.path.exists(os.path.join(possible_topdir, 'trove', '__init__.py')):
 CONF = cfg.CONF
 CONF.register_opts([openstack_cfg.StrOpt('taskmanager_manager')])
 
-if __name__ == '__main__':
+
+def main():
     cfg.parse_args(sys.argv)
     logging.setup(None)
 
@@ -68,3 +69,7 @@ if __name__ == '__main__':
         import traceback
         print(traceback.format_exc())
         sys.exit("ERROR: %s" % error)
+
+
+if __name__ == '__main__':
+    main()