Skip to content

[static route] Ignore BGP summary failure#4154

Merged
shi-su merged 1 commit intosonic-net:masterfrom
shi-su:static_ignore_bgp
Sep 2, 2021
Merged

[static route] Ignore BGP summary failure#4154
shi-su merged 1 commit intosonic-net:masterfrom
shi-su:static_ignore_bgp

Conversation

@shi-su
Copy link
Copy Markdown
Contributor

@shi-su shi-su commented Aug 31, 2021

Description of PR

Summary: Ignore BGP summary failure
Fixes # (issue)

Type of change

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

Back port request

  • 201911

Approach

What is the motivation for this PR?

The command show ip bgp summary may fail if there are no BGP sessions up. Since static route feature has no dependency on BGP, the test cases should continue even if the command fails.

How did you do it?

Ignore failure on show ip bgp summary command.

How did you verify/test it?

Any platform specific information?

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

Documentation

@shi-su shi-su requested a review from a team as a code owner August 31, 2021 23:42
@shi-su shi-su requested a review from prsunny August 31, 2021 23:43
@shi-su shi-su merged commit 6e98e95 into sonic-net:master Sep 2, 2021
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…lly (#25637)

#### Why I did it
src/sonic-swss
```
* dbe0768e - (HEAD -> master, origin/master, origin/HEAD) Fix the "Invalid BUFFER QUEUE" error (sonic-net#4224) (6 hours ago) [saksarav-nokia]
* d29d7ea1 - This patch is to fix a major bug in the for loop of the function AclRange::remove. (sonic-net#4256) (9 hours ago) [ashishalok-arista]
* 35288a65 - Fix performance issue in fpmsyncd for non-ZMQ configuration (sonic-net#4219) (9 hours ago) [venkit-nexthop]
* 1c384fb6 - Merge pull request sonic-net#4022 from ksravani-hcl/swss_1 (12 hours ago) [StephenWangGoogle]
|\ 
| failure_prs.log skip_prs.log 36d98d61 - Enable response pipeline for P4Orch. (22 hours ago) [mint570]
|/ 
* 63860664 - [build] Add docker-based build environment (sonic-net#3715) (22 hours ago) [Lawrence Lee]
* 26ec7561 - [buffermgrd] Wait for lossless buffer profile to be applied to SAI (sonic-net#4154) (23 hours ago) [ganglv]
* 599eae77 - [tunnel] use warning message when setting a create-only attribute (sonic-net#4212) (27 hours ago) [Yakiv Huryk]
* ebf8c73f - [orchagent] Fix getopt string for -R option (ring_thread_enabled regression) (sonic-net#4207) (27 hours ago) [Chiranjeevi U - MapleLabs]
* aa081017 - Avoid using an uninitialized source MAC address (sonic-net#4201) (28 hours ago) [Andriy Yurkiv]
* 2a258c2 - Added mux neighbor_mode support with prefix-route (sonic-net#4152) (31 hours ago) [manamand2020]
* 9616cd52 - [ci] Install redis from Debian (sonic-net#4241) (34 hours ago) [Saikrishna Arcot]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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