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

[config-bcm] set bcm_tunnel_term_compatible_mode=1 for all platforms #1953

Closed
wants to merge 1 commit into from

Conversation

yxieca
Copy link
Contributor

@yxieca yxieca commented Aug 20, 2018

According to Broadcom:

  • This setting doesn't affect currently Broadcom SAI 3.1 behavior.
  • This setting is required for upcoming SAI version(s).

Signed-off-by: Ying Xie ying.xie@microsoft.com

- What I did
set bcm_tunnel_term_compatible_mode=1 for all platforms

- How I did it
This change is done by a bash script. The choice of location to insert might not be the best as result.

- How to verify it
I tested this change on a few platforms that I have access to. Did not exhausting all platforms.

According to Broadcom:
- This setting doesn't affect currently Broadcom SAI 3.1 behavior.
- This setting is required for upcoming SAI version(s).

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
@@ -1,5 +1,6 @@
#/******************************************************************************
# *
bcm_tunnel_term_compatible_mode=1
Copy link
Contributor

Choose a reason for hiding this comment

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

why this line of the configuration is inserted inside the comment header?

Copy link
Contributor Author

@yxieca yxieca Aug 20, 2018

Choose a reason for hiding this comment

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

It was added by a script. I didn't look closely at the end result. I can move it if you prefer.

@lguohan
Copy link
Collaborator

lguohan commented Aug 20, 2018

@yxieca , if this has to be enabled for every platform, we should ask broadcom add this into SAI. It is difficult to enforce this for all platform (new ones).

@yxieca
Copy link
Contributor Author

yxieca commented Aug 20, 2018

@lguohan I think you got a great point there. Let me talk to Broadcom.

@yxieca
Copy link
Contributor Author

yxieca commented Aug 23, 2018

Broadcom will take care of this in SAI.

@yxieca yxieca closed this Aug 23, 2018
liorghub added a commit to liorghub/sonic-buildimage that referenced this pull request Mar 23, 2022
Update sonic-swss submodule to include below commits:
d80094b [aclorch] Do not fail ACL rule remove flow if rule already deleted (sonic-net#2183)
bea0b70 [gcov]: Change coverage.xml file references (sonic-net#2120)
829b219 [tunnelmgrd]: Warm boot support (sonic-net#2166)
ad65b0a Fix issue: sometimes PFC WD unable to create zero buffer pool (sonic-net#2164)
608acc3 [doc] Moving Configuration.md from swss to yang sub-folder (sonic-net#2177)
0294376 [orchagent] NVGRE Tunnel orchestration agent implementation (sonic-net#1953)
ce88696 [ci] Update default sonic image downloading build ID. (sonic-net#2175)
liat-grozovik pushed a commit that referenced this pull request Mar 29, 2022
Update sonic-swss submodule to include below commits:
d80094b [aclorch] Do not fail ACL rule remove flow if rule already deleted (#2183)
bea0b70 [gcov]: Change coverage.xml file references (#2120)
829b219 [tunnelmgrd]: Warm boot support (#2166)
ad65b0a Fix issue: sometimes PFC WD unable to create zero buffer pool (#2164)
608acc3 [doc] Moving Configuration.md from swss to yang sub-folder (#2177)
0294376 [orchagent] NVGRE Tunnel orchestration agent implementation (#1953)
ce88696 [ci] Update default sonic image downloading build ID. (#2175)

Co-authored-by: liora <liora@nvidia.com>
Ndancejic pushed a commit to Ndancejic/sonic-buildimage that referenced this pull request May 3, 2022
…t#1953)

- What I did
Implemented the NVGRE Tunnel orchestration agent according to the - SONiC NVGRE Tunnel HLD

- Why I did it
Part of the NVGRE feature support.

- How I verified it
Implemented VS test.

Signed-off-by: Vadym Hlushko <vadymh@nvidia.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