Skip to content

Conversation

@Pterosaur
Copy link
Contributor

What I did
Enable SAI_TAM_TEL_TYPE_ATTR_SWITCH_ENABLE_OUTPUT_QUEUE_STATS to TAM_TEL_TYPE

Why I did it
This attribute is needed if the HFT want to support stats of IPG

How I verified it
Check Azp

Details if related

@Pterosaur Pterosaur requested a review from prsunny as a code owner October 29, 2025 23:24
Copilot AI review requested due to automatic review settings October 29, 2025 23:24
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enables output queue statistics collection in the high frequency telemetry (HFT) profile by adding the SAI_TAM_TEL_TYPE_ATTR_SWITCH_ENABLE_OUTPUT_QUEUE_STATS attribute to the TAM telemetry type object configuration.

  • Adds a new SAI attribute to enable output queue stats collection
  • Follows the existing pattern for enabling other statistics types (port stats, MMU stats, etc.)
  • Enables comprehensive telemetry coverage by including queue-level metrics

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@DavidZagury DavidZagury left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, I will test it, but I believe this will fix our issue.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Pterosaur
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Collaborator

prsunny commented Nov 14, 2025

Ready to merge after PR checkers.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

6 participants