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.

Sign up for GitHub

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

Jump to bottom

#0: Fix T3000 unit tests and suppress chatty logs #17520

Merged
merged 1 commit into from
Feb 4, 2025
Merged

#0: Fix T3000 unit tests and suppress chatty logs #17520

merged 1 commit into from
Feb 4, 2025

Conversation

cfjchu
Copy link
Collaborator

@cfjchu cfjchu commented Feb 3, 2025 •
edited
Loading

Ticket

None

Problem description

  • T3000 unit test suite erroring out because of incorrect binary path for programming examples
  • Hard to read CI log because of chatty log_info message

What's changed

  • Corrected binary path
  • log_info -> log_debug

Checklist

@cfjchu cfjchu merged commit da947e1 into main Feb 4, 2025
211 of 215 checks passed
@cfjchu cfjchu deleted the jchu/fix-t3k-unit-tests branch February 4, 2025 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@afuller-TT afuller-TT afuller-TT approved these changes

@blozano-tt blozano-tt blozano-tt approved these changes

@tt-aho tt-aho tt-aho approved these changes

@abhullar-tt abhullar-tt Awaiting requested review from abhullar-tt abhullar-tt is a code owner

@pgkeller pgkeller Awaiting requested review from pgkeller pgkeller is a code owner

@aliuTT aliuTT Awaiting requested review from aliuTT aliuTT is a code owner

@tt-dma tt-dma Awaiting requested review from tt-dma tt-dma is a code owner

@tt-asaigal tt-asaigal Awaiting requested review from tt-asaigal tt-asaigal is a code owner

@ubcheema ubcheema Awaiting requested review from ubcheema ubcheema is a code owner

@omilyutin-tt omilyutin-tt Awaiting requested review from omilyutin-tt omilyutin-tt is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@cfjchu @tt-aho @blozano-tt @afuller-TT