Fix: missing sonic-db-cli in docker-sonic-vs image by qiluo-msft · Pull Request #2357 · sonic-net/sonic-swss

qiluo-msft · 2022-06-27T19:17:04Z

What I did
Fix: missing sonic-db-cli in docker-sonic-vs image

Why I did it

How I verified it

Details if related

liuh-80 · 2022-06-28T00:48:20Z

/azp run

azure-pipelines · 2022-06-28T00:48:29Z

Azure Pipelines successfully started running 1 pipeline(s).

liuh-80 · 2022-06-28T04:53:25Z

/azp run

azure-pipelines · 2022-06-28T04:53:34Z

Azure Pipelines successfully started running 1 pipeline(s).

bingwang-ms · 2022-06-28T13:32:51Z

/azp run

azure-pipelines · 2022-06-28T13:33:03Z

Azure Pipelines successfully started running 1 pipeline(s).

qiluo-msft · 2022-06-28T22:58:02Z

/azpw run

mssonicbld · 2022-06-28T22:58:04Z

/AzurePipelines run

azure-pipelines · 2022-06-28T22:58:14Z

Azure Pipelines successfully started running 1 pipeline(s).

liuh-80 · 2022-06-29T05:26:11Z

/AzurePipelines run

azure-pipelines · 2022-06-29T05:26:21Z

Azure Pipelines successfully started running 1 pipeline(s).

liuh-80 · 2022-06-29T05:29:07Z

This PR blocked by some randomly failed UTs:

2022-06-29T05:12:36.1057643Z test_sub_port_intf.py::TestSubPortIntf::test_sub_port_intf_creation FAILED [ 82%]
2022-06-29T05:12:36.2205230Z test_sub_port_intf.py::TestSubPortIntf::test_sub_port_intf_add_ip_addrs FAILED [ 82%]
2022-06-29T05:12:36.2932181Z test_sub_port_intf.py::TestSubPortIntf::test_sub_port_intf_appl_db_proc_seq FAILED [ 82%]
2022-06-29T05:12:36.3673395Z test_sub_port_intf.py::TestSubPortIntf::test_sub_port_intf_admin_status_change FAILED [ 82%]
2022-06-29T05:12:36.4672083Z test_sub_port_intf.py::TestSubPortIntf::test_sub_port_intf_remove_ip_addrs FAILED [ 83%]
2022-06-29T05:12:36.5390488Z test_sub_port_intf.py::TestSubPortIntf::test_sub_port_intf_removal FAILED [ 83%]
2022-06-29T05:12:36.6127161Z test_sub_port_intf.py::TestSubPortIntf::test_sub_port_intf_mtu FAILED [ 83%]
2022-06-29T05:12:36.7097591Z test_sub_port_intf.py::TestSubPortIntf::test_sub_port_intf_nhg_accel FAILED [ 83%]
2022-06-29T05:12:36.7853144Z test_sub_port_intf.py::TestSubPortIntf::test_sub_port_intf_oper_down_with_pending_neigh_route_tasks FAILED [ 83%]

2022-06-28T21:50:18.0131695Z _________________ TestSubPortIntf.test_sub_port_intf_creation __________________
2022-06-28T21:50:18.0131990Z
2022-06-28T21:50:18.0132733Z self = <test_sub_port_intf.TestSubPortIntf object at 0x7f29ce750550>
2022-06-28T21:50:18.0133202Z dvs = <conftest.DockerVirtualSwitch object at 0x7f29ceacdbb0>
2022-06-28T21:50:18.0133402Z
2022-06-28T21:50:18.0133771Z def test_sub_port_intf_creation(self, dvs):
2022-06-28T21:50:18.0135106Z > self.connect_dbs(dvs)
2022-06-28T21:50:18.0136417Z
2022-06-28T21:50:18.0136685Z test_sub_port_intf.py:585:
2022-06-28T21:50:18.0137137Z _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
2022-06-28T21:50:18.0137489Z test_sub_port_intf.py:86: in connect_dbs
2022-06-28T21:50:18.0137840Z self.default_vrf_oid = self.get_default_vrf_oid()
2022-06-28T21:50:18.0138213Z _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
2022-06-28T21:50:18.0138395Z
2022-06-28T21:50:18.0138720Z self = <test_sub_port_intf.TestSubPortIntf object at 0x7f29ce750550>
2022-06-28T21:50:18.0138949Z
2022-06-28T21:50:18.0139206Z def get_default_vrf_oid(self):
2022-06-28T21:50:18.0139657Z oids = self.get_oids(ASIC_VIRTUAL_ROUTER_TABLE)
2022-06-28T21:50:18.0140093Z > assert len(oids) == 1, "Wrong # of default vrfs: %d, expected #: 1." % (len(oids))
2022-06-28T21:50:18.0140508Z E AssertionError: Wrong # of default vrfs: 2, expected #: 1.
2022-06-28T21:50:18.0140833Z E assert 2 == 1
2022-06-28T21:50:18.0141416Z E -2
2022-06-28T21:50:18.0141662Z E +1

liuh-80 · 2022-06-29T09:03:05Z

/AzurePipelines run

azure-pipelines · 2022-06-29T09:03:16Z

Azure Pipelines successfully started running 1 pipeline(s).

liuh-80 · 2022-06-29T12:50:33Z

The cli issue been fixed by this PR: sonic-net/sonic-buildimage#11282
However, this PR still need complete and merge, after that, we can add the reverted PR back.

liuh-80 · 2022-06-29T12:50:41Z

/AzurePipelines run

azure-pipelines · 2022-06-29T12:50:54Z

Azure Pipelines successfully started running 1 pipeline(s).

liuh-80 · 2022-06-30T01:59:01Z

/AzurePipelines run

azure-pipelines · 2022-06-30T01:59:12Z

Azure Pipelines successfully started running 1 pipeline(s).

liuh-80 · 2022-06-30T07:16:08Z

/AzurePipelines run

azure-pipelines · 2022-06-30T07:16:18Z

Azure Pipelines successfully started running 1 pipeline(s).

liuh-80 · 2022-07-01T03:47:58Z

/AzurePipelines run

azure-pipelines · 2022-07-01T03:48:08Z

Azure Pipelines successfully started running 1 pipeline(s).

liuh-80 · 2022-07-04T00:00:26Z

/azp run

azure-pipelines · 2022-07-04T00:00:36Z

Azure Pipelines successfully started running 1 pipeline(s).

liuh-80 · 2022-07-06T03:08:52Z

/azp run

azure-pipelines · 2022-07-06T03:09:01Z

Azure Pipelines successfully started running 1 pipeline(s).

*Fix: missing sonic-db-cli in docker-sonic-vs image

Fix: missing sonic-db-cli in docker-sonic-vs image

976686a

qiluo-msft requested a review from prsunny as a code owner June 27, 2022 19:17

qiluo-msft requested review from bingwang-ms and yxieca June 27, 2022 19:20

yxieca approved these changes Jun 27, 2022

View reviewed changes

prsunny approved these changes Jun 28, 2022

View reviewed changes

bingwang-ms approved these changes Jun 28, 2022

View reviewed changes

liuh-80 added the Request for 202111 Branch label Jun 30, 2022

Merge branch 'master' into qiluo/fixcli

31da762

Merge branch 'master' into qiluo/fixcli

920396d

prsunny merged commit f565f7d into sonic-net:master Jul 7, 2022

qiluo-msft deleted the qiluo/fixcli branch July 7, 2022 23:18

liuh-80 mentioned this pull request Aug 1, 2022

sonic-db-cli high CPU usage on SONiC startup sonic-net/sonic-buildimage#10218

Closed

liuh-80 added the Request for 202205 Branch label Aug 1, 2022

preetham-singh pushed a commit to preetham-singh/sonic-swss that referenced this pull request Aug 6, 2022

Fix: missing sonic-db-cli in docker-sonic-vs image (sonic-net#2357)

622c181

*Fix: missing sonic-db-cli in docker-sonic-vs image

yxieca added the Included in 202205 Branch label Aug 8, 2022

yxieca pushed a commit that referenced this pull request Aug 8, 2022

Fix: missing sonic-db-cli in docker-sonic-vs image (#2357)

450e9ec

*Fix: missing sonic-db-cli in docker-sonic-vs image

liushilongbuaa pushed a commit to liushilongbuaa/sonic-swss that referenced this pull request Mar 20, 2023

Fix: missing sonic-db-cli in docker-sonic-vs image (sonic-net#2357)

ee29587

*Fix: missing sonic-db-cli in docker-sonic-vs image

Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025

Fix: missing sonic-db-cli in docker-sonic-vs image (sonic-net#2357)

873a638

*Fix: missing sonic-db-cli in docker-sonic-vs image

Conversation

qiluo-msft commented Jun 27, 2022

Uh oh!

liuh-80 commented Jun 28, 2022

Uh oh!

azure-pipelines bot commented Jun 28, 2022

Uh oh!

liuh-80 commented Jun 28, 2022

Uh oh!

azure-pipelines bot commented Jun 28, 2022

Uh oh!

bingwang-ms commented Jun 28, 2022

Uh oh!

azure-pipelines bot commented Jun 28, 2022

Uh oh!

qiluo-msft commented Jun 28, 2022

Uh oh!

mssonicbld commented Jun 28, 2022

Uh oh!

azure-pipelines bot commented Jun 28, 2022

Uh oh!

liuh-80 commented Jun 29, 2022

Uh oh!

azure-pipelines bot commented Jun 29, 2022

Uh oh!

liuh-80 commented Jun 29, 2022

Uh oh!

liuh-80 commented Jun 29, 2022

Uh oh!

azure-pipelines bot commented Jun 29, 2022

Uh oh!

liuh-80 commented Jun 29, 2022

Uh oh!

liuh-80 commented Jun 29, 2022

Uh oh!

azure-pipelines bot commented Jun 29, 2022

Uh oh!

liuh-80 commented Jun 30, 2022

Uh oh!

azure-pipelines bot commented Jun 30, 2022

Uh oh!

liuh-80 commented Jun 30, 2022

Uh oh!

azure-pipelines bot commented Jun 30, 2022

Uh oh!

liuh-80 commented Jul 1, 2022

Uh oh!

azure-pipelines bot commented Jul 1, 2022

Uh oh!

liuh-80 commented Jul 4, 2022

Uh oh!

azure-pipelines bot commented Jul 4, 2022

Uh oh!

liuh-80 commented Jul 6, 2022

Uh oh!

azure-pipelines bot commented Jul 6, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants