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

[vxlanmgrd]: Add vxlanmgrd start command #2705

Merged
merged 7 commits into from
Apr 24, 2019
Merged

[vxlanmgrd]: Add vxlanmgrd start command #2705

merged 7 commits into from
Apr 24, 2019

Conversation

Pterosaur
Copy link
Contributor

@Pterosaur Pterosaur commented Mar 26, 2019

  • What I did
    Make vxlanmgrd will automatically start in swss container.

  • How I did it

  • How to verify it
    Do config reload -y

  • Description for the changelog

Submodule update pointers:

d612d5e - [vxlanmgrd]: Add vxlanmgrd (#807)
fd5ad50 - Add fake_platform environment
eb30871 - [vnetorch]: Fix incorrect creation flow for bitmap VNET (#849)
186fdce - [teammgr] use boolean value for active/fallback in teamd conf (#793)
3a2f60d - Create ingress table group during the PFCWD stats list installment (#815)
cc6f8f6 - [fpmsyncd] Improve VNET routes support (#838)
a64f9ad - [vnetorch]: Add VNET interface removal flow for Bitmap VNET implement\u2026 (#820)
f8be024 - [routeorch]: Move extra log into INFO level (#846)
98aa960 - modify the string from IPV6_Neighbor to IPV6_NEIGHBOR for crmResTypeNameMap. (#828)
0179917 - [vs]: Add vs test support for bitmap VNET (#792)
3a80898 - [fpmsyncd]: Remove redundant new line character and trailing spaces (#834
7d6ae0e - [fpmsyncd] align port variable type with sockaddr_in.sin_port (#831)
640547a - [Pfc-wd]: Add pfc-wd support for nephos (#817)

Vxlan need the brctl tool to manage bridge for it

Signed-off-by: Pterosaur <ganze718@gmail.com>
- Add vxlanmgrd to supervisorctl in docker -orchagent

Signed-off-by: Ze Gan zegan@microsoft.com
- Add vxlanmgrd to supervisorctl in vs

Signed-off-by: Ze Gan zegan@microsoft.com
Signed-off-by: Ze Gan zegan@microsoft.com
@lguohan
Copy link
Collaborator

lguohan commented Mar 29, 2019

retest this please

@lguohan lguohan requested a review from prsunny April 1, 2019 08:20
Copy link
Contributor

@prsunny prsunny left a comment

Choose a reason for hiding this comment

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

Once swss changes for VxlanMgrD is merged, we can update the submodule to this PR before final commit.

@lguohan
Copy link
Collaborator

lguohan commented Apr 9, 2019

what is the dependent PR for this PR?

@Pterosaur
Copy link
Contributor Author

what is the dependent PR for this PR?

I think the PR(sonic-net/sonic-swss#807) is dependent on this PR. Because vxlanmgrd cannot automatically start if this PR isn't merged, which will make the PR(sonic-net/sonic-swss#807) cannot pass the testing.
Because we change the table name from cfg* to app* in swss Orch, so that the app db will lose the information of vnet and vxlan_tunnel without the vxlanmgrd.

@prsunny prsunny merged commit 2e86caa into sonic-net:master Apr 24, 2019
MichelMoriniaux pushed a commit to criteo-forks/sonic-buildimage that referenced this pull request May 28, 2019
* Add bridge-utils to orchagent image

- Add vxlanmgrd to supervisorctl in docker -orchagent

Signed-off-by: Ze Gan zegan@microsoft.com

* Update submodule pointer for swss to include Vxlanmgrd changes
yxieca pushed a commit that referenced this pull request Mar 20, 2023
Why I did it
src/sonic-dbsyncd

* 4dcfb61 - (HEAD -> 202205, origin/202205) Handled the error case of negative age (#57) (2 days ago) [Vivek]
src/sonic-swss

* 142abdf - (HEAD -> 202205, origin/202205) swss: Fix egress queue counters in voq systems. (#2705) (2 days ago) [Sambath Kumar Balasubramanian]
src/sonic-utilities

* 42a57f4c - (HEAD -> 202205, origin/202205) [202205] Update the ref guide to reflect the vlan brief output (#2741) (2 days ago) [Vivek]
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