Skip to content

Commit 78ce6b9

Browse files
rodnymolinalguohan
authored andcommitted
Fixing a couple of issues to enable FRR to run with latest SONiC code. (#895)
* Adjusting FRR's jinja template to meet latest sonic-cfgen requirements. Basically, i'm just extending #448 changes into FRR. * Eliminate FRR's integrated-config file to prevent daemons from bypassing their own config files. FRR daemons now default to an integrated-config file for config-parsing purposes. But we are still interested in having each daemon looking in their specific config file (bgpd.conf, zebra.conf, etc). So here i'm just deleting this integrating-config file to prevent FRR from running from a bogus config-file. RB= G=lnos-reviewers R=ntrianta,rjonnadu,rmolina,sfardeen,zxu A=
1 parent c52e876 commit 78ce6b9

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

dockers/docker-fpm-frr/config.sh

+5
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,10 @@ sonic-cfggen -m /etc/sonic/minigraph.xml -t /usr/share/sonic/templates/unisolate
1212
chown root:root /usr/sbin/bgp-unisolate
1313
chmod 0755 /usr/sbin/bgp-unisolate
1414

15+
# If there's an integrated-config file, go ahead and remote it
16+
if [ -f /etc/frr/frr.conf ]; then
17+
rm -rf /etc/frr/frr.conf
18+
fi
19+
1520
mkdir -p /var/sonic
1621
echo "# Config files managed by sonic-config-engine" >/var/sonic/config_status

dockers/docker-fpm-frr/zebra.conf.j2

+3-3
Original file line numberDiff line numberDiff line change
@@ -15,12 +15,12 @@ enable password zebra
1515
{% block interfaces %}
1616
! Enable link-detect (default disabled)
1717
{% for interface in minigraph_interfaces %}
18-
interface {{ interface['alias'] }}
18+
interface {{ interface['attachto'] }}
1919
link-detect
2020
!
2121
{% endfor %}
22-
{% for interface in minigraph_portchannel_interfaces %}
23-
interface {{ interface['name'] }}
22+
{% for interface in minigraph_portchannels.keys() %}
23+
interface {{ interface }}
2424
link-detect
2525
!
2626
{% endfor %}

0 commit comments

Comments
 (0)