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.

Sign up for GitHub

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

Jump to bottom

fix(backend): Increase logging level threshold of RPC service to WARNING #9576

Merged
merged 2 commits into from
Mar 5, 2025
Merged

fix(backend): Increase logging level threshold of RPC service to WARNING #9576

merged 2 commits into from
Mar 5, 2025

Conversation

majdyz
Copy link
Contributor

@majdyz majdyz commented Mar 5, 2025 •
edited by Pwuts
Loading

Increase the logging threshold to WARNING to avoid chatty RPC service request logs.

Before:
RPC calls show up in backend log

@majdyz majdyz requested a review from Pwuts March 5, 2025 12:29
@majdyz majdyz requested a review from a team as a code owner March 5, 2025 12:29
@majdyz majdyz requested review from Bentlybro and removed request for a team March 5, 2025 12:29
@qodo-merge-pro Qodo Merge Pro
Copy link

qodo-merge-pro bot commented Mar 5, 2025

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions github-actions bot added the platform/backend AutoGPT Platform - Back end label Mar 5, 2025
@netlify Netlify
Copy link

netlify bot commented Mar 5, 2025 •
edited
Loading

✅ Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit 6772bec
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/67c843bdd244f0000893c5af

@github-actions github-actions bot added the size/s label Mar 5, 2025
@deepsource-io deepsource.io
Copy link

deepsource-io bot commented Mar 5, 2025 •
edited
Loading

Here's the code health analysis summary for commits b5c1007..6772bec. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗
DeepSource Python LogoPython✅ Success
❗ 21 occurences introduced
🎯 18 occurences resolved
View Check ↗
💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@netlify Netlify
Copy link

netlify bot commented Mar 5, 2025

✅ Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 6772bec
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/67c843bd8153ad0008de2630

@Pwuts Pwuts enabled auto-merge March 5, 2025 12:35
@Pwuts Pwuts added this pull request to the merge queue Mar 5, 2025
Merged via the queue into dev with commit 55d6495 Mar 5, 2025
23 checks passed
@Pwuts Pwuts deleted the zamilmajdy/fix-rpc-logging-level branch March 5, 2025 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pwuts Pwuts Pwuts approved these changes

@Bentlybro Bentlybro Awaiting requested review from Bentlybro Bentlybro is a code owner automatically assigned from Significant-Gravitas/maintainers

Assignees
No one assigned
Labels
platform/backend AutoGPT Platform - Back end Review effort 1/5 size/s
Projects
AutoGPT development kanban
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@majdyz @Pwuts