Skip to content

Add Orchestrator ci to run other ci - #184

Merged
Rahul Devikar (rahuldevikar761) merged 3 commits into
mainfrom
users/radevika/mergeCIPipes
Jan 23, 2026
Merged

Add Orchestrator ci to run other ci#184
Rahul Devikar (rahuldevikar761) merged 3 commits into
mainfrom
users/radevika/mergeCIPipes

Conversation

@rahuldevikar761

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request introduces a CI orchestrator workflow that centralizes and coordinates the execution of existing CI workflows across the repository. The orchestrator detects changes in specific directories and conditionally triggers the appropriate workflows, providing a unified entry point for branch protection requirements.

Changes:

  • Added a new CI orchestrator workflow that uses path filtering to detect changes and conditionally call existing workflows
  • Updated 10 existing CI workflows to support workflow_call and workflow_dispatch triggers for orchestrator integration
  • Added Microsoft copyright headers to 7 workflow files that were missing them

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.github/workflows/ci-orchestrator.yml New orchestrator workflow with change detection, conditional job execution, and unified status reporting
.github/workflows/python-claude-sample.yml Added copyright header and workflow_call/workflow_dispatch triggers
.github/workflows/ci-python-openai-sampleagent.yml Added copyright header and workflow_call/workflow_dispatch triggers
.github/workflows/ci-python-googleadk-sampleagent.yml Added copyright header and workflow_call/workflow_dispatch triggers
.github/workflows/ci-python-agentframework-sampleagent.yml Added copyright header and workflow_call/workflow_dispatch triggers
.github/workflows/ci-nodejs-vercelsdk-sampleagent.yml Added workflow_call/workflow_dispatch triggers (already had copyright header)
.github/workflows/ci-nodejs-openai-sampleagent.yml Added workflow_call/workflow_dispatch triggers (already had copyright header)
.github/workflows/ci-nodejs-langchain-sampleagent.yml Added workflow_call/workflow_dispatch triggers (already had copyright header)
.github/workflows/ci-nodejs-claude-sampleagent.yml Added workflow_call/workflow_dispatch triggers (already had copyright header)
.github/workflows/ci-dotnet-semantickernel-sampleagent.yml Added copyright header and workflow_call/workflow_dispatch triggers
.github/workflows/ci-dotnet-agentframework-sampleagent.yml Added copyright header and workflow_call/workflow_dispatch triggers
Comments suppressed due to low confidence (1)

.github/workflows/python-claude-sample.yml:12

  • The python-claude-sample workflow is configured to trigger on branches [main, develop], which differs from all other workflows that use [main, master]. The orchestrator uses [main, master], meaning python-claude-sample will be called by the orchestrator for master branch changes (via workflow_call) but would not trigger directly on master. This creates inconsistent behavior. Consider standardizing all workflows to use the same branch configuration, or document why python-claude-sample needs different branch triggers.
    branches:
      - main
      - develop

Comment thread .github/workflows/ci-orchestrator.yml Outdated
Comment thread .github/workflows/ci-orchestrator.yml Outdated
@rahuldevikar761
Rahul Devikar (rahuldevikar761) merged commit 8fc74ea into main Jan 23, 2026
18 checks passed
@rahuldevikar761
Rahul Devikar (rahuldevikar761) deleted the users/radevika/mergeCIPipes branch January 23, 2026 19:59
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.

4 participants