Skip to content

Force use unix domain socket when namespace not empty #797

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.

Sign up for GitHub

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

Merged
qiluo-msft merged 7 commits into from
Sep 11, 2023
Merged

Force use unix domain socket when namespace not empty #797

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
c2b282c
Force use UDS when namespace not empty
liuh-80 Jun 16, 2023
e71877a
Merge branch 'master' into dev/liuh/fix-cli-connection
qiluo-msft Jun 29, 2023
5a52108
Fix code issue
liuh-80 Jul 14, 2023
598d77c
Merge remote-tracking branch 'origin' into dev/liuh/fix-cli-connection
liuh-80 Jul 14, 2023
cfddc57
Fix UT socket config issue
liuh-80 Jul 14, 2023
90ed89f
Merge remote-tracking branch 'origin' into dev/liuh/fix-cli-connection
liuh-80 Aug 3, 2023
e04a24c
Fix PR comments
liuh-80 Aug 30, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 4 additions & 7 deletions sonic-db-cli/sonic-db-cli.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -284,14 +284,11 @@ int sonic_db_cli(
if (!netns.empty())
{
initializeGlobalConfig();

// Use the unix domain connectivity if namespace not empty.
isTcpConn = false;
}
else
{
// Use the tcp connectivity if namespace is local and unixsocket cmd_option is present.
isTcpConn = true;
netns = "";
}


if (options.m_cmd.size() != 0)
{
auto commands = options.m_cmd;
Expand Down