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

fix(feg): Adjust diameter config logs to v=1 #12244

Merged
merged 1 commit into from Mar 22, 2022

Conversation

emakeev
Copy link
Contributor

@emakeev emakeev commented Mar 22, 2022

Signed-off-by: Evgeniy Makeev evgeniym@fb.com

Summary

Adjust diameter config logs to v=1.
new diameter client configuration logs are printed only once during the client instantiation, so it's safe to reduce log level to 1 from current 3 for them.

Test Plan

unit tests

Additional Information

  • This change is backwards-breaking

Signed-off-by: Evgeniy Makeev <evgeniym@fb.com>
@emakeev emakeev requested review from a team and uri200 March 22, 2022 18:38
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines. label Mar 22, 2022
@emakeev emakeev merged commit 77e75e2 into magma:master Mar 22, 2022
tmdzk pushed a commit that referenced this pull request Mar 23, 2022
Signed-off-by: Evgeniy Makeev <evgeniym@fb.com>
ardzoht pushed a commit that referenced this pull request Mar 30, 2022
Signed-off-by: Evgeniy Makeev <evgeniym@fb.com>
@emakeev emakeev deleted the feg_tune_session_proxy_logs branch August 5, 2022 23:13
emakeev added a commit to emakeev/magma that referenced this pull request Aug 5, 2022
Signed-off-by: Evgeniy Makeev <evgeniym@fb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apply-v1.7 size/XS Denotes a PR that changes 0-9 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants