[Molter73]

Description

RHCOS tests are taking 30 minutes because it needs to run the tests on 9 VMs. By default ansible only runs 5 parallel builds, this change bumps the parallel builds to 20 like we have in the collector repo.

Checklist

• Patch has a change log entry OR does not need one.
• Investigated and inspected CI test results
• Updated documentation accordingly

Automated testing

• Added unit tests
• Added integration tests
• Added regression tests

If any of these don't apply, please comment below.

Testing Performed

CI should be enough.

Summary by CodeRabbit

• Improvements
  • Ansible operations now process up to 20 hosts concurrently by default. This increases parallel execution capacity; actual completion times depend on the environment and the tasks being run.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Source mode: Auto
• Repository: stackrox/fact
• Branch: mauro/fix/more-ansible-parallel-builds
• File: .coderabbit.yml
• Commit: 0ba8ec2
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Enterprise
• Run ID: 952e0ed9-ecd6-4048-9c8b-ff3abd2900e3

📥 Commits

Reviewing files that changed from the base of the PR and between 77fbaeb and 0ba8ec2.

📒 Files selected for processing: 1

• ansible/ansible.cfg

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 10 remain after this review.

📝 Walkthrough

The Ansible defaults configuration adds forks=20.

Changes: Ansible configuration

Layer / File(s)	Summary
Set Ansible fork count ansible/ansible.cfg	The defaults configuration adds forks=20.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Bug fix

Suggested reviewers: erthalion

Merge Risk: ⚪ Minimal · up to 0ba8e

The configuration change is limited to increasing Ansible parallelism and is ready to merge with normal checks.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title clearly and concisely describes the main change: increasing parallel Ansible workers.
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Description check		The pull request includes the required description, checklist, and testing sections. It explains the change and states that CI is sufficient. Documentation and automated test items remain unchecked, b…

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Full details: Description check

Explanation

The pull request includes the required description, checklist, and testing sections. It explains the change and states that CI is sufficient. Documentation and automated test items remain unchecked, but the configuration-only change is otherwise adequately documented.

✨ Finishing Touches: 💡 1

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

🛠️ Fix failing CI checks: 💡

• Commit to this branch
• Create a new PR

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

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.45%. Comparing base (d8e0ff9) to head (1eba0c8).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2210   +/-   ##
=======================================
  Coverage   33.45%   33.45%           
=======================================
  Files          22       22           
  Lines        3623     3623           
  Branches     3623     3623           
=======================================
  Hits         1212     1212           
  Misses       2402     2402           
  Partials        9        9           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.