Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated with PAC CLI commands #1655

Merged
merged 3 commits into from
Apr 16, 2024

Conversation

vijaya-ops
Copy link
Contributor

Updated the PAC HLD with config , show and clear commands, that are listed below.

Config commands

config dot1x system-auth-control enable
config interface authentication port-control Ethernet0 auto
config interface authentication host-mode Ethernet0 multi-auth
config interface authentication order Ethernet0 dot1x
config interface authentication priority Ethernet0 dot1x
config interface dot1x pae Ethernet0 authenticator
config interface authentication periodic Ethernet1 enable
config interface authentication reauth-period Ethernet1 120
config interface authentication max-users Ethernet1 6
config interface mab Ethernet1 enable -a pap

Show commands

show authentication interface
show authentication clients
show mab interface
show dot1x

Clear commands

sonic-clear authentication sessions

Copy link

linux-foundation-easycla bot commented Apr 4, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@matiAlfaro
Copy link

It is possible to combine these PRs to a single one, maybe by using a tool such as this - combine-prs · Actions · GitHub Marketplace · GitHub?
It will make review much easier

@adyeung
Copy link
Collaborator

adyeung commented Apr 16, 2024

@matiAlfaro code PRs are split intentionally to manageable size for review and merge, this is to conform with the SONiC community practice

@sonic-net sonic-net deleted a comment from vijaya-ops Apr 16, 2024
@adyeung
Copy link
Collaborator

adyeung commented Apr 16, 2024

Code PR tracking table #1315

@adyeung adyeung merged commit f799bf2 into sonic-net:master Apr 16, 2024
1 check passed
@matiAlfaro
Copy link

@adyeung Got it, Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants