Refactor(eos_cli_config_gen): Improve the aaa accounting j2 template

ansible_collections/arista/avd/molecule/eos_cli_config_gen/documentation/devices/host1.md

@@ -278,12 +278,16 @@ aaa authorization commands 10,15 default group tacacs+ local
 | Commands - Console | all | start-stop | TACACS | True |
 | Commands - Console | 0 | start-stop |  -  | True |
 | Commands - Console | 1 | start-stop | TACACS1 | False |
+| Commands - Console | 2 | none |  -  | True |
+| Commands - Console | 3 | start-stop |  -  | False |
 | Exec - Default | - | start-stop | TACACS | True |
 | System - Default | - | start-stop | TACACS | - |
 | Dot1x - Default  | - | start-stop | RADIUS | - |
 | Commands - Default | all | start-stop | TACACS | True |
 | Commands - Default | 0 | start-stop | - | True |
 | Commands - Default | 1 | start-stop | TACACS | False |
+| Commands - Default | 2 | none | - | True |
+| Commands - Default | 3 | start-stop | - | False |
 
 #### AAA Accounting Device Configuration
 
@@ -292,10 +296,12 @@ aaa accounting exec console start-stop group TACACS logging
 aaa accounting commands all console start-stop group TACACS logging
 aaa accounting commands 0 console start-stop logging
 aaa accounting commands 1 console start-stop group TACACS1
+aaa accounting commands 2 console none
 aaa accounting exec default start-stop group TACACS logging
 aaa accounting system default start-stop group TACACS
 aaa accounting dot1x default start-stop group RADIUS
 aaa accounting commands all default start-stop group TACACS logging
 aaa accounting commands 0 default start-stop logging
 aaa accounting commands 1 default start-stop group TACACS
+aaa accounting commands 2 default none
 ```

ansible_collections/arista/avd/molecule/eos_cli_config_gen/intended/configs/host1.cfg

@@ -81,12 +81,14 @@ aaa accounting exec console start-stop group TACACS logging
 aaa accounting commands all console start-stop group TACACS logging
 aaa accounting commands 0 console start-stop logging
 aaa accounting commands 1 console start-stop group TACACS1
+aaa accounting commands 2 console none
 aaa accounting exec default start-stop group TACACS logging
 aaa accounting system default start-stop group TACACS
 aaa accounting dot1x default start-stop group RADIUS
 aaa accounting commands all default start-stop group TACACS logging
 aaa accounting commands 0 default start-stop logging
 aaa accounting commands 1 default start-stop group TACACS
+aaa accounting commands 2 default none
 !
 interface Management1
    description OOB_MANAGEMENT

ansible_collections/arista/avd/molecule/eos_cli_config_gen/inventory/host_vars/host1/aaa-accounting.yml

@@ -31,6 +31,14 @@ aaa_accounting:
         type: start-stop
         logging: false
         group: TACACS1
+      # As type is none hence logging will not render in config as per actual EOS
+      - commands: 2
+        type: none
+        logging: true
+      # If type is start-stop or stop-only then logging should be true or group should be defined to render the config
+      - commands: 3
+        type: start-stop
+        logging: false
     default:
       - commands: all
         type: start-stop
@@ -44,3 +52,11 @@ aaa_accounting:
         type: start-stop
         logging: false
         group: TACACS
+      # As type is none hence logging will not render in config as per actual EOS
+      - commands: 2
+        type: none
+        logging: true
+      # If type is start-stop or stop-only then logging should be true or group should be defined to render the config
+      - commands: 3
+        type: start-stop
+        logging: false

python-avd/pyavd/_eos_cli_config_gen/j2templates/eos/aaa-accounting.j2

@@ -5,49 +5,45 @@
 #}
 {# eos - aaa accounting #}
 {% if aaa_accounting is arista.avd.defined %}
-{%     if aaa_accounting.exec.console.type is arista.avd.defined %}
-{%         set aaa_accounting_exec_console_cli = "aaa accounting exec console" %}
-{%         if aaa_accounting.exec.console.type is arista.avd.defined('none') %}
-{%             set aaa_accounting_exec_console_cli = aaa_accounting_exec_console_cli ~ " none" %}
-{%         else %}
-{%             set aaa_accounting_exec_console_cli = aaa_accounting_exec_console_cli ~ " " ~ aaa_accounting.exec.console.type %}
-{%             if aaa_accounting.exec.console.group is arista.avd.defined %}
-{%                 set aaa_accounting_exec_console_cli = aaa_accounting_exec_console_cli ~ " group " ~ aaa_accounting.exec.console.group %}
-{%             endif %}
-{%             if aaa_accounting.exec.console.logging is arista.avd.defined(true) %}
-{%                 set aaa_accounting_exec_console_cli = aaa_accounting_exec_console_cli ~ " logging" %}
-{%             endif %}
+{%     if aaa_accounting.exec.console.type is arista.avd.defined('none') %}
+aaa accounting exec console none
+{%     elif aaa_accounting.exec.console.group is arista.avd.defined or aaa_accounting.exec.console.logging is arista.avd.defined(true) %}
+{%         set exec_console_cli = "aaa accounting exec console " ~ aaa_accounting.exec.console.type %}
+{%         if aaa_accounting.exec.console.group is arista.avd.defined %}
+{%             set exec_console_cli = exec_console_cli ~ " group " ~ aaa_accounting.exec.console.group %}
+{%         endif %}
+{%         if aaa_accounting.exec.console.logging is arista.avd.defined(true) %}
+{%             set exec_console_cli = exec_console_cli ~ " logging" %}
 {%         endif %}
-{{ aaa_accounting_exec_console_cli }}
+{{ exec_console_cli }}
 {%     endif %}
 {%     if aaa_accounting.commands.console is arista.avd.defined %}
 {%         for command_default in aaa_accounting.commands.console %}
-{%             if command_default.commands is arista.avd.defined and command_default.type is arista.avd.defined %}
-{%                 set aaa_accounting_commands_commands_console_cli = "aaa accounting commands " ~ command_default.commands ~ " console " ~ command_default.type %}
+{%             if command_default.type is arista.avd.defined('none') %}
+aaa accounting commands {{ command_default.commands }} console none
+{%             elif command_default.group is arista.avd.defined or command_default.logging is arista.avd.defined(true) %}
+{%                 set commands_console_cli = "aaa accounting commands " ~ command_default.commands ~ " console " ~ command_default.type %}
 {%                 if command_default.group is arista.avd.defined %}
-{%                     set aaa_accounting_commands_commands_console_cli = aaa_accounting_commands_commands_console_cli ~ " group " ~ command_default.group %}
+{%                     set commands_console_cli = commands_console_cli ~ " group " ~ command_default.group %}
 {%                 endif %}
 {%                 if command_default.logging is arista.avd.defined(true) %}
-{%                     set aaa_accounting_commands_commands_console_cli = aaa_accounting_commands_commands_console_cli ~ " logging" %}
+{%                     set commands_console_cli = commands_console_cli ~ " logging" %}
 {%                 endif %}
+{{ commands_console_cli }}
 {%             endif %}
-{{ aaa_accounting_commands_commands_console_cli }}
 {%         endfor %}
 {%     endif %}
-{%     if aaa_accounting.exec.default.type is arista.avd.defined %}
-{%         set aaa_accounting_exec_default_cli = "aaa accounting exec default" %}
-{%         if aaa_accounting.exec.default.type is arista.avd.defined('none') %}
-{%             set aaa_accounting_exec_default_cli = aaa_accounting_exec_default_cli ~ " none" %}
-{%         else %}
-{%             set aaa_accounting_exec_default_cli = aaa_accounting_exec_default_cli ~ " " ~ aaa_accounting.exec.default.type %}
-{%             if aaa_accounting.exec.default.group is arista.avd.defined %}
-{%                 set aaa_accounting_exec_default_cli = aaa_accounting_exec_default_cli ~ " group " ~ aaa_accounting.exec.default.group %}
-{%             endif %}
-{%             if aaa_accounting.exec.default.logging is arista.avd.defined(true) %}
-{%                 set aaa_accounting_exec_default_cli = aaa_accounting_exec_default_cli ~ " logging" %}
-{%             endif %}
+{%     if aaa_accounting.exec.default.type is arista.avd.defined('none') %}
+aaa accounting exec default none
+{%     elif aaa_accounting.exec.default.group is arista.avd.defined or aaa_accounting.exec.default.logging is arista.avd.defined(true) %}
+{%         set exec_default_cli = "aaa accounting exec default " ~ aaa_accounting.exec.default.type %}
+{%         if aaa_accounting.exec.default.group is arista.avd.defined %}
+{%             set exec_default_cli = exec_default_cli ~ " group " ~ aaa_accounting.exec.default.group %}
+{%         endif %}
+{%         if aaa_accounting.exec.default.logging is arista.avd.defined(true) %}
+{%             set exec_default_cli = exec_default_cli ~ " logging" %}
 {%         endif %}
-{{ aaa_accounting_exec_default_cli }}
+{{ exec_default_cli }}
 {%     endif %}
 {%     if aaa_accounting.system.default.type is arista.avd.defined and aaa_accounting.system.default.group is arista.avd.defined %}
 aaa accounting system default {{ aaa_accounting.system.default.type }} group {{ aaa_accounting.system.default.group }}
@@ -57,16 +53,18 @@ aaa accounting dot1x default {{ aaa_accounting.dot1x.default.type }} group {{ aa
 {%     endif %}
 {%     if aaa_accounting.commands.default is arista.avd.defined %}
 {%         for command_default in aaa_accounting.commands.default %}
-{%             if command_default.commands is arista.avd.defined and command_default.type is arista.avd.defined %}
-{%                 set aaa_accounting_commands_commands_default_cli = "aaa accounting commands " ~ command_default.commands ~ " default " ~ command_default.type %}
+{%             if command_default.type is arista.avd.defined('none') %}
+aaa accounting commands {{ command_default.commands }} default none
+{%             elif command_default.group is arista.avd.defined or command_default.logging is arista.avd.defined(true) %}
+{%                 set commands_default_cli = "aaa accounting commands " ~ command_default.commands ~ " default " ~ command_default.type %}
 {%                 if command_default.group is arista.avd.defined %}
-{%                     set aaa_accounting_commands_commands_default_cli = aaa_accounting_commands_commands_default_cli ~ " group " ~ command_default.group %}
+{%                     set commands_default_cli = commands_default_cli ~ " group " ~ command_default.group %}
 {%                 endif %}
 {%                 if command_default.logging is arista.avd.defined(true) %}
-{%                     set aaa_accounting_commands_commands_default_cli = aaa_accounting_commands_commands_default_cli ~ " logging" %}
+{%                     set commands_default_cli = commands_default_cli ~ " logging" %}
 {%                 endif %}
+{{ commands_default_cli }}
 {%             endif %}
-{{ aaa_accounting_commands_commands_default_cli }}
 {%         endfor %}
 {%     endif %}
 {% endif %}