Skip to content

Remove the bgp_v6 check, which adds failure since the IPv6 BGP doesn'…#8746

Merged
saiarcot895 merged 1 commit intosonic-net:masterfrom
saiarcot895:really-disable-ipv6-check
Jun 28, 2023
Merged

Remove the bgp_v6 check, which adds failure since the IPv6 BGP doesn'…#8746
saiarcot895 merged 1 commit intosonic-net:masterfrom
saiarcot895:really-disable-ipv6-check

Conversation

@saiarcot895
Copy link
Copy Markdown
Contributor

…t come up

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@saiarcot895 saiarcot895 merged commit 5db3a06 into sonic-net:master Jun 28, 2023
@saiarcot895 saiarcot895 deleted the really-disable-ipv6-check branch June 28, 2023 21:55
saiarcot895 added a commit to saiarcot895/sonic-mgmt that referenced this pull request Jul 12, 2023
saiarcot895 added a commit that referenced this pull request Jul 14, 2023
* Revert "Temporarily disable checking IPv6 BGP routing on neighbors for warm-reboot (#8735)"

This reverts commit ff4bbf3.

* Revert "Remove the bgp_v6 check, which adds failure since the IPv6 BGP doesn't come up (#8746)"

This reverts commit 5db3a06.
mrkcmo pushed a commit to Azarack/sonic-mgmt that referenced this pull request Oct 3, 2023
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Jan 25, 2024
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Jan 25, 2024
* Revert "Temporarily disable checking IPv6 BGP routing on neighbors for warm-reboot (sonic-net#8735)"

This reverts commit ff4bbf3.

* Revert "Remove the bgp_v6 check, which adds failure since the IPv6 BGP doesn't come up (sonic-net#8746)"

This reverts commit 5db3a06.
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.

2 participants