isis: T4333: migrate to new vyos_defined Jinja2 test

1 files changed, 78 insertions, 88 deletions

diff --git a/data/templates/frr/isisd.frr.tmpl b/data/templates/frr/isisd.frr.tmpl
index c68dda443..2bf3a3b8a 100644
--- a/data/templates/frr/isisd.frr.tmpl
+++ b/data/templates/frr/isisd.frr.tmpl
@@ -1,46 +1,46 @@
 !
-{% if interface is defined and interface is not none %}
+{% if interface is vyos_defined %}
 {%   for iface, iface_config in interface.items() %}
 interface {{ iface }}
  ip router isis VyOS
  ipv6 router isis VyOS
-{%     if iface_config.bfd is defined %}
+{%     if iface_config.bfd is vyos_defined %}
  isis bfd
-{%       if iface_config.bfd.profile is defined and iface_config.bfd.profile is not none %}
+{%       if iface_config.bfd.profile is vyos_defined %}
  isis bfd profile {{ iface_config.bfd.profile }}
 {%       endif %}
 {%     endif %}
-{%     if iface_config.network is defined and iface_config.network.point_to_point is defined %}
+{%     if iface_config.network.point_to_point is vyos_defined %}
  isis network point-to-point
 {%     endif %}
-{%     if iface_config.circuit_type is defined %}
+{%     if iface_config.circuit_type is vyos_defined %}
  isis circuit-type {{ iface_config.circuit_type }}
 {%     endif %}
-{%     if iface_config.hello_interval is defined and iface_config.hello_interval is not none %}
+{%     if iface_config.hello_interval is vyos_defined %}
  isis hello-interval {{ iface_config.hello_interval }}
 {%     endif %}
-{%     if iface_config.hello_multiplier is defined and iface_config.hello_multiplier is not none %}
+{%     if iface_config.hello_multiplier is vyos_defined %}
  isis hello-multiplier {{ iface_config.hello_multiplier }}
 {%     endif %}
-{%     if iface_config.hello_padding is defined %}
+{%     if iface_config.hello_padding is vyos_defined %}
  isis hello padding
 {%     endif %}
-{%     if iface_config.metric is defined and iface_config.metric is not none %}
+{%     if iface_config.metric is vyos_defined %}
  isis metric {{ iface_config.metric }}
 {%     endif %}
-{%     if iface_config.passive is defined %}
+{%     if iface_config.passive is vyos_defined %}
  isis passive
 {%     endif %}
-{%     if iface_config.password is defined and iface_config.password.plaintext_password is defined and iface_config.password.plaintext_password is not none %}
+{%     if iface_config.password.plaintext_password is vyos_defined %}
  isis password clear {{ iface_config.password.plaintext_password }}
 {%     endif %}
-{%     if iface_config.priority is defined and iface_config.priority is not none %}
+{%     if iface_config.priority is vyos_defined %}
  isis priority {{ iface_config.priority }}
 {%     endif %}
-{%     if iface_config.psnp_interval is defined and iface_config.psnp_interval is not none %}
+{%     if iface_config.psnp_interval is vyos_defined %}
  isis psnp-interval {{ iface_config.psnp_interval }}
 {%     endif %}
-{%     if iface_config.no_three_way_handshake is defined %}
+{%     if iface_config.no_three_way_handshake is vyos_defined %}
  no isis three-way-handshake
 {%     endif %}
 exit
@@ -48,98 +48,94 @@ exit
 {%   endfor %}
 {% endif %}
 !
-router isis VyOS {{ 'vrf ' + vrf if vrf is defined and vrf is not none }}
+router isis VyOS {{ 'vrf ' + vrf if vrf is vyos_defined }}
  net {{ net }}
-{% if dynamic_hostname is defined %}
+{% if dynamic_hostname is vyos_defined %}
  hostname dynamic
 {% endif %}
-{% if purge_originator is defined %}
+{% if purge_originator is vyos_defined %}
  purge-originator
 {% endif %}
-{% if set_attached_bit is defined %}
+{% if set_attached_bit is vyos_defined %}
  set-attached-bit
 {% endif %}
-{% if set_overload_bit is defined %}
+{% if set_overload_bit is vyos_defined %}
  set-overload-bit
 {% endif %}
-{% if domain_password is defined and domain_password is not none %}
-{%   if domain_password.md5 is defined and domain_password.md5 is not none %}
+{% if domain_password.md5 is vyos_defined %}
  domain-password md5 {{ domain_password.plaintext_password }}
-{%   elif domain_password.plaintext_password is defined and domain_password.plaintext_password is not none %}
+{% elif domain_password.plaintext_password is vyos_defined %}
  domain-password clear {{ domain_password.plaintext_password }}
-{%   endif %}
 {% endif %}
-{% if log_adjacency_changes is defined %}
+{% if log_adjacency_changes is vyos_defined %}
  log-adjacency-changes
 {% endif %}
-{% if lsp_gen_interval is defined and lsp_gen_interval is not none %}
+{% if lsp_gen_interval is vyos_defined %}
  lsp-gen-interval {{ lsp_gen_interval }}
 {% endif %}
-{% if lsp_mtu is defined and lsp_mtu is not none %}
+{% if lsp_mtu is vyos_defined %}
  lsp-mtu {{ lsp_mtu }}
 {% endif %}
-{% if lsp_refresh_interval is defined and lsp_refresh_interval is not none %}
+{% if lsp_refresh_interval is vyos_defined %}
  lsp-refresh-interval  {{ lsp_refresh_interval }}
 {% endif %}
-{% if max_lsp_lifetime is defined and max_lsp_lifetime is not none %}
+{% if max_lsp_lifetime is vyos_defined %}
  max-lsp-lifetime {{ max_lsp_lifetime }}
 {% endif %}
-{% if spf_interval is defined and spf_interval is not none %}
+{% if spf_interval is vyos_defined %}
  spf-interval {{ spf_interval }}
 {% endif %}
-{% if traffic_engineering is defined and traffic_engineering is not none %}
-{%   if traffic_engineering.enable is defined %}
+{% if traffic_engineering.enable is vyos_defined %}
  mpls-te on
-{%   endif %}
-{%   if traffic_engineering.address is defined %}
+{% endif %}
+{% if traffic_engineering.address is vyos_defined %}
  mpls-te router-address {{ traffic_engineering.address }}
+{% endif %}
+{% if traffic_engineering.inter_as is vyos_defined %}
+{%   set level = '' %}
+{%   if traffic_engineering.inter_as.level_1 is vyos_defined %}
+{%     set level = ' level-1' %}
 {%   endif %}
-{%   if traffic_engineering.inter_as is defined %}
-{%     if traffic_engineering.inter_as.level_1 is defined %}
- mpls-te inter-as level-1
-{%     endif %}
-{%     if traffic_engineering.inter_as.level_1_2 is defined %}
- mpls-te inter-as level-1-2
-{%     endif %}
-{%     if traffic_engineering.inter_as.level_2 is defined %}
- mpls-te inter-as level-2-only
-{%     endif %}
-{%   else %}
- mpls-te inter-as
+{%   if traffic_engineering.inter_as.level_1_2 is vyos_defined %}
+{%     set level = ' level-1-2' %}
+{%   endif %}
+{%   if traffic_engineering.inter_as.level_2 is vyos_defined %}
+{%     set level = ' level-2-only' %}
 {%   endif %}
+ mpls-te inter-as{{ level }}
 {% endif %}
-{% if segment_routing is defined %}
-{%   if segment_routing.enable is defined %}
+{% if segment_routing is vyos_defined %}
+{%   if segment_routing.enable is vyos_defined %}
  segment-routing on
 {%   endif %}
-{%   if segment_routing.maximum_label_depth is defined %}
+{%   if segment_routing.maximum_label_depth is vyos_defined %}
  segment-routing node-msd {{ segment_routing.maximum_label_depth }}
 {%   endif %}
-{%   if segment_routing.global_block is defined %}
+{%   if segment_routing.global_block is vyos_defined %}
  segment-routing global-block {{ segment_routing.global_block.low_label_value }} {{ segment_routing.global_block.high_label_value }}
 {%   endif %}
-{%   if segment_routing.local_block is defined %}
+{%   if segment_routing.local_block is vyos_defined %}
  segment-routing local-block {{ segment_routing.global_block.low_label_value }} {{ segment_routing.local_block.high_label_value }}
 {%   endif %}
-{%   if segment_routing.prefix is defined %}
+{%   if segment_routing.prefix is vyos_defined %}
 {%     for prefixes in segment_routing.prefix %}
-{%       if segment_routing.prefix[prefixes].absolute is defined %}
-{%         if segment_routing.prefix[prefixes].absolute.value is defined %}
+{%       if segment_routing.prefix[prefixes].absolute is vyos_defined %}
+{%         if segment_routing.prefix[prefixes].absolute.value is vyos_defined %}
  segment-routing prefix {{ prefixes }} absolute {{ segment_routing.prefix[prefixes].absolute.value }}
-{%           if segment_routing.prefix[prefixes].absolute.explicit_null is defined %}
+{%           if segment_routing.prefix[prefixes].absolute.explicit_null is vyos_defined %}
  segment-routing prefix {{ prefixes }} absolute {{ segment_routing.prefix[prefixes].absolute.value }} explicit-null
 {%           endif %}
-{%           if segment_routing.prefix[prefixes].absolute.no_php_flag is defined %}
+{%           if segment_routing.prefix[prefixes].absolute.no_php_flag is vyos_defined %}
  segment-routing prefix {{ prefixes }} absolute {{ segment_routing.prefix[prefixes].absolute.value }} no-php-flag
 {%           endif %}
 {%         endif %}
-{%         if segment_routing.prefix[prefixes].index is defined %}
-{%           if segment_routing.prefix[prefixes].index.value is defined %}
+{%         if segment_routing.prefix[prefixes].index is vyos_defined %}
+{%           if segment_routing.prefix[prefixes].index.value is vyos_defined %}
  segment-routing prefix {{ prefixes }} index {{ segment_routing.prefix[prefixes].index.value }}
-{%             if segment_routing.prefix[prefixes].index.explicit_null is defined %}
+{%             if segment_routing.prefix[prefixes].index.explicit_null is vyos_defined %}
  segment-routing prefix {{ prefixes }} index {{ segment_routing.prefix[prefixes].index.value }} explicit-null
 {%             endif %}
-{%             if segment_routing.prefix[prefixes].index.no_php_flag is defined %}
+{%             if segment_routing.prefix[prefixes].index.no_php_flag is vyos_defined %}
  segment-routing prefix {{ prefixes }} index {{ segment_routing.prefix[prefixes].index.value }} no-php-flag
 {%             endif %}
 {%           endif %}
@@ -148,57 +144,51 @@ router isis VyOS {{ 'vrf ' + vrf if vrf is defined and vrf is not none }}
 {%     endfor %}
 {%   endif %}
 {% endif %}
-{% if spf_delay_ietf is defined and spf_delay_ietf.init_delay is defined and spf_delay_ietf.init_delay is not none %}
+{% if spf_delay_ietf.init_delay is vyos_defined %}
  spf-delay-ietf init-delay {{ spf_delay_ietf.init_delay }} short-delay {{ spf_delay_ietf.short_delay }} long-delay {{ spf_delay_ietf.long_delay }} holddown {{ spf_delay_ietf.holddown }} time-to-learn {{ spf_delay_ietf.time_to_learn }}
 {% endif %}
-{% if area_password is defined and area_password is not none %}
-{%   if area_password.md5 is defined and area_password.md5 is not none %}
+{% if area_password.md5 is vyos_defined %}
  area-password md5 {{ area_password.md5 }}
-{%   elif area_password.plaintext_password is defined and area_password.plaintext_password is not none %}
+{% elif area_password.plaintext_password is vyos_defined %}
  area-password clear {{ area_password.plaintext_password }}
-{%   endif %}
 {% endif %}
-{% if default_information is defined and default_information.originate is defined and default_information.originate is not none %}
+{% if default_information.originate is vyos_defined %}
 {%   for afi, afi_config in default_information.originate.items() %}
 {%     for level, level_config in afi_config.items() %}
  default-information originate {{ afi }} {{ level | replace('_', '-') }} {{ 'always' if level_config.always is defined }} {{ 'route-map ' ~ level_config.route_map if level_config.route_map is defined }} {{ 'metric ' ~ level_config.metric if level_config.metric is defined }}
 {%     endfor %}
 {%   endfor %}
 {% endif %}
-{% if redistribute is defined %}
-{%   if redistribute.ipv4 is defined and redistribute.ipv4 is not none %}
-{%     for protocol, protocol_options in redistribute.ipv4.items() %}
-{%       for level, level_config in protocol_options.items() %}
-{%         if level_config.metric is defined and level_config.metric is not none %}
+{% if redistribute.ipv4 is vyos_defined %}
+{%   for protocol, protocol_options in redistribute.ipv4.items() %}
+{%     for level, level_config in protocol_options.items() %}
+{%       if level_config.metric is vyos_defined %}
  redistribute ipv4 {{ protocol }} {{ level | replace('_', '-') }} metric {{ level_config.metric }}
-{%         elif level_config.route_map is defined and level_config.route_map is not none %}
+{%       elif level_config.route_map is vyos_defined %}
  redistribute ipv4 {{ protocol }} {{ level | replace('_', '-') }} route-map {{ level_config.route_map }}
-{%         else %}
+{%       else %}
  redistribute ipv4 {{ protocol }} {{ level | replace('_', '-') }}
-{%         endif %}
-{%       endfor %}
+{%       endif %}
 {%     endfor %}
-{%   endif %}
-{%   if redistribute.ipv6 is defined and redistribute.ipv6 is not none %}
-{%     for protocol, protocol_options in redistribute.ipv6.items() %}
-{%       for level, level_config in protocol_options.items() %}
-{%         if level_config.metric is defined and level_config.metric is not none %}
+{%   endfor %}
+{% endif %}
+{% if redistribute.ipv6 is vyos_defined %}
+{%   for protocol, protocol_options in redistribute.ipv6.items() %}
+{%     for level, level_config in protocol_options.items() %}
+{%       if level_config.metric is defined and level_config.metric is not none %}
  redistribute ipv6 {{ protocol }} {{ level | replace('_', '-') }} metric {{ level_config.metric }}
-{%         elif level_config.route_map is defined and level_config.route_map is not none %}
+{%       elif level_config.route_map is defined and level_config.route_map is not none %}
  redistribute ipv6 {{ protocol }} {{ level | replace('_', '-') }} route-map {{ level_config.route_map }}
-{%         else %}
+{%       else %}
  redistribute ipv6 {{ protocol }} {{ level | replace('_', '-') }}
-{%         endif %}
-{%       endfor %}
+{%       endif %}
 {%     endfor %}
-{%   endif %}
+{%   endfor %}
 {% endif %}
-{% if level is defined and level is not none %}
-{%   if level == 'level-2'  %}
+{% if level is vyos_defined('level-2') %}
  is-type level-2-only
-{%   else %}
+{% elif level is vyos_defined %}
  is-type {{ level }}
-{%   endif %}
 {% endif %}
 exit
 !
\ No newline at end of file