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.

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

Add support for DASH ENI Counters #1418

Merged
merged 4 commits into from
Sep 16, 2024
Merged

Conversation

vivekrnv
Copy link
Contributor

Depends on sonic-net/sonic-swss-common#904

  1. Add support for DASH ENI Flex Counter and added a UT
  2. Refactored the Flexcounter UT to not use hardcoded oid and instead generate in the test based on object type.
Note: Google Test filter = *FlexCounter*

[==========] Running 7 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 7 tests from FlexCounter
[ RUN      ] FlexCounter.addRemoveCounter
[       OK ] FlexCounter.addRemoveCounter (16804 ms)
[ RUN      ] FlexCounter.queryCounterCapability
[       OK ] FlexCounter.queryCounterCapability (1050 ms)
[ RUN      ] FlexCounter.noSupportedCounters
[       OK ] FlexCounter.noSupportedCounters (0 ms)
[ RUN      ] FlexCounter.addRemoveCounterPlugin
[       OK ] FlexCounter.addRemoveCounterPlugin (1 ms)
[ RUN      ] FlexCounter.addRemoveCounterForPort
[       OK ] FlexCounter.addRemoveCounterForPort (2000 ms)
[ RUN      ] FlexCounter.bulkCounter
[       OK ] FlexCounter.bulkCounter (12604 ms)
[ RUN      ] FlexCounter.counterIdChange
[       OK ] FlexCounter.counterIdChange (6300 ms)
[----------] 7 tests from FlexCounter (38760 ms total)

[----------] Global test environment tear-down
[==========] 7 tests from 1 test suite ran. (38761 ms total)
[  PASSED ] 7 tests.

@vivekrnv
Copy link
Contributor Author

/azpw run Azure.sonic-sairedis

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-sairedis

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vivekrnv vivekrnv requested review from prsunny and removed request for prsunny September 14, 2024 23:14
@saiarcot895 saiarcot895 merged commit 9b9d330 into sonic-net:master Sep 16, 2024
17 checks passed
@vivekrnv vivekrnv deleted the eni_cntrs branch September 16, 2024 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants