lvmdbustest.py: Print to stderr

It's easier to follow if we print to stderr, so that our print messages
are in the lvm error log in the correct position to other printed
messages.

1 files changed, 13 insertions, 9 deletions

diff --git a/test/dbus/lvmdbustest.py b/test/dbus/lvmdbustest.py
index e9b27a0a0..ef2c07a22 100755
--- a/test/dbus/lvmdbustest.py
+++ b/test/dbus/lvmdbustest.py
@@ -29,6 +29,10 @@ else:
 	bus = dbus.SystemBus(mainloop=DBusGMainLoop())
 
 
+def std_err_print(*args, **kwargs):
+	print(*args, file=sys.stderr, **kwargs)
+
+
 def get_objects():
 	rc = {MANAGER_INT: [], PV_INT: [], VG_INT: [], LV_INT: [],
 			THINPOOL_INT: [], JOB_INT: [], SNAPSHOT_INT: [], LV_COMMON_INT: [],
@@ -63,14 +67,14 @@ class TestDbusService(unittest.TestCase):
 		# we are not mucking with someones data on their system
 		self.objs, self.bus = get_objects()
 		if len(self.objs[PV_INT]) == 0:
-			print('No PVs present exiting!')
+			std_err_print('No PVs present exiting!')
 			sys.exit(1)
 		if len(self.objs[MANAGER_INT]) != 1:
-			print('Expecting a manager object!')
+			std_err_print('Expecting a manager object!')
 			sys.exit(1)
 
 		if len(self.objs[VG_INT]) != 0:
-			print('Expecting no VGs to exist!')
+			std_err_print('Expecting no VGs to exist!')
 			sys.exit(1)
 
 		self.pvs = []
@@ -808,7 +812,7 @@ class TestDbusService(unittest.TestCase):
 			yes = self._test_expired_timer(i)
 			if yes:
 				break
-			print('Attempt (%d) failed, trying again...' % (i))
+			std_err_print('Attempt (%d) failed, trying again...' % (i))
 
 		self.assertTrue(yes)
 
@@ -918,7 +922,7 @@ class TestDbusService(unittest.TestCase):
 		# TODO renable test case when
 		# https://bugzilla.redhat.com/show_bug.cgi?id=1264169 gets fixed
 		# This was tested with lvmetad disabled and we passed
-		print("\nSkipping Vg.UuidGenerate until BZ: 1264169 resolved\n")
+		std_err_print("\nSkipping Vg.UuidGenerate until BZ: 1264169 resolved\n")
 
 		if False:
 			vg = self._vg_create().Vg
@@ -1320,22 +1324,22 @@ if __name__ == '__main__':
 	set_execution(False)
 
 	if int(test_shell) == 0:
-		print('\n Shortened fork & exec test ***\n')
+		std_err_print('\n Running only lvm fork & exec test ***\n')
 		r.register_result(unittest.main(exit=False))
 	else:
-		print('\n *** Testing fork & exec *** \n')
+		std_err_print('\n *** Testing with lvm fork & exec *** \n')
 		r.register_result(unittest.main(exit=False))
 		g_tmo = 15
 		r.register_result(unittest.main(exit=False))
 		# Test lvm shell
-		print('\n *** Testing lvm shell *** \n')
+		std_err_print('\n *** Testing with lvm shell *** \n')
 		if set_execution(True):
 			g_tmo = 0
 			r.register_result(unittest.main(exit=False))
 			g_tmo = 15
 			r.register_result(unittest.main(exit=False))
 		else:
-			print("WARNING: Unable to dynamically configure "
+			std_err_print("ERROR: Unable to dynamically configure "
 				"service to use lvm shell!")
 
 	r.exit_run()