Skip to content

[swss.sh] optimize macsec feature state query#85

Closed
stepanblyschak wants to merge 1 commit intomasterfrom
swss-optimize
Closed

[swss.sh] optimize macsec feature state query#85
stepanblyschak wants to merge 1 commit intomasterfrom
swss-optimize

Conversation

@stepanblyschak
Copy link
Owner

Signed-off-by: Stepan Blyschak stepanb@nvidia.com

Why I did it

There's a slowdown in bootup related to the execution of a show command during startup of swss service. show is a pretty heavy command and takes long time to execute ~2 sec.

How I did it

I replaced show with sonic-db-cli which takes a ms to run.

How to verify it

Boot the switch and verify swss is active.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
stepanblyschak pushed a commit that referenced this pull request Feb 13, 2023
[sonic-linkmgrd][202012] submodule update

0839af2 Longxiang Lyu Wed Jun 15 08:46:21 2022 +0800 [202012] Fix IP header checksum in handleSendSwitchCommand (#89)
afc4972 Jing Zhang Wed Jun 1 10:33:12 2022 -0700 Revert "Update log level for mux probing and mux state chance (#23)" (#85)
ed52d0a Longxiang Lyu Tue May 31 10:28:30 2022 +0800 Add a command line option to store logs into a separate file (#83)

sign-off: Jing Zhang zhangjing@microsoft.com
stepanblyschak pushed a commit that referenced this pull request Apr 8, 2024
…e latest HEAD automatically (sonic-net#18450)

#### Why I did it
src/wpasupplicant/sonic-wpa-supplicant
```
* 3c7fd8e10 - (HEAD -> master, origin/master, origin/HEAD) Make Openssl CMAC API (omac1_aes_vector) Availalable in Non FIPs Mode (#85) (2 hours ago) [wumiao_nokia]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stepanblyschak pushed a commit that referenced this pull request Aug 6, 2025
…ically (sonic-net#23389)

#### Why I did it
src/sonic-dash-ha
```
* aeac5d0 - (HEAD -> master, origin/master, origin/HEAD) Remove fvs and rename backup_fvs to fields in show hamgrd actor command (#85) (6 hours ago) [yue-fred-gao]
* cf03fe8 - Add test case to verify zmq reconnect behaviour (#84) (6 hours ago) [yue-fred-gao]
```
#### 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.

2 participants