aristanetworks / avd

Arista Validated Designs
https://avd.arista.com
Apache License 2.0
299 stars 219 forks source link

Feat(eos_cli_config_gen): extend aaa_accounting with options for dot1x #2343

Closed dmilyt closed 1 year ago

dmilyt commented 1 year ago

Enhancement summary

aaa_accounting global configuration will be extended with missing options for dot1x accounting

Which component of AVD is impacted

eos_cli_config_gen

Use case example

aaa accounting dot1x default start-stop group < group-name > logging

Describe the solution you would like

aaa_accounting:
  dot1x:
    default:
      type: < none | start-stop | stop-only >
      group: < group_name >
      logging: < true | false >

Describe alternatives you have considered

This can be accomplished today with cvp configlet or eos_cli:.

Additional context

I would like to provide the implementation on this.

Contributing Guide

ClausHolbechArista commented 1 year ago

Agree on the data model. There are some other options like having multiple groups or multicast in the cli, but we don't cover that on any of the others, so I think this is fine for now.