Skip to content

[b/474913468, 479101966] CI/CD Optimization - #544

Merged
TalShafir1 merged 13 commits into
mainfrom
split-github-actions
Feb 11, 2026
Merged

[b/474913468, 479101966] CI/CD Optimization#544
TalShafir1 merged 13 commits into
mainfrom
split-github-actions

Conversation

@AmitJ98

@AmitJ98 AmitJ98 commented Feb 11, 2026

Copy link
Copy Markdown
Contributor

CI/CD Optimization: uv Adoption and Conditional Windows Workflows

This PR focuses on optimizing the CI/CD pipeline for speed, consistency, and resource efficiency.

Key Changes:

  • 🎯 Targeted Workflow Execution:

    • Configured every workflow to run strictly based on the specific content type modified.
  • ⚡ Fast Dependency Management (uv):

    • Migrated all package installation steps across both Linux and Windows workflows to use uv instead of standard pip.
    • This ensures faster setup times and consistent environment resolution across all CI jobs.
  • 🪟 Optimized Windows Workflows:

    • Refactored windows_workflows.yml to include a changes job (using dorny/paths-filter).
    • Windows jobs (Tests, Validations, Builds) now run conditionally based on modified files.
      • Example: Touching a playbook will no longer trigger integration tests on Windows.
    • This aligns the Windows logic with the existing validations.yml structure, saving significant CI resources.

@AmitJ98 AmitJ98 self-assigned this Feb 11, 2026
@AmitJ98
AmitJ98 requested a review from a team as a code owner February 11, 2026 10:12
@AmitJ98 AmitJ98 added Enhancement New feature or request Github Actions Pull requests that update GitHub Actions code labels Feb 11, 2026
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@TalShafir1 TalShafir1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have one main question:

Do we want to keep these in different files?
If so let's separate them to folders based on content type and then on contribution source

integrations/google
integrations/third_party
playbooks...
mp
tipcommon
...

If not (this is the plan we wanted to go with when more content types are added):
Let's have one mega flow that controls everything, and based on the PR's changed files it will know which other flows to run - tests, code checks, builds etc...

Comment thread .github/workflows/code_checks.yml Outdated
Comment thread .github/workflows/test_third_party_integrations.yml Outdated
Comment thread .github/workflows/test_third_party_integrations_commenter.yml Outdated
@TalShafir1

Copy link
Copy Markdown
Member

I have one main question:

Do we want to keep these in different files? If so let's separate them to folders based on content type and then on contribution source

integrations/google integrations/third_party playbooks... mp tipcommon ...

If not (this is the plan we wanted to go with when more content types are added): Let's have one mega flow that controls everything, and based on the PR's changed files it will know which other flows to run - tests, code checks, builds etc...

This might will also help us determine what's required and what not. One single required to pass flow that knows which sub flows to run

@TalShafir1

Copy link
Copy Markdown
Member

FYI Since some job names were changed they will not run here

@TalShafir1
TalShafir1 merged commit 2efc72a into main Feb 11, 2026
59 checks passed
@TalShafir1
TalShafir1 deleted the split-github-actions branch February 11, 2026 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request Github Actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants