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

Store tables without yang #69

Merged

Conversation

praveen-li
Copy link
Collaborator

@praveen-li praveen-li commented May 1, 2020

- What I did

Store Tables without YANG models seperately for user warning.

- How I did it

Changes:
1.) Store Tables with YANG models seperately, so that a warning to user
can be issued with enough information.
2.) print the tables without YANG models properly in cropConfigDB().

- How to verify it

Changes:
1.) Added config for extra tables in yangTest.json.
2.) Added new test case to find tables without yang models.
3.) Move common part in a helper function.
pchaudha@be1e6e58be56:/sonic$ make -f slave.mk target/python-wheels/sonic_yang_mgmt-1.0-py2-none-any.whl
SONiC Build System

Build Configuration
"CONFIGURED_PLATFORM"             : "vs"
"CONFIGURED_ARCH"                 : "amd64"
"SONIC_CONFIG_PRINT_DEPENDENCIES" : ""
"SONIC_BUILD_JOBS"                : "1"
"SONIC_CONFIG_MAKE_JOBS"          : "24"
"SONIC_USE_DOCKER_BUILDKIT"       : ""
"USERNAME"                        : "admin"
"PASSWORD"                        : "YourPaSsWoRd"
"ENABLE_DHCP_GRAPH_SERVICE"       : ""
"SHUTDOWN_BGP_ON_START"           : ""
"ENABLE_PFCWD_ON_START"           : ""
"INSTALL_DEBUG_TOOLS"             : ""
"ROUTING_STACK"                   : "frr"
"FRR_USER_UID"                    : "300"
"FRR_USER_GID"                    : "300"
"ENABLE_SYNCD_RPC"                : ""
"ENABLE_ORGANIZATION_EXTENSIONS"  : "y"
"HTTP_PROXY"                      : ""
"HTTPS_PROXY"                     : ""
"ENABLE_SYSTEM_TELEMETRY"         : "y"
"ENABLE_RESTAPI"                  : "n"
"ENABLE_ZTP"                      : ""
"SONIC_DEBUGGING_ON"              : ""
"SONIC_PROFILING_ON"              : ""
"KERNEL_PROCURE_METHOD"           : "build"
"BUILD_TIMESTAMP"                 : ""
"BLDENV"                          : ""
"VS_PREPARE_MEM"                  : "yes"
"ENABLE_SFLOW"                    : "y"
"ENABLE_NAT"                      : "y"

pchaudha@be1e6e58be56:/sonic$ cat target/python-wheels/sonic_yang_mgmt-1.0-py2-none-any.whl.log

tests/test_sonic_yang_mgmt.py .
tests/libyang-python-tests/test_sonic_yang.py ........................

========================== 25 passed in 0.27 seconds ===========================

Test on 06/29/2020

pchaudha@server05:/home/pchaudha/srcCode/dpb_repo$ ls -l target/python-wheels/sonic_yang_mgmt-1.0-py2-none-any.whl
-rw-r--r-- 1 pchaudha pchaudha 19385 Jun 29 23:04 target/python-wheels/sonic_yang_mgmt-1.0-py2-none-any.whl
pchaudha@server05:/home/pchaudha/srcCode/dpb_repo$ date
Mon Jun 29 23:09:25 PDT 2020
pchaudha@server05:/home/pchaudha/srcCode/dpb_repo$

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@praveen-li praveen-li force-pushed the store_tables_without_yang branch 2 times, most recently from 20e48c0 to d2b6ec7 Compare June 30, 2020 01:17
Praveen Chaudhary added 5 commits June 30, 2020 14:29
…user warning.

Changes:
1.) Store Tables with YANG models seperately, so that a warning to user
can be issued with enough information.
2.) print the tables without YANG models properly in cropConfigDB().

Signed-off-by: Praveen Chaudhary [email protected]
…ly for user warning.

Changes:
1.) Added config for extra tables in yangTest.json.
2.) Added new test case to find tables without yang models.
3.) Move common part in a helper function.

Signed-off-by: Praveen Chaudhary [email protected]
Changes:
-- fix add_data_node in sonic-yang.
-- Fixing tests of sonic-yang.

Signed-off-by: Praveen Chaudhary [email protected]
@praveen-li praveen-li force-pushed the store_tables_without_yang branch from 9b9c692 to de9f907 Compare June 30, 2020 21:35
@vasant17 vasant17 merged commit afcf45d into zhenggen-xu:sonic-cfg-mgmt Jun 30, 2020
praveen-li pushed a commit to praveen-li/sonic-utilities that referenced this pull request Jun 30, 2020
…ble in config.

After merging zhenggen-xu/sonic-buildimage#69
enabling test for extra table in config.

Signed-off-by: Praveen Chaudhary [email protected]
vasant17 pushed a commit to zhenggen-xu/sonic-utilities that referenced this pull request Jun 30, 2020
…ble in config. (#33)

After merging zhenggen-xu/sonic-buildimage#69
enabling test for extra table in config.

Signed-off-by: Praveen Chaudhary [email protected]
praveen-li pushed a commit to praveen-li/sonic-buildimage that referenced this pull request Aug 12, 2020
)

**sonic-platform-common:**

Commits on Jul 07, 2020
Changes in fan and psu base classes (1.0 platform API) related to pdd… 17292e4
Commits on Jul 10, 2020
update get_pcie_check() to use sysfs instead of lspci output (zhenggen-xu#95) d4eb804
Commits on Jul 11, 2020
[eeprom] Fix UnboundLocalError (zhenggen-xu#93) 7c8bed1
Commits on Jul 14, 2020
[Transceiver] Add parser for QSFP-DD cable type and dictionaries for … be1cc24 

**sonic-platform-daemons:**

Commits on Jul 06, 2020
[psud] Store PSU temperature and voltage information to database (zhenggen-xu#61) ef9716a
Commits on Jul 14, 2020
[xcvrd] Add support for QSFP-DD cables (zhenggen-xu#66) c530587
Commits on Jul 18, 2020
Initial version of pcied (zhenggen-xu#60) e665ee8
[xcvrd] Return non-zero error code on SFP error (zhenggen-xu#67) 4f42a79
Commits on Jul 19, 2020
[README.md] Add LTGM badges (zhenggen-xu#69) 23757a3
Remove unused imports (zhenggen-xu#70) 029d5a5 

Signed-off-by: Nazarii Hnydyn <[email protected]>
praveen-li pushed a commit to praveen-li/sonic-buildimage that referenced this pull request Aug 9, 2022
[master][sonic-linkmgrd] submodule updates

df51322 Longxiang Lyu   Fri May 6 10:01:46 2022 +0800   Add `ActiveActiveStateMachine` implementation (zhenggen-xu#64)
e721ceb Jing Zhang      Wed May 4 10:07:14 2022 -0700   Add doc for default route related changes  (zhenggen-xu#63)
7bb06fb Jing Zhang      Tue May 3 09:48:28 2022 -0700   Add Cli support to enable or disable default route related feature (zhenggen-xu#68)
e4b02cb Jing Zhang      Mon May 2 13:27:54 2022 -0700   Reset WaitActiveUp count before switching to active (zhenggen-xu#70)
212d960 Jing Zhang      Wed Apr 27 10:35:05 2022 -0700  lower log level to warning (zhenggen-xu#69)
48abc9e Jing Zhang      Thu Apr 14 16:50:04 2022 -0700  Add support to enable switchover time measurement (with link prober interval decreased to 10ms) feature  (zhenggen-xu#61)
c4858a6 Jing Zhang      Thu Apr 14 11:27:55 2022 -0700  Avoid proactively switching to `active` if default route is missing  (zhenggen-xu#62)

sign-off: Jing Zhang [email protected]
praveen-li pushed a commit to praveen-li/sonic-buildimage that referenced this pull request Dec 23, 2022
[sonic-linkmgrd][202012] submodule update
3d13ff2 Jing Zhang      Wed May 4 10:07:14 2022 -0700   Add doc for default route related changes  (zhenggen-xu#63)
c703be4 Jing Zhang      Mon May 2 13:27:54 2022 -0700   Reset WaitActiveUp count before switching to active (zhenggen-xu#70)
86eb727 Jing Zhang      Wed Apr 27 10:35:05 2022 -0700  lower log level to warning (zhenggen-xu#69)
e22c736 Jing Zhang      Mon May 2 13:33:24 2022 -0700   [202012] Avoid proactively switching to active if default route is missing (zhenggen-xu#67)
d4f282b Jing Zhang      Thu Apr 28 18:35:11 2022 -0700  [202012] Add support to enable switchover time measurement (with link prober interval decreased to 10ms) feature (zhenggen-xu#66)

sign-off: Jing Zhang [[email protected]](mailto:[email protected])
zhenggen-xu pushed a commit that referenced this pull request May 24, 2024
…utomatically (sonic-net#16842)

#### Why I did it
src/sonic-host-services
```
* fc88254 - (HEAD -> 202305, origin/202305) Support to config fips state (#69) (#78) (16 hours ago) [xumia]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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

Successfully merging this pull request may close these issues.

3 participants