Conversation
…erfaces #4536, adjust comments
| ! Zebra configuration saved from vty | ||
| ! 2017/03/03 20:21:04 | ||
| ! | ||
| {% if helpers.exists('OPNsense.quagga.general') %} |
There was a problem hiding this comment.
As information why this has been removed:
frr defaults and log syslog only need to be declared once in the frr.conf file, which is why in zebra.conf these lines exist, but in the other conf files they have been removed.
| ! | ||
| ! | ||
| ! | ||
| {% set passive_interfaces = [] %} |
There was a problem hiding this comment.
This here is the passive interface rewrite to make it compatible with frr-reload.
| @@ -1,13 +1,7 @@ | |||
| bfdd.conf:/usr/local/etc/frr/bfdd.conf | |||
There was a problem hiding this comment.
I might be missing something, but how this file populated now? (same as the ones below)
There was a problem hiding this comment.
The file will not be generated anymore. It is now part of frr.conf as include and will be generated inside of it.
There was a problem hiding this comment.
(face-palm) for some reason I misread the includes, these are jinja includes, thanks for pointing that out
|
I would suggest removing the old conf files via setup.sh |
In my tests they were ignored and only frr.conf was used. I want to try without deleting anything first. |
|
If something comes up in my final tests I will create a new PR for file deletion. |
3 participants
Fixes: #4510
Fixes: #4534
Requires the new rc.d script from: opnsense/ports@f005f9c