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 route flow counter #576

Merged
merged 2 commits into from
Feb 16, 2022

Conversation

Junchao-Mellanox
Copy link
Collaborator

What I did?

Add new table names for route flow counter feature.

HLD:
sonic-net/SONiC#908

keboliu
keboliu previously approved these changes Jan 20, 2022
@Junchao-Mellanox
Copy link
Collaborator Author

Hi @prsunny , could you please review this? It is a small change which only contains new schema definitions.

common/schema.h Outdated Show resolved Hide resolved
prsunny
prsunny previously approved these changes Jan 26, 2022
@prsunny
Copy link
Contributor

prsunny commented Jan 26, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik
Copy link
Collaborator

/azp run Azure.sonic-swss-common

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 576 in repo Azure/sonic-swss-common

@Junchao-Mellanox
Copy link
Collaborator Author

/azp run Azure.sonic-swss-common

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 576 in repo Azure/sonic-swss-common

@Junchao-Mellanox
Copy link
Collaborator Author

Hi @prsunny , could you please help re-trigger the test? Thanks.

@Junchao-Mellanox
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Junchao-Mellanox
Copy link
Collaborator Author

Did a rebase, hope it could fix the vstest issue.

@Junchao-Mellanox
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Junchao-Mellanox
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Junchao-Mellanox
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Junchao-Mellanox
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Junchao-Mellanox
Copy link
Collaborator Author

/azpw run Azure.sonic-swss-common

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-swss-common

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny prsunny merged commit 9eac0ae into sonic-net:master Feb 16, 2022
@Junchao-Mellanox Junchao-Mellanox deleted the route-flow-counter branch February 17, 2022 05:59
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.

5 participants