Skip to content

[202412][PR: 22679] add authtype for consoleserver#1053

Merged
r12f merged 2 commits intoAzure:202412from
sdszhang:console_ssh_202412
Mar 9, 2026
Merged

[202412][PR: 22679] add authtype for consoleserver#1053
r12f merged 2 commits intoAzure:202412from
sdszhang:console_ssh_202412

Conversation

@sdszhang
Copy link

@sdszhang sdszhang commented Mar 7, 2026

cherry-pick sonic-net/sonic-mgmt#22679 into 202412

Description of PR

Summary:
support specifying authentication credential choice for console server (e.g. TACACS or AAD),

With the following device:

Hostname,ManagementIp,HwSku,Type,Protocol,Os,AuthType
console-1,192.168.10.1/23,Cisco,ConsoleServer,ssh,sonic,,
console-2,192.168.10.2/23,Cisco,ConsoleServer,ssh,sonic,,tacacs

console-1 will use the default credentials in secrets.yaml "console_login".
console-2 will use more specific credentials in secrets.yaml "console_login_options['tacacs']"

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Approach

What is the motivation for this PR?

user can use different credential for console server.

How did you do it?

introduced new console_type, CONSOLE_SSH_TACACS

How did you verify/test it?

Verified locally.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

sdszhang added 2 commits March 7, 2026 22:49
Signed-off-by: Dashuai Zhang <[email protected]>

# Conflicts:
#	tests/common/helpers/dut_utils.py
Signed-off-by: Dashuai Zhang <[email protected]>
@sdszhang sdszhang changed the title [PR: 22679][202412] add authtype for consoleserver [202412][PR: 22679] add authtype for consoleserver Mar 7, 2026
@r12f r12f merged commit e5bb9c5 into Azure:202412 Mar 9, 2026
3 checks passed
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.

2 participants