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

Explicitly initialize seelog #2644

Merged
merged 1 commit into from
Oct 28, 2020
Merged

Explicitly initialize seelog #2644

merged 1 commit into from
Oct 28, 2020

Commits on Sep 21, 2020

  1. Explicitly initialize seelog

    The logger package's initialization logic is init() which will be
    executed by loading the package.
    
    However some of our programs are using the agent as a library and
    haven't configured seelog beforehand. Due to that, loading the package
    logs
    
    ```
    1600397429848419666 [Error] node must have children
    ```
    which breaks the programs.
    
    I think it would be safer to do less on init() and explicitly
    initialize the logger from the agent's main().
    
    Signed-off-by: Kazuyoshi Kato <katokazu@amazon.com>
    kzys committed Sep 21, 2020
    Configuration menu
    Copy the full SHA
    294d466 View commit details
    Browse the repository at this point in the history