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

Test for quagga livelock fix #2751

Merged
merged 5 commits into from
Apr 9, 2019
Merged

Test for quagga livelock fix #2751

merged 5 commits into from
Apr 9, 2019

Conversation

pavel-shirshov
Copy link
Contributor

- What I did
Added a test for quagga livelock fix. See sonic-net/sonic-quagga#32

- How I did it
Create a bgp session with r-bit set to 1 (default for exabgp) and wait until r-bit got inactive in quagga.

- How to verify it
Build an image and run the test

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@pavel-shirshov
Copy link
Contributor Author

The vs check must fail for now because this PR doesn't have the sonic-net/sonic-quagga#32 yet. I'll add it after the test shows it can catch the quagga livelock issue.

@pavel-shirshov
Copy link
Contributor Author

retest this please

@pavel-shirshov
Copy link
Contributor Author

vs test failed as expected with old version of quagga.
Now I've updated bgpd to the latest version and we run the test again.

@lguohan lguohan merged commit d6cf075 into master Apr 9, 2019
lguohan pushed a commit that referenced this pull request Apr 10, 2019
* Test for quagga livelock fix

* Create /usr/local/etc for the test

* Add more debug info

* Install specific version of exabgp

* Update sonic-quagga
yxieca pushed a commit that referenced this pull request Apr 16, 2019
* Test for quagga livelock fix

* Create /usr/local/etc for the test

* Add more debug info

* Install specific version of exabgp

* Update sonic-quagga
@pavel-shirshov pavel-shirshov deleted the pavelsh/quagga_open_test branch May 9, 2019 17:47
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