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

Enable log file support #672

Merged
merged 6 commits into from
Jul 31, 2024
Merged

Enable log file support #672

merged 6 commits into from
Jul 31, 2024

Commits on Jul 29, 2024

  1. Enable multiple log streams for Logger 

    Create a file handler to dump logs to files.
Use stream and file handlers for all the
pipeline services to redirect logs to both
the streams including `stdout` and log file.

Signed-off-by: Jeny Sadadia <jeny.sadadia@collabora.com>
    Jeny Sadadia committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    5837521 View commit details
    Browse the repository at this point in the history

  2. config/logger.conf: update logger configuration 

    As `notifier` service has been renamed to `monitor`,
update its logger configurations accordingly.
Also, add section for `result_summary` service.

Signed-off-by: Jeny Sadadia <jeny.sadadia@collabora.com>
    Jeny Sadadia committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    f4970bc View commit details
    Browse the repository at this point in the history

  3. Create logs directory and mount it 

    Create a directory named `logs` to dump
log files to. Mount the directory for all
the services in `docker-compose.yaml`.

Signed-off-by: Jeny Sadadia <jeny.sadadia@collabora.com>
    Jeny Sadadia committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    ff34630 View commit details
    Browse the repository at this point in the history

  4. src: fix log file related error for patchset service 

    Fix the below error for `patchset` service:
```
FileNotFoundError: [Errno 2] No such file or directory: '/home/kernelci/logs/tarball_2024_07_04-06:24:16.log'
```

Signed-off-by: Jeny Sadadia <jeny.sadadia@collabora.com>
    Jeny Sadadia committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    ee9a1ec View commit details
    Browse the repository at this point in the history

  5. Create different log file for all scheduler containers 

    Add `name` docker argument to provide service name to
scheduler services. Without it, all 4 container i.e
`scheduler`, `scheduler-docker`, `scheduler-lava`, and
`scheduler-k8s` were using the same log file.
That was leading in-accessbility of log files and raised
the below error:
`PermissionError: [Errno 13] Permission denied:
'/home/kernelci/logs/scheduler_2024_07_04-06:42:21.log'`

Signed-off-by: Jeny Sadadia <jeny.sadadia@collabora.com>
    Jeny Sadadia committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    922239a View commit details
    Browse the repository at this point in the history

  6. config/logger.conf: add logger sections 

    Add logger config sections for remaining pipeline
services.

Signed-off-by: Jeny Sadadia <jeny.sadadia@collabora.com>
    Jeny Sadadia committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    5099c67 View commit details
    Browse the repository at this point in the history