Skip to content

Commit

Permalink
DAOS-15943 test: Remove server logging from pre-teardown (#15282) (#1…
Browse files Browse the repository at this point in the history
…5386)

Signed-off-by: Maureen Jean <[email protected]>
  • Loading branch information
mjean308 authored Oct 28, 2024
1 parent eb95b55 commit bde13c3
Showing 1 changed file with 2 additions and 13 deletions.
15 changes: 2 additions & 13 deletions src/tests/ftest/util/soak_test_base.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@
from soak_utils import (SoakTestError, add_pools, build_job_script, cleanup_dfuse,
create_app_cmdline, create_dm_cmdline, create_fio_cmdline,
create_ior_cmdline, create_macsio_cmdline, create_mdtest_cmdline,
create_racer_cmdline, ddhhmmss_format, get_daos_server_logs, get_harassers,
get_journalctl, launch_exclude_reintegrate, launch_extend, launch_reboot,
create_racer_cmdline, ddhhmmss_format, get_harassers,
launch_exclude_reintegrate, launch_extend, launch_reboot,
launch_server_stop_start, launch_snapshot, launch_vmd_identify_check,
reserved_file_copy, run_event_check, run_metrics_check, run_monitor_check)

Expand Down Expand Up @@ -164,17 +164,6 @@ def pre_tear_down(self):

# display final metrics
run_metrics_check(self, prefix="final")
# Gather server logs
try:
get_daos_server_logs(self)
except SoakTestError as error:
errors.append(f"<<FAILED: Failed to gather server logs {error}>>")
# Gather journalctl logs
hosts = list(set(self.hostlist_servers))
since = time.strftime('%Y-%m-%d %H:%M:%S', time.localtime(self.start_time))
until = time.strftime('%Y-%m-%d %H:%M:%S', time.localtime(self.end_time))
for journalctl_type in ["kernel", "daos_server"]:
get_journalctl(self, hosts, since, until, journalctl_type, logging=True)

if self.all_failed_harassers:
errors.extend(self.all_failed_harassers)
Expand Down

0 comments on commit bde13c3

Please sign in to comment.