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] store jinja2 cache in log level db. #5646

Merged
merged 1 commit into from
Oct 16, 2020

Conversation

stepanblyschak
Copy link
Collaborator

@stepanblyschak stepanblyschak commented Oct 16, 2020

ADO: 24508555

This PR makes two changes:
- Store Jinja2 cache in LOGLEVEL DB instead of STATE DB
- Store bytecode cache encoded in base64

Tested with the following command: "redis-dump -d 3 -k JINJA2_CACHE"

Signed-off-by: Stepan Blyschak [email protected]

- Why I did it

- How I did it

- How to verify it

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

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

This PR makes two changes:
    - Store Jinja2 cache in LOGLEVEL DB instead of STATE DB
    - Store bytecode cache encoded in base64

Tested with the following command: "redis-dump -d 3 -k JINJA2_CACHE"

Signed-off-by: Stepan Blyschak <[email protected]>
@lguohan lguohan merged commit 8df0e2b into sonic-net:master Oct 16, 2020
@lguohan lguohan linked an issue Oct 16, 2020 that may be closed by this pull request
@qiluo-msft
Copy link
Collaborator

Now JINJA2_CACHE appears in the output of swssloglevel. @stepanblyschak Could you fix it?

admin@sonic:~/qiluo$ swssloglevel -p
JINJA2_CACHE                  Unknown log level
SAI_API_ACL                   SAI_LOG_LEVEL_NOTICE
SAI_API_BFD                   SAI_LOG_LEVEL_NOTICE
SAI_API_BMTOR                 SAI_LOG_LEVEL_NOTICE
SAI_API_BRIDGE                SAI_LOG_LEVEL_NOTICE

santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
This PR makes two changes:
    - Store Jinja2 cache in LOGLEVEL DB instead of STATE DB
    - Store bytecode cache encoded in base64

Tested with the following command: "redis-dump -d 3 -k JINJA2_CACHE"

Signed-off-by: Stepan Blyschak <[email protected]>
@stepanblyschak stepanblyschak deleted the redis_bcc_dump_fix branch September 23, 2022 13:31
abdosi pushed a commit that referenced this pull request Jul 14, 2023
This PR makes two changes:
    - Store Jinja2 cache in LOGLEVEL DB instead of STATE DB
    - Store bytecode cache encoded in base64

Tested with the following command: "redis-dump -d 3 -k JINJA2_CACHE"

Signed-off-by: Stepan Blyschak <[email protected]>
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.

redis-dump -d 6 -k JINJA2_CACHE error
4 participants