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

Merge hotfix/v0.7-supervisor-logs into release/v0.7 #2753

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

rkuo-danswer
Copy link
Contributor

Automated PR to merge hotfix/v0.7-supervisor-logs into release/v0.7.

Copy link

vercel bot commented Oct 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
internal-search ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 10, 2024 0:58am

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

This PR updates the supervisord.conf file to improve log management and consistency for the document indexing and slack bot programs.

  • Added explicit stdout_logfile and stdout_logfile_maxbytes settings for document_indexing and slack_bot programs in /backend/supervisord.conf
  • Removed environment variables for log file names, standardizing the logging configuration
  • Updated log file paths in the log-redirect-handler to reflect the new logging setup
  • These changes enhance monitoring and debugging capabilities for the system components

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings

@rkuo-danswer rkuo-danswer merged commit a4f1bb9 into release/v0.7 Oct 10, 2024
6 checks passed
@rkuo-danswer rkuo-danswer deleted the hotfix/v0.7-supervisor-logs branch October 16, 2024 18:35
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.

1 participant