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

[copp] bind copp-config.service to sonic.target #8969

Merged
merged 1 commit into from
Oct 14, 2021
Merged

Conversation

yxieca
Copy link
Contributor

@yxieca yxieca commented Oct 13, 2021

Why I did it

Noticed that on T0 testbed, the dhcp copp rule might be missing from the copp config.

How I did it

copp-config service needs to be started after sonic.target so that it could render the copp-config with the latest information.

It also needs to be restarted when config reload or load_minigraph is invoked.

How to verify it

Tested on a testbed hit this issue.

Signed-off-by: Ying Xie [email protected]

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

copp-config service needs to be started after sonic.target so that it could
render the copp-config with the latest information.

It also needs to be restarted when config reload or load_minigraph is invoked.

Signed-off-by: Ying Xie <[email protected]>
@yxieca yxieca merged commit 638c287 into sonic-net:master Oct 14, 2021
@yxieca yxieca deleted the copp branch October 14, 2021 04:07
judyjoseph pushed a commit that referenced this pull request Oct 14, 2021
copp-config service needs to be started after sonic.target so that it could
render the copp-config with the latest information.

It also needs to be restarted when config reload or load_minigraph is invoked.

Signed-off-by: Ying Xie <[email protected]>
qiluo-msft pushed a commit that referenced this pull request Oct 15, 2021
copp-config service needs to be started after sonic.target so that it could
render the copp-config with the latest information.

It also needs to be restarted when config reload or load_minigraph is invoked.

Signed-off-by: Ying Xie <[email protected]>
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.

5 participants