Skip to content

[EVERFLOW]: Add EVERFLOW policer test with DSCP value/mask#932

Merged
stcheng merged 1 commit intosonic-net:masterfrom
stcheng:policer
Jun 25, 2019
Merged

[EVERFLOW]: Add EVERFLOW policer test with DSCP value/mask#932
stcheng merged 1 commit intosonic-net:masterfrom
stcheng:policer

Conversation

@stcheng
Copy link
Contributor

@stcheng stcheng commented May 29, 2019

Signed-off-by: Shu0T1an ChenG shuche@microsoft.com

Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
@stcheng stcheng merged commit 55830ea into sonic-net:master Jun 25, 2019
@stcheng stcheng deleted the policer branch June 25, 2019 22:33

- name: Create a policer enforced mirror session
shell: |
config mirror_session add {{policer_session_name}} {{session_src_ip}} {{session_dst_ip}} {{session_dscp}} {{session_ttl}} --policer {{policer_name}}
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @stcheng . It seems that on public master and 201811 images this command is not supported (see below). Is sonic-utilities submodule pointer going to be updated soon?

Log:

"cmd": "config mirror_session add TEST_POLICER_SESSION 1.1.1.1 2.2.2.2 8 1 --policer TEST_POLICER"
"stderr": "Error: no such option: --policer",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mykolaf I'll update the sonic-utilities pointer very soon

stcheng added a commit to stcheng/sonic-mgmt that referenced this pull request Jul 15, 2019
…#932)

Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
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.

3 participants