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

[sonic-cfggen]: Optimize template rendering and database access #17740

Merged
merged 1 commit into from
Jan 20, 2024

Conversation

nazariig
Copy link
Collaborator

Signed-off-by: Nazarii Hnydyn nazariig@nvidia.com

Why I did it

  • Improved switch init time
Work item tracking
  • N/A

How I did it

  • Replaced: sonic-cfggen -> sonic-db-cli
  • Aggregated template list for sonic-cfggen

How to verify it

  1. Run warm-reboot

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

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

Tested branch (Please provide the tested image version)

  • master

Description for the changelog

  • N/A

Link to config_db schema for YANG module changes

  • N/A

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

      .---.        .-----------
     /     \  __  /    ------
    / /     \(  )/    -----
   //////   ' \/ `   ---
  //// / // :    : ---
 // /   /  /`    '--
//          //..\\
       ====UU====UU====
           '//||\\`
             ''``

Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
@nazariig
Copy link
Collaborator Author

@qiluo-msft / @yxieca just a kind reminder

@qiluo-msft qiluo-msft merged commit ac09abd into sonic-net:master Jan 20, 2024
23 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Jan 31, 2024
…ic-net#17740)

#### Why I did it
* Improved switch init time

### How I did it
* Replaced: `sonic-cfggen` -> `sonic-db-cli`
* Aggregated template list for `sonic-cfggen`

#### How to verify it
1. Run `warm-reboot`
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202311: #17970

mssonicbld pushed a commit that referenced this pull request Feb 1, 2024
)

#### Why I did it
* Improved switch init time

### How I did it
* Replaced: `sonic-cfggen` -> `sonic-db-cli`
* Aggregated template list for `sonic-cfggen`

#### How to verify it
1. Run `warm-reboot`
@StormLiangMS
Copy link
Contributor

Hi @nazariig, 202305 is done, any justification for necessary to pick up this?

@nazariig
Copy link
Collaborator Author

Hi @nazariig, 202305 is done, any justification for necessary to pick up this?

@StormLiangMS it is already in 202305. Just need a label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants