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.

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

Jump to bottom

readis-cli: set app db ,but no change to asic db #1067

Closed
zhuzhuzhu23 opened this issue Oct 23, 2017 · 2 comments
Closed

readis-cli: set app db ,but no change to asic db #1067

zhuzhuzhu23 opened this issue Oct 23, 2017 · 2 comments
Assignees
@qiluo-msft
Labels
Question ❓

Comments

@zhuzhuzhu23
Copy link

zhuzhuzhu23 commented Oct 23, 2017 •
edited by qiluo-msft
Loading

app db:
127.0.0.1:6379> hgetall "NEIGH_TABLE:Ethernet0:10.0.0.1"

  1. "neigh"
  2. "00:01:04:4C:49:F6"
  3. "family"
  4. "IPv4"

I deleted field "neigh".
127.0.0.1:6379> hgetall "NEIGH_TABLE:Ethernet0:10.0.0.1"

  1. "family"
  2. "IPv4"

and I queried asic db.But，not change to this key。
127.0.0.1:6379[1]> hgetall "ASIC_STATE:SAI_OBJECT_TYPE_NEIGHBOR_ENTRY:{"ip":"10.0.0.1","rif":"oid:0x6000000000135"}"

  1. "SAI_NEIGHBOR_ATTR_DST_MAC_ADDRESS"
  2. "00:01:04:4C:49:F6"

So do I have a problem with my operation? And I want to know how to look up with log of app db and asic db.

Thank you.
@qiluo-msft
Copy link
Collaborator

Hi @zhuzhuzhu23, you are not supposed to manipulate ApplDB contents directly. Roughly speaking, applications use ApplDB to interprocess communication. The providers are using ProducerStateTable, and the consumer (orchagent) is using ConsuerStateTable. The table you saw is manipulated by ConsuerStateTable after popping the data from the queue, and it is for state saving purpose only.

@qiluo-msft
Copy link
Collaborator

There is no specific log related to your ask. You need to check related components logs to know what happens. In your case, there is no components involved.

abdosi added a commit that referenced this issue Sep 3, 2020
@abdosi
[submodule update] sonic-util
c480db1 
Fix error when running 'show lldp table' or 'show lldp neighbor'
through SSH command. (#1067)
[201911]: Multi asic show interface support (#1070)
[counterpoll] Add new FC group for port buffer drop counters
(#1024)
[201911] show interface portchannel support for Multi ASIC
(#1071)
Fix a typo in mellanox_buffer_migrator (#1090)
Fix pfcwd stats crash with invalid queue name (#1077)
[PFCWD] Fix issue with "pfcwd show stats" command during SONiC init
(#1018)
enable watchdog before running platform specific reboot plugin
(#1037)
Add namespace of the process in the coredump filename. (#1091)
@qiwang4 qiwang4 mentioned this issue Jul 22, 2022
Merged
7 tasks
taras-keryk pushed a commit to taras-keryk/sonic-buildimage that referenced this issue Jul 22, 2022
@jimmyzhai
[ci] Add option --privileged for running slave docker (sonic-net#1067)
bc3487e 
In azp run, the following failure always happens at the stage of `make check` of building syncd. 

```
Making check in syncd
make[2]: Entering directory '/__w/1/s/syncd'
make check-TESTS
make[3]: Entering directory '/__w/1/s/syncd'
tests: tests.cpp:843: void test_watchdog_timer_clock_rollback(): Assertion `settimeofday(&currentTime, NULL) == 0' failed.
/bin/bash: line 5: 13004 Aborted (core dumped) ${dir}$tst
FAIL: tests
```
The execution of `settimeofday(&currentTime, NULL)` fails in slave docker with errno **EPERM**, because CAP_SYS_TIME capability is dropped in docker. Using option `--privileged` gives docker extended privileges for its success.

This failure has existed for a long time in azp build and is not exposed till sonic-net/sonic-sairedis#1050.
taras-keryk pushed a commit to taras-keryk/sonic-buildimage that referenced this issue Jul 22, 2022
@jimmyzhai
[pipeline] Set CAP_SYS_TIME for settimeofday ok in syncd test (sonic-…
bd47844 
…net#1068)

The fix sonic-net#1067 is not enough. If docker user is non-root, set capability CAP_SYS_TIME for settimeofday success in syncd test, then test_watchdog_timer_clock_rollback can be run.

Co-authored-by: junhuazhai <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qiluo-msft qiluo-msft

Labels
Question ❓
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jleveque @qiluo-msft @zhuzhuzhu23