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

after a day running in debug mode I get an error #250

Open
denics opened this issue Jun 1, 2024 · 0 comments
Open

after a day running in debug mode I get an error #250

denics opened this issue Jun 1, 2024 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@denics
Copy link

denics commented Jun 1, 2024

==> .local/state/mycroft/phal.log <==
    logger = cls.create_logger(name, tostdout=True)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/denix/.venvs/ovos/lib/python3.11/site-packages/ovos_utils/log.py", line 125, in create_logger
    handler = RotatingFileHandler(path, maxBytes=cls.max_bytes,
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/logging/handlers.py", line 155, in __init__
  File "/usr/lib/python3.11/logging/handlers.py", line 58, in __init__
  File "/usr/lib/python3.11/logging/__init__.py", line 1181, in __init__
  File "/usr/lib/python3.11/logging/__init__.py", line 1213, in _open
OSError: [Errno 24] Too many open files: '/home/denix/.local/state/mycroft/phal.log'
@JarbasAl JarbasAl added the bug Something isn't working label Jun 1, 2024
NeonDaniel added a commit that referenced this issue Jul 16, 2024
Add test coverage for log rotation
Outline test coverage for added log module functions
Relates to #2 #239 #233 #250 #253
JarbasAl pushed a commit that referenced this issue Sep 2, 2024
* Add docstrings and type annotations to `log` module
Add test coverage for log rotation
Outline test coverage for added log module functions
Relates to #2 #239 #233 #250 #253

* Update logs to resolve order-related failure

* Add test coverage for `get_logs_config`
Refactor variable in `get_logs_config` for clarity
Refactor `get_logs_config` for more predictable handling of an empty service name

* Add test coverage for `get_log_path`

* Add test coverage for `get_log_paths`
Update `get_log_paths` to check all log names with `-` normalized to `_` (appeared to be the original intent)
Add `enclosure` and `admin` service logs used by Neon and legacy Mycroft/OVOS setups

* Add test coverage for `get_available_logs`

* Add test coverage for `_monitor_log_level`

* Add test of `get_config` call count

* Update `get_log_paths` to reference config directly instead of test some well-known values
Update unit test for `get_log_paths`

* Remove unused `ALL_SERVICES` variable

* Remove unused import

---------

Co-authored-by: Daniel McKnight <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants