Skip to content

ci: Run packit tests on CentOS 9 and 10 Stream too - #181

Open
vojtechtrefny wants to merge 1 commit into
storaged-project:mainfrom
vojtechtrefny:main_packit-tmt-centos
Open

vojtechtrefny wants to merge 1 commit into
storaged-project:mainfrom
vojtechtrefny:main_packit-tmt-centos

Conversation

@vojtechtrefny

@vojtechtrefny vojtechtrefny commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Tests
    • Automated pull request checks now cover CentOS Stream 9 and CentOS Stream 10 on both x86_64 and aarch64. This expands platform coverage for validating changes across supported CentOS Stream environments.

@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Warning

Review limit reached

You'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 25 minutes.

Check out review usage here.

View limit details

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

Learn how review limits work.

Review configuration:

Run configuration
Commits

Reviewing files that changed from the base of the PR and between 13347d0 and d1327ff.

Files selected for processing: 1
  • .packit.yaml

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 045b003 and 13347d0.

Files selected for processing: 1
  • .packit.yaml

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

Walkthrough

The pull-request tests job now targets CentOS Stream 9 and 10 on x86_64 and aarch64.

Changes: Pull-request test targets

Layer / File(s) Summary
Add CentOS Stream test targets
.packit.yaml
The pull-request tests job adds CentOS Stream 9 and 10 targets for x86_64 and aarch64.

Priority: Low

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

Merge Risk: Minimal · previous review 13347

The added CentOS Stream targets present no identified merge-blocking risk.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: adding CentOS Stream 9 and 10 to Packit tests.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

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

@vojtechtrefny
vojtechtrefny force-pushed the main_packit-tmt-centos branch from 13347d0 to 16932a6 Compare October 30, 2026 08:30
@vojtechtrefny
vojtechtrefny force-pushed the main_packit-tmt-centos branch from 16932a6 to d1327ff Compare October 30, 2026 08:39

This branch has not been deployed

No deployments
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.

1 participant