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

[configdb]: Fixed a bug that leads to the emerging of records with "None" value in redis-db #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Mrak-IW
Copy link

@Mrak-IW Mrak-IW commented Dec 27, 2017

What I did:
Fixed a bug that leads to the emerging of records with "None" value in redis-db
Particularly, bgp_asn = "None" inside the DEVICE_METADATA|localhost entry
leads bgp daemon to crash

How to verify it:
Load a minigraph without BGP settings. Check DEVICE_METADATA|localhost in redis-db. You mustn't see bgp_asn field.
Or try to add to the config_db.json any other configuration field with the value "null" and check if it has appeared in redis-db.

Signed-off-by: Denis Maslov Denis.Maslov@cavium.com

…one" value in redis-db

Particularly, bgp_asn = "None" inside the DEVICE_METADATA|localhost entry
leads bgp daemon to crash

Signed-off-by: Denis Maslov <Denis.Maslov@cavium.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.

2 participants