diff --git a/sos/report/plugins/__init__.py b/sos/report/plugins/__init__.py index 46ca80306e..925939d8f9 100644 --- a/sos/report/plugins/__init__.py +++ b/sos/report/plugins/__init__.py @@ -3088,8 +3088,15 @@ def add_journal(self, units=None, boot=None, since=None, until=None, if output: journal_cmd += output_opt % output + fname = journal_cmd + tac = False + if log_size > 0: + journal_cmd = f"{journal_cmd} --reverse" + tac = True + self._log_debug(f"collecting journal: {journal_cmd}") self._add_cmd_output(cmd=journal_cmd, timeout=timeout, + tac=tac, to_file=True, suggest_filename=fname, sizelimit=log_size, pred=pred, tags=tags, priority=priority) diff --git a/tests/report_tests/plugin_tests/defaults.py b/tests/report_tests/plugin_tests/defaults.py index 4d790114d1..26efdf7f3b 100644 --- a/tests/report_tests/plugin_tests/defaults.py +++ b/tests/report_tests/plugin_tests/defaults.py @@ -36,7 +36,7 @@ def test_journal_collected(self): _m = self.get_plugin_manifest('cups') ent = None for cmd in _m['commands']: - if cmd['exec'] == 'journalctl --no-pager --unit cups': + if cmd['exec'] == 'journalctl --no-pager --unit cups --reverse': ent = cmd assert ent, "No manifest entry for journalctl cups"