-
Notifications
You must be signed in to change notification settings - Fork 266
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
port counter support on sonic-vs #1275
Conversation
please fix build errors |
sonic-sairedis is successful. Build error is in orchagent. How to refresh other repo and build? In file included from dash/dashaclorch.h:19, |
@Pterosaur hey, you have been working on dash_api/ in swss repo, can you fix pipeline here on sairedis ? sonic-net/sonic-swss#2722 seems like your PR added some some libs libdashapi |
my attempt to fix: #1276, this will probably fail for the same reason, since az pipeline needs to be checked in first to take effect |
ok, seems like my attempt fixed swss build, test failure seems to be related to dash apii python module missing:
@Pterosaur can you take a look ? |
build was fixed by #1277, let restart /azp/run |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
4f2ba26
to
8c73fc0
Compare
8c73fc0
to
cd7c174
Compare
nice error XD
|
please change pr name to more meaningful |
@vishnushetty Can you please update the Quality Metric (Alpha/Beta/GA) for the feature either in this PR comments or in HLD itself based on https://github.com/sonic-net/SONiC/blob/master/doc/SONiC%20feature%20quality%20definition.md |
In sonic-vs 'show interface counters' is not supported (port counters set to zero). The counter support would be useful for debugging and automation. As part of this feature the basic port counters are fetched from corresponding host interface net stat "/sys/class/net/[hostif_name]/statistics".
The following SAI port counters are mapped to host interface statistics as below.