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

Added Flex Counters support for tunnel counters #886

Merged
merged 12 commits into from Nov 2, 2021

Conversation

dgsudharsan
Copy link
Collaborator

Added Flex counter support for tunnel counters and rates implementation

syncd/FlexCounter.cpp Outdated Show resolved Hide resolved
syncd/FlexCounter.cpp Show resolved Hide resolved
syncd/FlexCounter.cpp Outdated Show resolved Hide resolved
@kcudnik
Copy link
Collaborator

kcudnik commented Aug 12, 2021

seems like build its failing because it requires your changes from swss-common

@dgsudharsan
Copy link
Collaborator Author

/azpw run

@kcudnik
Copy link
Collaborator

kcudnik commented Oct 21, 2021

Checkig for white spaces ...
../syncd/FlexCounter.cpp:            } 
ERROR: some files contain white spaces at the end of line, please fix

@dgsudharsan
Copy link
Collaborator Author

@kcudnik Solved the build issues and all comments address. Can you please review and sign off?

kcudnik
kcudnik previously approved these changes Oct 25, 2021
@dgsudharsan
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dgsudharsan
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dgsudharsan
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dgsudharsan
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dgsudharsan
Copy link
Collaborator Author

@kcudnik The pipeline is passing now. Could you please help merging the PR?

@kcudnik kcudnik merged commit 88b62ce into sonic-net:master Nov 2, 2021
dgsudharsan added a commit to dgsudharsan/sonic-sairedis that referenced this pull request Aug 26, 2022
Added Flex counter support for tunnel counters and rates implementation
kcudnik pushed a commit that referenced this pull request Sep 14, 2022
Added Flex counter support for tunnel counters and rates implementation
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
Added Flex counter support for tunnel counters and rates implementation
@dgsudharsan dgsudharsan deleted the evpn_vlan_tun_cntrs branch March 9, 2023 02:09
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.

None yet

3 participants