Skip to content

Conversation

@hnaeisse
Copy link
Contributor

Associated JIRA ticket number/Github issue number

Applies changed mandated by SWDEV-543981

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Continuous Integration

What were the changes?

The .cc version of this test was out of date in comparison to the .cpp version so it was brought up to date. Additionally, all the features requested in the first task of https://ontrack-internal.amd.com/browse/SWDEV-543981 were implemented.

Why are these changes needed?

Need to diversify communication testing to cover all scenarios, device to device, device to host etc.

Updated CHANGELOG?

  • Yes
  • No, Does not apply to this PR.

Added/Updated documentation?

  • Yes
  • No, Does not apply to this PR.

Additional Checks

  • I have added tests relevant to the introduced functionality, and the unit tests are passing locally.
  • Any dependent changes have been merged.

🔁 Imported from ROCm/hip-tests#501
🧑‍💻 Originally authored by @rocm-devops

@hnaeisse hnaeisse requested a review from a team as a code owner August 20, 2025 19:45
@hnaeisse hnaeisse merged commit b2857b5 into develop Aug 21, 2025
10 of 11 checks passed
@hnaeisse hnaeisse deleted the SWDEV-543981.ii branch August 21, 2025 13:06
@hnaeisse hnaeisse self-assigned this Aug 22, 2025
@hnaeisse hnaeisse requested a review from sruscica August 22, 2025 13:40
systems-assistant bot pushed a commit to ROCm/hip-tests that referenced this pull request Aug 26, 2025
 errors in the previous PR due to migration (#672)

Co-authored-by: hnaeisse_amdeng <[email protected]>
[rocm-systems] ROCm/rocm-systems#672 (commit b2857b5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants