Skip to content

BGP Default Policies README.md#2272

Merged
sachendras merged 9 commits into
mainfrom
sachendras-patch-17
Oct 20, 2023
Merged

BGP Default Policies README.md#2272
sachendras merged 9 commits into
mainfrom
sachendras-patch-17

Conversation

@sachendras
Copy link
Copy Markdown
Contributor

@sachendras sachendras commented Oct 20, 2023

Summary

Following expectation for default-policies at the peer-group and neighbor levels for BGP

  • Default import and export policies are expected to be functional for both IBGP and EBGP peers when policy definition in the policy chain is not satisfied.
  • For IBGP and EBGP peers, when no policy is attached, actions defined in the default-policy (when one exists) should apply.
  • For IBGP peers when no policy is attached including the default-policy, default should be import and export all BGP routes.
  • For EBGP peers when no policy is attached includign the default-policy, default should be to disallow import and export of all BGP routes.
  • In all the above cases, BGP default-policy is applicable only to the BGP learnt routes or the routes that are redistributed in to BGP from other protocols. Import/Export of routes from other protocols should be governed by their respective policies and should not either be exported or imported based on the actions of the BGP default-policy.

@OpenConfigBot
Copy link
Copy Markdown

OpenConfigBot commented Oct 20, 2023

Pull Request Functional Test Report for #2272 / fdd7366

No tests identified for validation.

Help

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 20, 2023

Pull Request Test Coverage Report for Build 6590342051

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 46.333%

Totals Coverage Status
Change from base Build 6581405337: 0.0%
Covered Lines: 1409
Relevant Lines: 3041

💛 - Coveralls

Comment thread feature/bgp/policybase/otg_tests/default_policies_test/README.md Outdated
@dplore dplore marked this pull request as ready for review October 20, 2023 22:13
@dplore dplore requested review from a team as code owners October 20, 2023 22:13
@dplore dplore requested review from a team and michaelore October 20, 2023 22:13
@sachendras sachendras merged commit eda47af into main Oct 20, 2023
@sachendras sachendras deleted the sachendras-patch-17 branch October 20, 2023 23:45
frasieroh pushed a commit to aristanetworks/openconfig-featureprofiles that referenced this pull request Oct 21, 2023
* BGP Default Policies README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update feature/bgp/policybase/otg_tests/default_policies_test/README.md

Co-authored-by: Darren Loher <dloher@google.com>

* Update README.md

* Update README.md

---------

Co-authored-by: Darren Loher <dloher@google.com>
frasieroh pushed a commit to aristanetworks/openconfig-featureprofiles that referenced this pull request Oct 30, 2023
* BGP Default Policies README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update feature/bgp/policybase/otg_tests/default_policies_test/README.md

Co-authored-by: Darren Loher <dloher@google.com>

* Update README.md

* Update README.md

---------

Co-authored-by: Darren Loher <dloher@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants