Skip to content

ci: update parallelism of unit tests - #254

Merged
crookedstorm merged 3 commits into
mainfrom
speed-up-tests/crookedstorm
Jun 10, 2026
Merged

ci: update parallelism of unit tests#254
crookedstorm merged 3 commits into
mainfrom
speed-up-tests/crookedstorm

Conversation

@crookedstorm

@crookedstorm crookedstorm commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Initially this is just testing the behavior of things. Then, hopefully we can tweak the paralellism and make things faster.

Summary by CodeRabbit

  • Chores
    • Increased parallel test workers in CI from 2 to 4 to speed up Python test runs.
    • Added a CI linting rule to enforce kebab-case naming for action inputs, improving workflow consistency and preventing naming errors.

@crookedstorm
crookedstorm requested a review from a team as a code owner June 9, 2026 23:36
@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 04c4fc87-6c4a-4b34-add2-ca0ebc5c3b06

📥 Commits

Reviewing files that changed from the base of the PR and between 683be80 and 0ce8258.

📒 Files selected for processing (2)
  • .github/actionlint.yaml
  • .github/workflows/ci.yaml
💤 Files with no reviewable changes (1)
  • .github/actionlint.yaml
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/ci.yaml

📝 Walkthrough

Walkthrough

CI workflow .github/workflows/ci.yaml increases PYTEST_WORKERS from "2" to "4" for both python-unit-test and python-integration-test jobs' test steps. .github/actionlint.yaml adds a custom actionlint rule input-names-kebab-case enforcing kebab-case for action input keys.

Changes

CI Workflow & Linting

Layer / File(s) Summary
Increase PYTEST_WORKERS
.github/workflows/ci.yaml
PYTEST_WORKERS changed from "2" to "4" in the python-unit-test and python-integration-test job test-step environments.
Add actionlint input-name rule
.github/actionlint.yaml
Adds input-names-kebab-case custom rule validating ^inputs\.[a-z0-9-]+$ and emitting input name must be kebab-case (lowercase and hyphens only) on violations.

Possibly related PRs

Suggested reviewers

  • mckornfield
  • svvarom
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Title directly matches the main change: increasing PYTEST_WORKERS from 2 to 4 to update test parallelism in CI.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch speed-up-tests/crookedstorm

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor
Suite Lines Covered Line Rate Branch Rate
Unit Tests 19027/25079 75.9% 62.5%
Integration Tests 12035/23851 50.5% 25.7%

@crookedstorm
crookedstorm force-pushed the speed-up-tests/crookedstorm branch from ac74e4f to 683be80 Compare June 9, 2026 23:46
Signed-off-by: Brooke Storm <brookes@nvidia.com>
Signed-off-by: Brooke Storm <brookes@nvidia.com>
Signed-off-by: Brooke Storm <brookes@nvidia.com>
@crookedstorm
crookedstorm force-pushed the speed-up-tests/crookedstorm branch from 683be80 to 0ce8258 Compare June 9, 2026 23:54
@crookedstorm
crookedstorm added this pull request to the merge queue Jun 10, 2026
Merged via the queue into main with commit 69e20df Jun 10, 2026
39 checks passed
@crookedstorm
crookedstorm deleted the speed-up-tests/crookedstorm branch June 24, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants