Skip to content

[202503][PR:22679] add authtype for consoleserver#1052

Merged
bingwang-ms merged 2 commits intoAzure:202503from
sdszhang:console_ssh_202503
Mar 9, 2026
Merged

[202503][PR:22679] add authtype for consoleserver#1052
bingwang-ms merged 2 commits intoAzure:202503from
sdszhang:console_ssh_202503

Conversation

@sdszhang
Copy link

@sdszhang sdszhang commented Mar 7, 2026

Cherry-pick sonic-net/sonic-mgmt#22679 to 202503

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:38
Signed-off-by: Dashuai Zhang <dashuaizhang@microsoft.com>

# Conflicts:
#	tests/common/helpers/dut_utils.py
Signed-off-by: Dashuai Zhang <dashuaizhang@microsoft.com>
@sdszhang sdszhang changed the title [202503] add authtype for consoleserver [202503][PR:22679] add authtype for consoleserver Mar 7, 2026
@bingwang-ms bingwang-ms merged commit d704b3d into Azure:202503 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