[pull] master from kevoreilly:master#370
Merged
pull[bot] merged 2 commits intothreatcode:masterfrom Jan 5, 2026
Merged
Conversation
* Add stuck VM monitoring and killing to scheduler The scheduler now monitors running analysis tasks and kills virtual machines if a task exceeds its allowed runtime (timeout + critical + 100s). Added tests to verify that stuck VMs are killed and healthy VMs are not affected. * Update test_scheduler_monitoring.py * fix * Update test_scheduler_monitoring.py * Update test_scheduler_monitoring.py * sync * Update scheduler.py * Update test_scheduler_monitoring.py * Update test_scheduler_monitoring.py * Update test_scheduler_monitoring.py * Update test_scheduler_monitoring.py * Improve error handling and timeout management Refactor error handling and timeout logic in guest.py. * Add guide for debugging stuck virtual machines This guide provides a systematic approach to diagnosing and resolving issues where the analysis process hangs indefinitely, detailing problem triage, immediate diagnostics, debugging steps for Python and QEMU, root cause analysis, and resolution strategies. * Add issues documentation and remove scheduler monitoring test Introduced a new 'Issues' section in the documentation with an initial page for debugging VM issues. Updated the main documentation index to include this section. Removed the test_scheduler_monitoring.py test file. * Update guest.py * Update config references for timeouts in guest handling Replaced references to cfg.cuckoo.timeouts with cfg.timeouts in both documentation and GuestManager code to reflect updated configuration structure.
* Bugfix: Always initialize Package.args in Linux Analyzer Package Fixes a bug introduced in d9be043 that prevents Package.args from being set if a string was not passed into the object during initialization. Package.args will now always be initialized with an empty list if no arguments are provided. This behavior aligns with the Windows analyzer. * Adds some basic Linux analyzer unit tests * Use poetry to run Linux analyzer unit tests in CI * Make analyzer.linux.lib.core.packages.Package.__init__._args more concise * Remove placeholder test * Increase pytest verbosity for Linux analyzer tests * Add additional test cases for Linux Analyzer package arguments * Remove unused import in Linux analyzer * Fix Linux analyzer test case * Fix Linux analyzer skip logic * Add exception handling to Process.get_parent_pid in Linux analyzer * Fix test_proc_dead_states test case
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )