Skip to content

Migrate "Linux CPU Minimal Build E2E CI Pipeline" to Github Actions #24222

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

snnn
Copy link
Member

@snnn snnn commented Mar 28, 2025

  1. Migrate "Linux CPU Minimal Build E2E CI Pipeline" and "onnxruntime-binary-size-checks-ci-pipeline" to Github Action
  2. Add the support for building ONNX Runtime minimal build with vcpkg.
  3. Change Github Action's triggers
  4. Auto format the yaml files with ruamel.yaml

Because this change, the pipelines' triggers are set as following:

on:
  push:
    branches: [ main, 'rel-*']
  pull_request:
    branches: [ main, 'rel-*']
concurrency:
  group: ${{ github.workflow }}-${{ github.ref }}
  cancel-in-progress: true

I set "cancel-in-progress: true" because for pipeline runs triggered by pull requests if the pull request was updated(a new commit was added there), the old pipeline runs can be cancelled. However, this setting doesn't work well for the runs triggered by "push" events for the main branch. Let's say, we merged a PR , then it triggered this pipeline. Then before the pipeline is finished, we merged another PR. Then the old pipeline run will be cancelled. But we do want it to be cancelled. Each commit in the main branch should be verified.

@snnn snnn changed the title Change triggers Change Github Action's triggers Mar 28, 2025
@snnn snnn changed the title Change Github Action's triggers Migrate "Linux CPU Minimal Build E2E CI Pipeline" to Github Actions Apr 23, 2025
@snnn snnn closed this Apr 24, 2025
@snnn snnn reopened this Apr 24, 2025
@snnn snnn closed this Apr 24, 2025
@snnn snnn deleted the snnn/trigger1 branch April 24, 2025 17:44
@snnn
Copy link
Member Author

snnn commented Apr 24, 2025

Closed and recreated this PR because the CLA check got stuck here.

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.

None yet

1 participant