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

Developer NVIDIA #40

Merged
merged 11 commits into from
Oct 7, 2024
Merged

Developer NVIDIA #40

merged 11 commits into from
Oct 7, 2024

Conversation

dhruvr-nvidia
Copy link
Collaborator

@dhruvr-nvidia dhruvr-nvidia commented Sep 24, 2024

  1. Added support for multipart ForceUpdate.
  2. Added option to choose between sshpass and sshtunnel for creating the portforwarding tunnel (By default sshtunnel will be used).
  3. Added a line for FailureReason in "TestScore_" file. (Done mainly for FW Update testcases)
  4. Some minor bugfixes and changes missed in the last PR.

khashaik and others added 4 commits September 24, 2024 23:35
  - Add check to wait for FwActivationTimeMax time if is_gpu_reachable()
    and when gpu device status.state is not enabled.
  - Convert NodeACReset to return bool
    - Returns False when either poweroffcommand string or poweroncommand
      is empty in the configuration file.

Signed-off-by: Khader Basha Shaik <[email protected]>
Signed-off-by: dhruvr <[email protected]>
@dhruvr-nvidia
Copy link
Collaborator Author

Tests results:

  • Passing required testcases.

Added option to choose between sshpass and sshtunnel.

See merge request dgx/sps/ocp-diag-ctam!26
Adding code for generating Failure Reason logs and Input-configurations log files.

See merge request dgx/sps/ocp-diag-ctam!24
@MeritedHobbit
Copy link
Collaborator

Can we move ssh_pass to dut_info.json instead? This can be added there along with other tunneling options.
This also avoids test_runner.json to be used by comptooldut.

@dhruvr-nvidia dhruvr-nvidia changed the title Developer NVIDIA (Support for multipart ForceUpdate) Developer NVIDIA Oct 1, 2024
@MeritedHobbit MeritedHobbit merged commit f54165b into developer Oct 7, 2024
1 check failed
@MeritedHobbit MeritedHobbit deleted the developer_nvidia branch October 7, 2024 06:43
@MeritedHobbit MeritedHobbit restored the developer_nvidia branch October 7, 2024 06:43
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.

4 participants