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

[autopkgserver] autopkgserver should report all errors to a file, rather than console output. #831

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

rslowe
Copy link

@rslowe rslowe commented Oct 17, 2022

Attempts to fix #830 where the error messages should be dumped to a file.

Since autopkgserver is not run interactively, this can reveal issues with autopkgserver in the running environment.
This acts as a preliminary fix, and is open for suggestions on how to allow the logging framework to log these early runtime errors.

Summary of Changes:

  1. Logger is created upon autopkgserver startup (before anything occurs).
  2. Early errors (errors that occur before the AutoPkgServer object is even created) are logged to the logger as level "Critical"
  3. The logger handle is passed through to the autopkgserver class, where it functions as before.

… logger into server class. Changed all prints to stderr to critical log messages, so that they show up in log as a different class of error than the existing errors.
@rslowe rslowe marked this pull request as ready for review October 24, 2022 19:11
@nmcspadden nmcspadden changed the base branch from master to dev August 30, 2023 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant