Skip to content

test: Add tests for the Triton log callback#8858

Open
pskiran1 wants to merge 6 commits into
mainfrom
spolisetty/tri-1194-integrate-triton-logging-into-dynamo
Open

test: Add tests for the Triton log callback#8858
pskiran1 wants to merge 6 commits into
mainfrom
spolisetty/tri-1194-integrate-triton-logging-into-dynamo

Conversation

@pskiran1

@pskiran1 pskiran1 commented Jun 25, 2026

Copy link
Copy Markdown
Member

What does the PR do?

Adds QA coverage for the log callback feature. Builds and runs the common logging_test as part of L0_logging, and runs the new test_logging_callback.py in L0_python_api.

Checklist

  • PR title reflects the change and is of format <commit_type>: <Title>
  • Changes are described in the pull request.
  • Related issues are referenced.
  • Populated github labels field
  • Added test plan and verified test passes.
  • Verified that the PR passes existing CI.
  • Verified copyright is correct on all changed files.
  • Added succinct git squash message before merging ref.
  • All template sections are filled out.
  • Optional: Additional screenshots for behavior/output changes with before/after.

Commit Type:

Check the conventional commit type
box here and add the label to the github PR.

  • build
  • ci
  • docs
  • feat
  • fix
  • perf
  • refactor
  • revert
  • style
  • test

Related PRs:

Where should the reviewer start?

Test plan:

  • CI Pipeline ID:

Caveats:

Background

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • closes GitHub issue: #xxx

 into spolisetty/tri-1194-integrate-triton-logging-into-dynamo
@pskiran1 pskiran1 requested a review from Vinya567 June 29, 2026 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: test Adding missing tests or correcting existing test

Development

Successfully merging this pull request may close these issues.

1 participant