[jakduch]

Summary

• run Foreman task preflight checks only on hosts with the foreman feature
• keep Pulp task checks enabled for content proxy backups
• document the server/proxy distinction and guard it with a regression test

Testing

• ansible-lint src/roles/backup
• targeted feature-guard regression test
• git diff --check

Fixes #790

[coderabbitai]

Warning

Review limit reached

Next included review available in 52 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 9eb9d1d4-36f1-4c4d-bbdf-f52f3aaed31d

📥 Commits

Reviewing files that changed from the base of the PR and between 6c79648 and 1572fc7.

📒 Files selected for processing (3)

• docs/user/backup-restore.md
• src/roles/backup/tasks/preflight.yaml
• tests/unit/check_role_test.py

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[jakduch]

The timer failures are outside this proxy-backup change and occur under the same resource pressure that killed services in the shared CI runs. #865 limits the Candlepin heap for development tuning and has passed the complete CI matrix.

[jakduch]

The remaining failures were unrelated timer/service flakes, and this branch was also missing the Candlepin heap fix from #865. I rebased it onto current master; CI is running again.