Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Config DB is incomplete right after config load_minigraph #2044

Closed
andriymoroz-mlnx opened this issue Sep 13, 2018 · 1 comment
Closed

Config DB is incomplete right after config load_minigraph #2044

andriymoroz-mlnx opened this issue Sep 13, 2018 · 1 comment
Assignees

Comments

@andriymoroz-mlnx
Copy link
Collaborator

Description

Due to counters start delay if “config save” is called right after “config load_minigraph” Config DBs on disk and in memory will be different ("FLEX_COUNTER_TABLE" will be missing)

Steps to reproduce the issue:

  1. config load_minigraph
  2. config save (right after load_minigraph)
  3. wait 3 min
  4. rename /etc/sonic/config_db.json
  5. config save
  6. compare new /etc/sonic/config_db.json and renamed in step 4

Describe the results you received:
DB saved later will be different. "FLEX_COUNTER_TABLE"

Describe the results you expected:
Saved DBs to be the same

Additional information you deem important (e.g. issue happens only occasionally):

**Output of show version:**
SONiC Software Version: SONiC.HEAD.6-6cf2d10
Distribution: Debian 8.11
Kernel: 3.16.0-5-amd64
Build commit: 6cf2d10
Build date: Thu Sep  6 18:06:54 UTC 2018
Built by: johnar@jenkins-worker-3

**Attach debug file `sudo generate_dump`:**

```
(paste your output here)
```
@yxieca
Copy link
Contributor

yxieca commented Sep 19, 2019

I think the diff came 3 minutes later was due to the buffer profiles that are generated by the buffermgrd. Save them or not doesn't hurt next boot. I think it is ok.

@yxieca yxieca closed this as completed Sep 19, 2019
wen587 added a commit that referenced this issue Mar 8, 2022
47c243e [show][muxcable] fix the sudo access error for show muxcable metrics (#2083)
f872516 [muxcable][show] enhance show mux status to show last switchover time (#2067)
d440df7 [warmboot] Migrate 10G ports during warm-reboot on s6100 (#2064)
494c6d7 [counterpoll] Display the correct default poll interval for watermark counters (#2082)
499988e [show][config] add muxcable command line support for retrieve / reset ICMP packet loss data (#2046)
8b01d3e Remove the warning message appear when there are no ports on CONFIG DB (#2050)
ed6e66e [GCU] Supporting Groupings during path-xpath translation (#2044)
25b3455 [ci] Use official build debian pkg instead and parameterize source branch (#2079)
taras-keryk pushed a commit to taras-keryk/sonic-buildimage that referenced this issue Apr 28, 2022
)

#### What I did
Fixes sonic-net#2041

Supporting groupings during `config-db path` <-> `config-yang xpath` translation

This mimics the `config-db` <-> `config-yang`  translation as added by sonic-net#8318

#### How I did it
Handled groupings of leaf commands only as that's what is supported in sonic-yang-mgmt

#### How to verify it
unit-test

#### Previous command output (if the output of a command-line utility has changed)

#### New command output (if the output of a command-line utility has changed)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants