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

[swss] Only run tunnel packet handler on dualtor devices #11627

Merged
merged 1 commit into from
Aug 9, 2022

Conversation

theasianpianist
Copy link
Contributor

@theasianpianist theasianpianist commented Aug 4, 2022

Signed-off-by: Lawrence Lee lawlee@microsoft.com

Master branch duplicate of #11626

Why I did it

Tunnel packet handler running on non dualtor devices generates error logs

How I did it

At SWSS docker init time, check the device subtype and enable tunnel packet handler only if it is dualtor

How to verify it

  • Apply changes to dualtor device, verify that tunnel packet handler is running after restarting swss
  • Apply changes to non dualtor device, verify that tunnel packet handler is not running and does not show in supervisorctl status after restarting swss (verified on both T0 and T1 devices)
  • The SWSS docker built from this PR has been validated on both dualtor and non-dualtor devices

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
@theasianpianist theasianpianist changed the title [swss]: Run tunnel_pkt_handler on dualtor only [swss] Only run tunnel packet handler on dualtor devices Aug 4, 2022
@theasianpianist theasianpianist merged commit 4a996f3 into sonic-net:master Aug 9, 2022
yxieca pushed a commit that referenced this pull request Aug 11, 2022
At SWSS docker init time, check the device subtype and enable tunnel packet handler only if it is dualtor

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
skbarista pushed a commit to skbarista/sonic-buildimage that referenced this pull request Aug 17, 2022
At SWSS docker init time, check the device subtype and enable tunnel packet handler only if it is dualtor

Signed-off-by: Lawrence Lee <lawlee@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