forked from sonic-net/sonic-mgmt-framework
-
Notifications
You must be signed in to change notification settings - Fork 15
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
Merge latest BRCM changes to master #18
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Change-Id: Ia15f95a56d2003a7fd9ea9b1b6689c01a95ec852
Change-Id: I95e42d3c6a79c6dadeb11d8b6100c407ffd190c3
Change-Id: I3fe10d31b19076624cfee5a56a544fd685c31ed4
Change-Id: I4757badf5db8b843c9af740eb8e98e7bd6574412
Change-Id: Id89375368991d59636fd3834855b5d3c0cd3c3d6
Change-Id: I13371692921faa60710ea3b33f3db908b1419137
Change-Id: I07063843322e17f23c3ac2e714d5ee79436db0ae
* Reworked the CAS Trans for different Redis Operations. * Added the placeholders for the CVL hooks. Change-Id: I39e907e94f486fbe37464697edfb0f658166f8d8
Change-Id: Ia3f6906ba207eeaa259f67a268cb25d66ed8d85c
Change-Id: I90816440913aabe4b74a6b266948e99c4d3d6a4d
Change-Id: Id8d16c065f3e290bf1310697287545b3645ae566
ebc71bd Render template files for ACL CLI commands (Arlo POC) d8bc76a clearing unnecessary cli xmls from clitree/cli-xml 60a3ac9 Merge branch 'dell_cli' of https://github.com/project-arlo/sonic-mgmt-framework into dell_cli 3e2fa10 - Added following support to process raw CLI XMLs to produce exanded XMLs that Klish parser engine can consume to create the UI : 1.) For each view and nested views add default commands like endi,exit 2.) Substitute macros and nested macros refrenced in CLI xml with actual contents 3.) Proper substitution of arguments passed to macros 4.) Add pipe option to all show/get commands 5.) Entity/Enumeration substitutions - Reorganized source as well as target directory structure 1d5ceec Merge branch 'dell_cli' of https://github.com/project-arlo/sonic-mgmt-framework into dell_cli
Change-Id: Ie7ee6330b33e1e00cb684d91f7d5bad6bd994dfe
Change-Id: I1ae2e743fa1017abbaa63e8e30e12a08741d9c7d
Change-Id: I391f0c1d5a0d1c21348864dbda4c552953ae6ea2
Change-Id: I25be30700c344fd1e566e81c7a8d6b6ee08fa3cf
Change-Id: Ia777962aee56b0f78d143dc44b8c3659daaa01d0
Change-Id: Ib386a73c1b74c7095a98a59bb58304d1518cb243
Change-Id: I9af4a7cf76ffce0eff64a79021edb1ae500f8e34
Change-Id: I8594db119db2fdbde947b648b0df91583d5f3746
Change-Id: Iaad42b5213d69a9396656abc49720f88a49f17a8
Change-Id: I2c0352bf5861b89f8836f838093f93e56a85af90
…oadcom.net:29420/sonic/sonic-mgmt-framework into sonic_management_feature Change-Id: If7801ae0b9bdbae72d760333ec54a60f14fb7369
…oadcom.net:29420/sonic/sonic-mgmt-framework into sonic_management_feature
…oadcom.net:29420/sonic/sonic-mgmt-framework into sonic_management_feature
Change-Id: I1c530c7b30afd213cb0d7ccecd9a8b2e72f3c6ca
Change-Id: Id473adfd725487c2487b72714d816b4884ce27b9
…oadcom.net:29420/sonic/sonic-mgmt-framework into sonic_management_feature
Change-Id: I8e4c8e480c8bb2076701a6c4cba0a2d27dc5f470
Change-Id: I8e4cc823c01e194a19d17abef8a0f0f6aca64f70
…oadcom.net:29420/sonic/sonic-mgmt-framework into sonic_management_feature
seiferteric
approved these changes
Jul 9, 2019
joyas-joseph
approved these changes
Jul 9, 2019
kwangsuk
added a commit
that referenced
this pull request
Nov 13, 2019
Addressing review comments in PR #18
Merged
seiferteric
added a commit
that referenced
this pull request
Mar 24, 2020
* [JIRA SONIC-9259] Add range limit to mtu in sonic-port. Change-Id: I5917eb03c44410fcf0aa0ceb5720cd5c12415d00 * Signature chnage for transformer functions * modified acl-key xfmr to return integer and enhanced the get-req handling in common app. * make common-app as default to handle acl, add the field-xfmr for acl type * [JIRA SONIC-9203] Added IP_TYPE field with value ANY in DEFAULT_RULE. Change-Id: I82ec0a1610da009f9906d0ac2ab7c5eb5aa4b13b * Fix for bugs 96, 97 and 107 * Addressed the comment to sort the speed * Fix sonic get query failure * fix to exclude key-leaf data populating db-filed value and removed commented lines in xmfr_acl * Change MAX_MTU to 9216 in platform_dummy file as well, since it overwrites it. * Fix for bugs 96, 97 and 107 (Speed is not displaying properly) (#109) * Fix for not showing description when description is configured as part of #show interface command. * Addressing the crash when max mtu is set as 9276. Currently supported max-mtu is 9216. * Fix for bugs 96, 97 and 107 * Addressed the comment to sort the speed * Change MAX_MTU to 9216 in platform_dummy file as well, since it overwrites it. * fix for show process not working (#110) * added unit8 yang data0type support for get-req * Fix crash for get leaf query * cleanup code to showcase overloaded methods * [JIRA SONIC-9466] merge mgmt-framework bug fixes from Dell - 12th Sept Change-Id: I2a92251040a816c24f8433e49dd508091dbb494b * fix the build error * Fis get leaf crash, remove unsed annotations * [JIRA SONIC-9426] CVL throws error when trying to configure IP address. Change-Id: Iec675bff98c912ef5d80a2d38524748d6a0b1683 * [JIRA SONIC-9494] CVL Error seen when performing no description on interface Change-Id: Ic3e89d74bd399e891a06e14bd4ff7dcd486a9090 * [JIRA SONIC-9420] Updated Help string to display max length of ACL is 63 characters. Added pattern and length check in CVL yang also to allow 72 characters (63 of acl name + 9 from acl type ex. _ACL_IPV4) Change-Id: Iba9ffc7aa34f1600c61b95ec9a3e23971533e829 * [JIRA SONIC-9496] Disabled nonyang demo app Disabled nonyang vlan demo app by adding .demo suffix to yaml and app module code. It can bemanually enabled by renaming those files back. Change-Id: I1714e772fb452ae246d44af65d43dd1cf65f5e18 * Adding Interface table entry with just name, before the one with ifname and IP. * add get-validate to validate a YANG node durin data translation back to YANG response * Fix crash during update request to leaflist * Fix top level delete * Fix: Incorrect LLDP neighbor TTL (#117) * Remove the TTL information from CLI and GNMI output because LLDP neighbor TTL information is currently not stored in appDB. Signed-off-by: Garrick He <[email protected]> * Bugfix for creating 2 interface entries when IP is configured, and when interface comes up as L2. (#116) * [JIRA SONIC-9314] Modified the MAX_PRIORITY constant to 65536, to help app module to translate between SeqId and priority when given max seqid of 65535. Change-Id: I314e5cabb89779e0410d05d18152425f2c0228b2 * [JIRA SONIC-9472] Modified logic in Jinja template for "show ip access-group" to display both Ingress and Egress ACLs for single port. Also updated help message for max allowed characters for acl name in "show" commands. sonic# show ip access-group Ingress IP access-list A1 on Ethernet0 Egress IP access-list A2 on Ethernet0 Ingress IP access-list A1 on Ethernet4 Egress IP access-list A2 on Ethernet4 sonic# Change-Id: I601e1b21b01e40ea6a285400e803092c66f1c941 * [JIRA SONIC-9521] Merge mgmt-framework changes by Dell team - 13th Sep Change-Id: I2a830e2198b506525ae970377a5bbfb49a98006b * [JIRA SONIC-9486] Permit only TLS1.2 or higher in REST Server Management REST Server is enhanced to use only TLS1.2 or higer version with one of below cipher suites (in order). TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384 TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305 TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305 TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256 TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 Similar changes to gNMI telemetry server will be done by Dell team. UT: 1) Verified ACL CLIs, which also verifies client SDK 2) Verified swagger UI with Chrome and Firefox browsers 3) Verified thru curl that sslv2, sslv3, tlsv1.0, tlsv1.1 clients are rejected by REST Server. 4) Verified thru curl that unsupported ciphers TLS_ECDHE_RSA_WITH_3DES_EDE_CBC_SHA and TLS_RSA_WITH_3DES_EDE_CBC_SHA are rejected by REST Server. 5) Mgmt-framework automated UT Change-Id: I378522335f69d9a73ff7d62dc7ceec79087faf2f * added yang annotations * Get case - leaf-list support in Transformer core for Sonic and OC yang. * add the field transformer for forwarding-action * Set case - leaf-list support in transformer core for OC yang * provision to get data for sibling container from the module level * Fixed variable scope issue while processing leaf data to redis DB format * fix license header * Updated license statement for REST server sources * Update license header * update broadcom license for pyang plugin for openAPI * Updated License statement * Update broadcom license * DB Access Layer: Update Broadcom License * Add validate callback function support for acl. Children will inherit parent validate handler * Add copyright/license headers (#123) Signed-off-by: Tejaswi Goel <[email protected]> * move json creation for sonic-yang to map * Moving unwanted CVL YANG to testdata * Updated Makefiles * Delete file checked in mistakenly. * Updated new Makefile with license, copy schema from testdata/schema also. * Updated License statement * Update License statement * update broadcom license and made changes to keep the diff file in the ygot-modified directory * error handling and code clean-up * update broadcom license to the diff files which are created for patching ygot files * Add error handling in xlate_utils.go * fix the uint64 encoding, fix the ygot bug in handling the union type of leafref * For a Get req on individual terminal node- Fix overloaded function processing, data-type conversion and leaf-list check handling * table transformer support * CLI skeletion for VLAN feature: config, show and its templates * Made changes to keep the patch files into single patch file * Cleanup of unused code * Updated the ygot patch file * Changes for supporting show command with dummy data * add the union type * Update Makefile Add rule to compile test YANG schema * CLI skeleton for Port channel Signed-off-by: Tejaswi Goel <[email protected]> * Add post transformer infra * CVL changes for SONiC YANG * CVL changes for SONiC YANG * Initial support for table transformer * Fix issues in initial commit for table transformer * copy the misisng lines from transformer-phase1 * add missing lines from transforer-phase1 * support for table transformer for crud and get operation * Extend annotations for augmentations in oc-yang * Addressing the comments mentioned * Fixed alignment of XML files * Cleanup of the xml files * Handle nested groupings for augmentation * changed the xSpecMap to xYangSpecMap for code clarity and added new file with transformer-constants * Changes to remove dummy json file Signed-off-by: Tejaswi Goel <[email protected]> * Enhanced transformer core to support user-defined key-delimiter/concatenater * basic support for sonic-yang annotation * Fill cvl KeySpec with DbName read from annotation * More changes for CVL YANG * Fix table get in Traverse for App DB with ":" separator * added support to invoke container level key transformer * added code to pass the current ygot node to the key xfmr function * added support for sonic yang non-config db get request * Data base specific key-delimiter support in transformer core - Sonic Get case * Changes in show port-channel command Signed-off-by: Tejaswi Goel <[email protected]> * Add checks for xpath entry in xYangSpecMap and xDbSpecMap * Chk for list type to create keySpec * workaround for VLAN_MEMBER_TABLE list level get due to translib.generateGetResponsePayload() failure * added key-name support for oc yang and support to create and push table with just the keys and no field-value data in to redis. * Updated Makefile * OC Yang Get case - DB type based key-delimiter support in transformer core * CVl changes for SONiC YANG * CVL changes for SONiC YANG * Add check for no xfmrFunc defined case * Correct naming convention for Interface commands Signed-off-by: Tejaswi Goel <[email protected]> * Changed processLeafList() function name to checkAndProcessLeafList() , added comments and removed unnecessary logs in common app * fix for vlan get-req crash * sonic yang get case handling for table instance level * Handle nested db-name annotations for oc * Import module name of the annotated module in annotate template file * Added table transformer method prototype. * Added table transformer extension * Include the prefix for the module where the grouping is used in annotation template * Include unique namespace, Use the prefix of the including module * xspec map and cru support in transformer infra per revised sonic-yang and changed the api-names from cvl to sonic * Management interface cli tree, actioner implementation and sonic yang for management port and interface config handling. (#153) * sFlow CLI Template (#147) * sFlow CLI template * Add XML skeleton for the sFlow CLI * Add actioner script * Add renderer template Signed-off-by: Garrick He <[email protected]> * added code to support reading db data for get req per revised sonic-yang * rearranged the code to call the table transformer and collect data before calling the subtree-transformer overloaded method. * Fixed table transformer GET case issues : 1.) calling table-transformer for each leaf when doing a get on parent conatiner(OC-interface) 2.) dbMapData not getting filled for container query in dbDataToYangJsonCreate * Handle delete for restructured sonic yang * Strip augmented module names in xpath to reference xYangSpecMap * transformer infra support for revised sonic-yang GET req * Handle delete for restructured sonic acl * Fix the leaf level get for restructured sonic yang * Make file changes to generate sonic yaml files * support table transformer - GET case on leaf or nested container * Get for sonic yang at instance level to return single instance data * Use new xpathPredicate function that removes augmented module name in the uri * Bug fixed for not showing GB instead of Gx * Add Copyright info * Sonic yang restructuring fix terminal node leaf/leaf-list cases * revert accidently added local changes in xspec.go * Clean up of files and add missed items during merge * exclude annotation file from generating ocbinds * fix Makefile in installing yang files from cv/schema * add back the cv/schema/*yin * Fixed list instance level GET issue * Revert "Merge pull request #173 from project-arlo/transformer-phase2" This reverts commit ca4b330, reversing changes made to 211f17e. * Revert "Merge branch 'transformer-phase1' into transformer_merge" This reverts commit 211f17e, reversing changes made to f032ec5. * commit Transformer codes for community PR * Update Makefile * Update Makefile * undo ygot patch changes done by kwan * update the patch file * Patch goyang files to make rules * Add missing models list file * Remove network-instance yang * Addressing PR review comments. * Add new file entry in goyang.patch file * Incorporating review comments - phase2 * Adding comment for TRACE_ONERROR * Moving go-server.sh to tools/test/rest-server.sh * Correcting regex pattern for key names * Correct error code if key does not exist * Address review comments * Address review comments * Address PR review comments - phase2 * Add validation for speed and description (#291) * Add validation for speed and description * Add pattern for alias * Updated the Makefile to have go pkg version, and renamed the ACL CLI actioner file * Update control * Moving jsonquery patch in patches/jsonquery.patch file, updating testcases as per latest YANG model. * Minor change in test case. * Updating makefile * Checkout jsonquery only once. * Use git clean before patching to ensure no files are left behind Co-authored-by: asha-behera <[email protected]> Co-authored-by: ranjinidn <[email protected]> Co-authored-by: s-mari <[email protected]> Co-authored-by: Kwan <[email protected]> Co-authored-by: Mayank Maheshwari <[email protected]> Co-authored-by: Justine Jose <[email protected]> Co-authored-by: Arunsundar Kannan <[email protected]> Co-authored-by: Kwan Kim <[email protected]> Co-authored-by: Sachin Holla <[email protected]> Co-authored-by: Garrick He <[email protected]> Co-authored-by: Partha Dutta <[email protected]> Co-authored-by: amrutasali <[email protected]> Co-authored-by: Faraaz mohammed <[email protected]> Co-authored-by: anand-kumar-subramanian <[email protected]> Co-authored-by: a-barboza <[email protected]> Co-authored-by: Tejaswi Goel <[email protected]> Co-authored-by: Balachandar Mani <[email protected]> Co-authored-by: rvasanthm <[email protected]> Co-authored-by: joyas-joseph <[email protected]> Co-authored-by: Eric Seifert <[email protected]>
dell-engops
pushed a commit
that referenced
this pull request
Feb 9, 2022
Remove ACTION for show ip/ipv6
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a periodic sync of BRCM changes into staging area (mater branch of project-arlo).
Please note that this PR also includes changes for go code profiling.