* Fastboot/warmboot platform plugin

* Added platform plugin

Add utility to translate text output of syseeprom dump into JSON, for
consumption by new management (1.0) command.

Signed-off-by: Howard Persh <[email protected]>

Signed-off-by: Mykola Faryma <[email protected]>

Currently it misses the case where data for a new key is empty.

#757)

* [neighbor advertiser] catch all exceptions while trying https endpoint

When connecting https endpoint failed, we need to try http endpoint.
Therefore we need to catch all exceptions.

Signed-off-by: Ying Xie <[email protected]>

* raise exception if https returned non-success status code

…re (#758)

* [neighbor advertiser] raise exception when http endpoint return failure

Signed-off-by: Ying Xie <[email protected]>

* Delete deprecated error checking code

* Add back check for None

…(s) (#761)

Once any critical service is shutdown (radv/swss/syncd), we have to
commit to the reboot. Failing in the middle will leave the system in
bad state.

Signed-off-by: Ying Xie <[email protected]>

#770)

* A generic JSON file updater, which can add/update-existing attributes.
This tool would be used to update /etc/sonic/core_analyzer.rc.json file to add
credentials by HW proxy.

* Updated per review comments.
The option is better named.

* [portstat] fix header used

Signed-off-by: Mykola Faryma <[email protected]>

* [portstat] fix print table_as_json

Signed-off-by: Mykola Faryma <[email protected]>

* Fix tx_ovr field

* [show] enhance 'show ip[v6] bgp summary' command
* changing ipaddr to ipaddress

…op counters (#784)

Signed-off-by: Danny Allen <[email protected]>

Merge two vrf configuration to avoid user can't add or delete management VRF through CLI.

Co-authored-by: Minkang-Tsai <[email protected]>

If ACL table name contains the substring "v6", set the EtherType of the rule to V6, otherwise set to V4.

… OID->interface name mapping (#789)

In release branches prior to 201911, FDB or ARP learnt on VLAN interface LAG port will cause nothing to appear in the "show arp" or "show mac" output because there existed no OID->interface name never had the mappings for LAG interface.

This issue has since been fixed in master (and 201911). However, making this change in master is not harmful and will prevent a regression from causing no output to be displayed for the interface name in the future.

* [neighbor advertiser] remove http endpoint access

Signed-off-by: Ying Xie <[email protected]>

* Remove obsolete comment

- Remove prefix-trimming from drop reason capability query
- Remove prefix-trimming from create/add safety checks

Signed-off-by: Danny Allen <[email protected]>

Signed-off-by: [email protected]

Implemented following commands to use Zero Touch Provisioning

show ztp status
config ztp enable
config ztp disable
config ztp run

Signed-off-by: Rajendra Dendukuri <[email protected]>
Co-authored-by: Akhilesh Samineni <[email protected]>
Co-authored-by: rlhui <[email protected]>

…or each container (#798)

Since we introduced the auto-restart feature for each container, we need add a show subcommand to display the current status of auto-restart feature for all containers or a specific container.

- How I did it
We define a function named autorestart to show the status of this features. This function will accept an option parameter which is the container name. If this parameter is not specified, this function will by default show the status of all containers. Otherwise it will show the status of specific container.

Signed-off-by: Yong Zhao <[email protected]>

…re for each container (#801)

…estart feature (#802)

…init_cfg.json implicitly (#812)

* [config/main.py] Modify the load() and reload() functions to load config
from config_db.json and init_cfg.json.

Signed-off-by: Yong Zhao <[email protected]>

* [config/main.py] Undo the changes which load the configuration from
init_cfg.json for load() function and define a
constant string for the path of init_cfg.json.

Signed-off-by: Yong Zhao <[email protected]>

* [config/main.py] Correct a typo.

Signed-off-by: Yong Zhao <[email protected]>

* [config/main.py] Added an else statement in line 551 to decide whether
the init_cfg.json exsits or not in reload function.

Signed-off-by: Yong Zhao <[email protected]>

* [config/main.py] Correct a typo error.

Signed-off-by: Yong Zhao <[email protected]>

* [config/main.py] Change the loading order and we should first load
init_cfg.json and then config_db.json.

Signed-off-by: Yong Zhao <[email protected]>

* [config/main.py] Use constant string to represent the path of
init_cfg.json.

Signed-off-by: Yong Zhao <[email protected]>

Signed-off-by: Akhilesh Samineni <[email protected]>

…815)

This is possible if some of docker are not built then corresponding
service file will not be present

…624)

This change is related to Platform Driver Development Framework (PDDF) which is being added to sonic-buildimage repo. More details can be found here, sonic-net/SONiC#406

PDDF supports its own CLI utilities, which use generic PDDF component plugins. I added these PDDF CLI utilities.

Add a "show platform fanstatus" command to allow user fetch FAN status data.
Add a "show platform temperature" command to allow user fetch thermal status data.

…ts (#772)

* [fwutil]: initial version.

Signed-off-by: Nazarii Hnydyn <[email protected]>

* [fwutil]: Fix UI: enable progressbar render finalizer.

Signed-off-by: Nazarii Hnydyn <[email protected]>

* [fwutil]: integrate utility with SONiC CLI.

Signed-off-by: Nazarii Hnydyn <[email protected]>

* [fwutil]: update CLI command reference documentation.

Signed-off-by: Nazarii Hnydyn <[email protected]>

* [fwutil]: Revisit CLI architecture: avoid direct imports.

Signed-off-by: Nazarii Hnydyn <[email protected]>

* [fwutil]: Fix review comments: refactor CLI command reference.

Signed-off-by: Nazarii Hnydyn <[email protected]>

* [fwutil]: Fix review comments: update CLI documentation.

Signed-off-by: Nazarii Hnydyn <[email protected]>

…819)

* [reboot] make sure the reboot happens even if platform reboot failed

Platform reboot tool could fail if the platform driver didn't load
properly. In this case, a reboot is required to recover the device.
We need to make sure the reboot tool is robust.

Signed-off-by: Ying Xie <[email protected]>

* Rewording

* add log message

Signed-off-by: Nazarii Hnydyn <[email protected]>

* Add pfcwd utility to the config/show CLI

Signed-off-by: Andriy Moroz <[email protected]>

* Add some more commands

Signed-off-by: Andriy Moroz <[email protected]>

* [doc] Add start_default description

Signed-off-by: Andriy Moroz <[email protected]>

* [fwutil]: Use overlay driver when mounting next image filesystem.

Signed-off-by: Nazarii Hnydyn <[email protected]>

* [fwutil]: Update command reference.

Signed-off-by: Nazarii Hnydyn <[email protected]>

Signed-off-by: Shlomi Bitton <[email protected]>

Signed-off-by: Nazarii Hnydyn <[email protected]>

…platform (#833)

* do DB migration for buffer pool size change with new SDK version

* fix review comments
enhace migration fail case

* make migrator can work with warm reboot case

* ehnahce the logic to cover more corner case
simplify the way to generate new buffer configuration.

* remove code to get info from config_db.json since it's not necessary
3800 platform need special buffer configuration

This is not an issue with normal and correct configuration. The
issue was exposed when there is an incorrect configuration, e.g.
contain wrong port names. These wrong port names will still get
populated to the app_db but will not have speed associated.

Lack of speed entry will cause "show interface status" to throw
exception.

Signed-off-by: Ying Xie <[email protected]>

Starting click 7.0. The default behavior of a command with under
scores will be replace with dashes.

this is to address the above default behavior change, so that
the command remains the same.

more details can be found:

pallets/click#1123

… platforms (#856)

* Update stop, reset failed status and restart of systemd services
to support multi-asic platforms.

* Create function to avoid code duplication.

* Fixed errors due to pervious commit and review comments.

* Minor update to fix spacing.

* Minor update to fix spacing.

* Minor update to fix spacing.

* For multi asic platform updated logic of stopping/restarting of
services to ensure that the right instances are stopped and
restarted if a service is both global and multi-instance.

* Fixed log error message with incorrect number of parameterts.

Fix the bug where the unicast/multicast queue watermark display was incorrect

- How I did it
Calculate the header list dynamically for PG and queue display
Display the correct queue numbers in the output instead of always starting the index from 0
Calculate the display position relative to the starting index

Signed-off-by: Neetha John <[email protected]>

#868)

Signed-off-by: Nazarii Hnydyn <[email protected]>

* [Vxlan] : adding show vnet/vxlan cmds

…e for FRR routing stack (#884)

Co-authored-by: Travis Van Duyn <[email protected]>

- This flag is currently no functional impact with just this change.
- This flag is intended to coordinate fast-reboot shutdown path.
- This flag enables moving fast-reboot per service knowledge to service scripts.
  which unblocks moving warm-reboot per service knowledge to service scripts.
- This flag is consistent with the bootup flag we setting on the boot up path.

Signed-off-by: Ying Xie <[email protected]>

Co-authored-by: Travis Van Duyn <[email protected]>

Signed-off-by: Nazarii Hnydyn <[email protected]>

* [utility] Filter FDB entries

FDB table can get large due to VM creation/deletion which cause
fast reboot to slow down. This utility fitlers FDB entries based on
current MAC entries in the ARP table.

signed-off-by: Tamer Ahmed <[email protected]>

* [Phase 1] Multi ASIC config command changes, db_mgrator.py script updates for handing namespace.

* Fixes and comment updates

* Comments addressed + added support for user to input the config files per namespace also.

* Updates per comments + based on the updated SonicV2Connector/ConfigDBConnector class design

* Review comments update.

* Help string updated for config save/reload/load

…t' subcommands (#904)

* Fix NameError for 'swsssdk', align neig output

* Changes:
1) Add syslog support.
2) Enable forever periodic scan
3) Skip link local addresses
4) Skip eth0 routes
3) Adopt to 20191130 changes
    3.1) APPl-DB INTF_TABLE may not have IP address
    3.2) nexthop is never empty

* No logical code change. A small name change.

* 1) Adopt to 201811 -- Filter out 'lo' & 'docker0' in addition to 'eth0' as local routes
2) Ensure to read route entry w/o prefix, if not present with prefix

- Modify the load_minigraph command handler to support multi NPU platforms

…nd 'reset' subcommands (#904)"

This reverts commit 0aed20a.

…multi-npu platforms. (#908)

* Changes to support acl-loader command for multi-npu platforms.

Move multi-npu related utility functions from config/main.py
to sonic-device-util.py so that it can be used by acl-loader/any other
module.

Updated Mirror Session add/remove for multi-npu platforms.
Needed for Everflow ACL rule programming.

* Address review comment to change comment from """ to #
except for doc string

* Changes to ignore services stop/start for config reload and
load_minigraph command. This will be needed when executed from
config-setup script.

* Updated Command Reference document

* Address Review Comments.

…g DB (#885)

Fix is_mgmt_vrf_enabled for the case where MGMT_VRF_CONFIG is not at all in the config DB.
This is the case where mgmt vrf is never configured. The function throws error at
File "/usr/lib/python2.7/dist-packages/show/main.py", line 651, in is_mgmt_vrf_enabled
mvrf_dict = json.loads(p.stdout.read())
Two show commands uses is_mgmt_vrf_enabled. "show mgmt-vrf" and "show ntp"
Both commands throw error if mgmt vrf is never configured

Co-authored-by: Bing Sun <[email protected]>

…t' subcommands (#904)

* Multi-asic changes for config bgp commands and utilities.

* Review comments update

* Optimized the logic of checking the internal hosts using AS number.

* API changes due to rebase. Also setting ignore_local_hosts=false explicitly in BGP commands so that existing single ASIC platform implementation is unchanged.

…nd 'reset' subcommands (#904)"

This reverts commit 83b88d5.

load_minigraph. The behaviour got changed as part of multi-npu change
so make it correct again.

* Changes to make lldp show command for multi-npu platforms.
We will display only front-panel port information.

* Address Review Comments.

* Added Comment

* Fix LGTM error

* Address Review Comments

* [Mellanox] add document for thermal control related cli

* Update review comments thermal zones -> thermal sensors

* Make example better

… and thermal status (#836)

* [thermal fix] use natural sort for fan status and thermal status
* [thermal fix] set fan status to N/A when fan is removed
* Adjust header name for show platform temperature output

* [thermal fix] use natural sort for fan status and thermal status
* [thermal fix] set fan status to N/A when fan is removed
* Adjust header name for show platform temperature output