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

Reg: Sequential IDF isolation: show runningconfiguration BGP has missing CHECK_IDF_ISOLATION Route_Map in the public image of Master #19380

Closed
vikshaw-Nokia opened this issue Jun 24, 2024 · 0 comments · Fixed by #19752
Assignees

Comments

@vikshaw-Nokia
Copy link

Description

Reg: Sequential IDF isolation: show runningconfiguration BGP has missing CHECK_IDF_ISOLATION Route_Map in the public image of Master
Please see the reference below
For the PR: https://github.com/sonic-net/sonic-buildimage/pull/19136/files
In the Latest Master Image.
In the bgp running configuration. The CHECK_IDF_ISOLATION call statement is MISSING.
image
So after the policy is applied for any isolated_withdraw_all isolated_no_export. The Running Configuration has the changes for the configuration but the missing call statement makes it ineffective

Steps to reproduce the issue:

  1. On the Latest Master Image
  2. After the fix for the Issue: Reg: Sequential IDF isolation skipping condition is based on switch type and not type leading to skipping of the feature #19379
  3. Execute the IDF Isolation for any of the type
  4. Observe the runningconfiguration bgp

Describe the results you received:

  1. The Running Configuration has the changes for CHECK_IDF_ISOLATION but the missing call statement makes it ineffective

Describe the results you expected:

  1. The Running Configuration should have the missing call statement in the Route Maps

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

@rlhui rlhui closed this as completed in 0b5f71f Aug 29, 2024
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this issue Sep 6, 2024
Fixes sonic-net#19380
To enable idf_isolation route-map towards BGP peer
mssonicbld pushed a commit that referenced this issue Sep 7, 2024
Fixes #19380
To enable idf_isolation route-map towards BGP peer
vvolam pushed a commit to vvolam/sonic-buildimage that referenced this issue Sep 12, 2024
Fixes sonic-net#19380
To enable idf_isolation route-map towards BGP peer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants