Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verbosity of smci_mi300x_platform_amdgpu_alllogs_collection.sh reduced. #73

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jatreon
Copy link

@jatreon jatreon commented Jan 20, 2025

Minor changes to smci_mi300x_platform_amdgpu_alllogs_collection.sh. Verbosity reduced and removing the logs folder, instead putting the output file directly in to the folder the script runs within.

@J1nk0
Copy link
Contributor

J1nk0 commented Jan 21, 2025

  • Thanks for adjustingit based on the feedback I provided.
  • Here is the new and very clean output looks like:
    $ sudo bash 73_smci_mi300x_platform_amdgpu_alllogs_collection.sh [sudo] password for p: Enter BMC Username: USERNAME Enter BMC Password: Enter valid BMC IP Address: 198.10.1.125 2025-01-21 17:22:42 - Collecting AllLogs file... Task 0 still running, elapsed time 264s 2025-01-21 17:27:43 - Downloading AllLogs... 2025-01-21 17:27:44 - All logs downloaded as 198.10.1.125_2025-01-21T17-27-43_all_logs.tar.xz $
  • It does create one compressed files only contain the ALL LOGS as proven to be needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants