Repository navigation
Conversation
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 59 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
The current exact-head failure is in the fapolicyd content-proxy job: all containers disappear and subsequent commands fail with |
f32f55d to
dbb648b
Compare
dbb648b to
f60a74c
Compare
Why are you introducing these changes? (Problem description, related links)
The backup success-path fixture can run while Foreman still has asynchronous tasks in progress, which makes the command correctly refuse to start and turns otherwise successful integration runs red.
Closes issue #894.
What are the changes introduced in this pull request?
--wait-for-tasks.How to test this pull request
Steps to reproduce:
OBSAH_STATE=/private/tmp/foremanctl-894-state python -m pytest tests/backup_test.py::test_backup_command_waits_for_tasks -vvruff check tests/backup_test.pyChecklist
This change was prepared with AI assistance.