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
bgp: T6106: Show complete FRR output on internal errors #3151
Conversation
python/vyos/frr.py
Outdated
LOG.debug(f'frr-reload output: {i:3} {e}') | ||
if code == 1: | ||
if emsg: | ||
raise ConfigError(emsg) | ||
|
||
raise CommitError('FRR configuration failed while running commit. Please ' \ | ||
'enable debugging to examine logs.\n\n\n' \ | ||
'To enable debugging run: "touch /tmp/vyos.frr.debug" ' \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should make people create files by hand. It's trivial to add an operational mode command that does it, I think we should do it.
python/vyos/frr.py
Outdated
except: | ||
# we just need to re-try the commit of the configuration | ||
# for the listed FRR issues above | ||
pass | ||
if count >= count_max: | ||
if emsg: | ||
raise ConfigError(emsg) | ||
raise ConfigurationNotValid(f'Config commit retry counter ({count_max}) exceeded for {daemon} dameon!') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/dameon/daemon/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
…ned in peer-group handle vtysh bgp error
@Mergifyio backport sagitta |
✅ Backports have been created
|
So with this merged it seems like CI is broken.
I also wonder, are those error messages the "last resort" and the implemented errors in verify() are used first? If so, can you please also add a proper verify() section for this issue of internal neighbors with a better error message? |
handle vtysh bgp error
Change Summary
Types of changes
Related Task(s)
Related PR(s)
Component(s) name
Proposed changes
How to test
Smoketest result
Checklist: