bgp: T7760: remove per vrf instance system-as node

VyOS 1.5 and onwards will no longer have the following CLI node available:
  set vrf name <name> protocols bgp system-as <asn>

Author: c-po

interface-definitions/include/version/bgp-version.xml.i

@@ -1,3 +1,3 @@
 <!-- include start from include/version/bgp-version.xml.i -->
-<syntaxVersion component='bgp' version='7'></syntaxVersion>
+<syntaxVersion component='bgp' version='8'></syntaxVersion>
 <!-- include end -->

interface-definitions/vrf.xml.in

@@ -57,18 +57,6 @@
                 </properties>
                 <children>
                   #include <include/bgp/protocol-common-config.xml.i>
-                  <leafNode name="system-as">
-                    <properties>
-                      <help>Autonomous System Number (ASN) - DEPRECATED</help>
-                      <valueHelp>
-                        <format>u32:1-4294967294</format>
-                        <description>Autonomous System Number</description>
-                      </valueHelp>
-                      <constraint>
-                        <validator name="numeric" argument="--range 1-4294967294"/>
-                      </constraint>
-                    </properties>
-                  </leafNode>
                 </children>
               </node>
               <node name="eigrp" owner="${vyos_conf_scripts_dir}/protocols_eigrp.py $VAR(../../@)">

src/conf_mode/protocols_bgp.py

@@ -18,7 +18,6 @@
 from sys import argv
 
 from vyos.base import Warning
-from vyos.base import DeprecationWarning
 from vyos.config import Config
 from vyos.configverify import has_frr_protocol_in_dict
 from vyos.configverify import verify_prefix_list
@@ -219,13 +218,6 @@ def verify(config_dict):
         if not system_as:
             raise ConfigError(ERR_MSG_GLOBAL_VRF_AS_MISSING)
 
-        if 'system_as' in bgp:
-            tmp_as = bgp['system_as']
-            DeprecationWarning(f'CLI command "vrf name {vrf} protocols bgp system-as ' \
-                               f'{tmp_as}" is ignored and will be removed in VyOS 1.5! ' \
-                               f'\n\nGlobal "protocols bgp system-as {system_as}" option ' \
-                                'applies, use per neighbor "local-as" option to override.')
-
     elif 'system_as' not in bgp:
         raise ConfigError(ERR_MSG_GLOBAL_VRF_AS_MISSING)
 

src/migration-scripts/bgp/7-to-8

@@ -0,0 +1,30 @@
+#!/usr/bin/env python3
+#
+# Copyright VyOS maintainers and contributors <[email protected]>
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License version 2 or later as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+# T7760: Remove per VRF setting for system-as option in VyOS 1.5 and onwards
+
+from vyos.configtree import ConfigTree
+
+def migrate(config: ConfigTree) -> None:
+    vrf_base = ['vrf', 'name']
+    if not config.exists(vrf_base):
+        return
+
+    for vrf in config.list_nodes(vrf_base):
+        # bail out early if there is no per VRF BGP instance defined
+        vrf_bgp_base = vrf_base + [vrf, 'protocols', 'bgp']
+        if config.exists(vrf_bgp_base + ['system-as']):
+            config.delete(vrf_bgp_base + ['system-as'])