<br/>
<br/>
<font color="#888888"><font color="#888888">Sent from my android device.</font></font><br/>
<br/>
-----Original Message-----<br/>
From: Chris Wopat <<a href="mailto:me@falz.net">me@falz.net</a>><br/>
To: <a href="mailto:rancid-discuss@shrubbery.net">rancid-discuss@shrubbery.net</a><br/>
Sent: Fri, 29 Sep 2017 17:52<br/>
Subject: [rancid] Juniper 'last commit' + Fortigate whitespace + DB<br/>
<br/>
Hey folks, <br/>
<br/>
<br/>
We recently updated to 3.7, a few comments/questions. <br/>
<br/>
<br/>
>* Juniper was updated to ignore 'last committed by' line. Changelog says 'useless last commit config line'. Curious what others think about this. We think it's<br/>
> quite valuable and is a nice way help correlate changes to accounts that made the change. For now we've manually restored it, which is easy enough.<br/>
<br/>
this sounds reasonable to me.<br/>
<br/>
<br/>
><font color="#000000">* We see the 'show chassis firmware' line come and go on some devices. This happened prior to 3.7 as well. This has been witnessed on MX running 15.1,</font><br/>
<font color="#000000">> QFX running 14.1X53-D45, and possibly other devices. </font><br/>
><br/>
><br/>
<font color="#000000">></font><font color="#000000">+ # show chassis firmware</font><br/>
<br/>
sounds like a race in the expect matching.<br/>
<br/>
><br/>
><br/>
><font color="#000000">* Fortigate suggestion: We manually add this to fnrancid still in GetSystem to keep the chatter down. </font>This was previously mentioned in this thread as well: <a href="http://www.shrubbery.net/pipermail/rancid-discuss/2017-June/009643.html">http://www.shrubbery.net/pipermail/rancid-discuss/2017-June/009643.html</a>, curious if others would like to see these added as well. <br/>
><br/>
><br/>
<font color="#000000">> </font>next if (/^\s*IPS-ETDB: .*/);<br/>
><br/>
> next if (/^\s*APP-DB: .*/);<br/>
><br/>
>next if (/^\s*IPS Malicious URL Database: .*/);<br/>
><br/>
>next if (/^\s*Botnet DB: .*/);<br/>
<br/>
 Sounds like a good contribution.<br/>
<br/>
<br/>