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.

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

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)
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

No branches or pull requests

3 participants