From 7b5ef8aede4afb5ed419219bdc91deb55b90fdec Mon Sep 17 00:00:00 2001 From: Florence Blanc-Renaud Date: Thu, 19 Aug 2021 10:51:01 +0200 Subject: [PATCH] ipatests: use whole date for journalctl --since MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When a test is executed around midnight and is checking the journal content with --since=date, it needs to specify the whole date (with day and time) to avoid missing entries. If for instance --since=23:59:00 is used and the current time is now 00:01:00, --since=23:59:00 would refer to a date in the future and no journal entry will be found. Fixes: https://pagure.io/freeipa/issue/8953 Reviewed-By: Stanislav Levin Reviewed-By: François Cami --- ipatests/test_integration/test_cert.py | 2 +- ipatests/test_integration/test_commands.py | 3 ++- ipatests/test_integration/test_nfs.py | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/ipatests/test_integration/test_cert.py b/ipatests/test_integration/test_cert.py index 9a90db5e2..7d51b76ee 100644 --- a/ipatests/test_integration/test_cert.py +++ b/ipatests/test_integration/test_cert.py @@ -69,7 +69,7 @@ class TestInstallMasterClient(IntegrationTest): # time to look into journal logs in # test_certmonger_ipa_responder_jsonrpc - cls.since = time.strftime('%H:%M:%S') + cls.since = time.strftime('%Y-%m-%d %H:%M:%S') def test_cacert_file_appear_with_option_F(self): """Test if getcert creates cacert file with -F option diff --git a/ipatests/test_integration/test_commands.py b/ipatests/test_integration/test_commands.py index f11c75609..13ae6b6c6 100644 --- a/ipatests/test_integration/test_commands.py +++ b/ipatests/test_integration/test_commands.py @@ -1208,7 +1208,8 @@ class TestIPACommand(IntegrationTest): # start to look at logs a bit before "now" # https://pagure.io/freeipa/issue/8432 since = time.strftime( - '%H:%M:%S', (datetime.now() - timedelta(seconds=10)).timetuple() + '%Y-%m-%d %H:%M:%S', + (datetime.now() - timedelta(seconds=10)).timetuple() ) password = 'WrongPassword' diff --git a/ipatests/test_integration/test_nfs.py b/ipatests/test_integration/test_nfs.py index 9a6153409..dc53a6da9 100644 --- a/ipatests/test_integration/test_nfs.py +++ b/ipatests/test_integration/test_nfs.py @@ -130,7 +130,7 @@ class TestNFS(IntegrationTest): nfsclt = self.clients[1] # for journalctl --since - since = time.strftime('%H:%M:%S') + since = time.strftime('%Y-%m-%d %H:%M:%S') nfsclt.run_command(["systemctl", "restart", "rpc-gssd"]) time.sleep(WAIT_AFTER_INSTALL) mountpoints = ("/mnt/krb", "/mnt/std", "/home")