]> git.puffer.fish Git - mirror/frr.git/commit
Make vtysh-integrated-config truly the default and fix quagga reload for this.
authorDinesh G Dutt <ddutt@cumulusnetworks.com>
Wed, 20 Jul 2016 15:24:47 +0000 (08:24 -0700)
committerDinesh G Dutt <ddutt@cumulusnetworks.com>
Wed, 20 Jul 2016 16:14:27 +0000 (09:14 -0700)
commit6ac9179ca77086c96a3c0a5de03420ec8b830db3
tree64ef055cad90fb0ab8c99b15d4029e47f55f5b2c
parent46827ae98a459e157803df6acdaf5c0ebc51f140
Make vtysh-integrated-config truly the default and fix quagga reload for this.

Ticket: CM-11910
Reviewed By: sharpd, routing-dev slack
Testing Done: Test with nothing in vtysh.conf, add no, remove it etc.

Even though we force integrated config to be the default, we do this by adding
a line to our default vtysh.conf which has integrated config enabled. When we
stopped printing integrated-config as part of wr mem or show running-config, we
broke quagga reload because it was explicitly looking for integrated config.
Furthermore, merely fixing quagga reload wouldn't work because subsequent saves
would result in config being saved to individual files since vtysh.conf no
longer forced the file to be integrated.

This patch fixes both issues. Makes integrated config the default in the code,
rather than via a shipped default file, and fixes quagga reload to look for
the "no integrated-vtysh-config" to deny attempting a reload.
tools/quagga-reload.py
vtysh/vtysh.c