Skip to content

Enabling sync for pytests workflow callers#117

Merged
joselsegura merged 1 commit into
RedHatInsights:masterfrom
joselsegura:pytest-example-and-sync
Apr 28, 2026
Merged

Enabling sync for pytests workflow callers#117
joselsegura merged 1 commit into
RedHatInsights:masterfrom
joselsegura:pytest-example-and-sync

Conversation

@joselsegura
Copy link
Copy Markdown
Contributor

@joselsegura joselsegura commented Apr 28, 2026

Description

Adding the workflow example for using reusable workflow for pytest and syncing it across all needed repositories.

Fixes #CCXDEV-15355

Type of change

  • Configuration update

Testing steps

Tested succesfully in ccx-upgrades-data-eng repository

@joselsegura joselsegura requested a review from a team as a code owner April 28, 2026 10:41
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 258d5f27-c26f-4399-a3c4-bd1adbe788d2

📥 Commits

Reviewing files that changed from the base of the PR and between e864013 and ea77bcb.

📒 Files selected for processing (2)
  • .github/sync.yml
  • workflows_examples/pytests.yaml
✅ Files skipped from review due to trivial changes (1)
  • .github/sync.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • workflows_examples/pytests.yaml

Summary by CodeRabbit

  • Chores
    • Updated Python test workflow to use a pinned, stable release for more reliable test runs.
    • Enabled branch coverage reporting to improve code coverage insights.
    • Extended workflow synchronization so the Python test workflow example is distributed across multiple repositories and uses the repository token when invoking the reusable workflow.

Walkthrough

Sync configuration updated to copy workflows_examples/pytests.yaml into target repos’ .github/workflows/pytests.yaml. The pytests workflow now pins the reusable workflow to tag v0.4.1, forwards the CODECOV_TOKEN secret, and enables branch coverage reporting (cov_branch: true).

Changes

Cohort / File(s) Summary
Workflow Sync Configuration
.github/sync.yml
Added rule to sync workflows_examples/pytests.yaml into destination repos at .github/workflows/pytests.yaml for multiple RedHatInsights/* repositories.
Python Test Workflow
workflows_examples/pytests.yaml
Updated workflow display name; tests job now calls the reusable pytests.yaml pinned to tag v0.4.1 (was master), passes CODECOV_TOKEN secret, and sets cov_branch: true to enable branch coverage reporting.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Enabling sync for pytests workflow callers' directly relates to the main changes: extending .github/sync.yml to sync the pytests.yaml workflow and updating the workflow itself.
Description check ✅ Passed The description accurately describes the changeset: adding a workflow example for pytest and syncing it across repositories, with appropriate context and issue reference.
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

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

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@workflows_examples/pytests.yaml`:
- Line 1: The workflow name string "Python unit Tests" should be updated to use
sentence-style capitalization; locate the name field value "Python unit Tests"
in the workflow definition and change it to "Python unit tests" so the workflow
reads consistently.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: d988e71d-1587-40a5-a62a-5d31d007c1f6

📥 Commits

Reviewing files that changed from the base of the PR and between 86eebd3 and e864013.

📒 Files selected for processing (2)
  • .github/sync.yml
  • workflows_examples/pytests.yaml

Comment thread workflows_examples/pytests.yaml Outdated
Bee-lee
Bee-lee previously approved these changes Apr 28, 2026
@joselsegura joselsegura enabled auto-merge April 28, 2026 10:48
@joselsegura joselsegura merged commit 24b85af into RedHatInsights:master Apr 28, 2026
3 checks passed
@joselsegura joselsegura deleted the pytest-example-and-sync branch April 28, 2026 10:52
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.

3 participants