-
Notifications
You must be signed in to change notification settings - Fork 102
[ci] optimize ci runtime ⚡️ #588
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,11 +5,19 @@ on: | |
| pull_request: | ||
| branches: [main] | ||
| types: [opened, synchronize, reopened] # trigger on PRs | ||
| paths-ignore: | ||
| - "docs/**" | ||
| - "*.md" | ||
| paths: | ||
| - "nix-builder/**" | ||
| - "kernel-builder/**" | ||
| - "examples/kernels/**" | ||
| - "**/*.nix" | ||
| - "**/flake.lock" | ||
| - ".github/workflows/build_kernel_macos.yaml" | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. See above. |
||
| workflow_dispatch: | ||
|
|
||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
| cancel-in-progress: ${{ github.event_name == 'pull_request' }} | ||
|
|
||
| jobs: | ||
| build: | ||
| name: Build kernel | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,11 +5,19 @@ on: | |
| pull_request: | ||
| branches: [main] | ||
| types: [opened, synchronize, reopened] # trigger on PRs | ||
| paths-ignore: | ||
| - "docs/**" | ||
| - "*.md" | ||
| paths: | ||
| - "nix-builder/**" | ||
| - "kernel-builder/**" | ||
| - "examples/kernels/**" | ||
| - "**/*.nix" | ||
| - "**/flake.lock" | ||
| - ".github/workflows/build_kernel_rocm.yaml" | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. See above. |
||
| workflow_dispatch: | ||
|
|
||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
| cancel-in-progress: ${{ github.event_name == 'pull_request' }} | ||
|
|
||
| jobs: | ||
| build: | ||
| name: Build kernel | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,11 +5,19 @@ on: | |
| pull_request: | ||
| branches: [main] | ||
| types: [opened, synchronize, reopened] # trigger on PRs | ||
| paths-ignore: | ||
| - "docs/**" | ||
| - "*.md" | ||
| paths: | ||
| - "nix-builder/**" | ||
| - "kernel-builder/**" | ||
| - "examples/kernels/**" | ||
| - "**/*.nix" | ||
| - "**/flake.lock" | ||
| - ".github/workflows/build_kernel_windows.yaml" | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not dependent on nix-builder, apart from that, see above. |
||
| workflow_dispatch: | ||
|
|
||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
| cancel-in-progress: ${{ github.event_name == 'pull_request' }} | ||
|
|
||
| jobs: | ||
| build: | ||
| strategy: | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,11 +5,18 @@ on: | |
| pull_request: | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't think there's any dependency on |
||
| branches: [main] | ||
| types: [opened, synchronize, reopened] # trigger on PRs | ||
| paths-ignore: | ||
| - "docs/**" | ||
| - "*.md" | ||
| paths: | ||
| - "nix-builder/**" | ||
| - "**/*.nix" | ||
| - "**/flake.lock" | ||
| - "docs/source/builder/build-variants.md" | ||
| - ".github/workflows/check_variants.yaml" | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This check is really cheap, so I think it's better not to run the risk of missing a dependency. |
||
| workflow_dispatch: | ||
|
|
||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
| cancel-in-progress: ${{ github.event_name == 'pull_request' }} | ||
|
|
||
| jobs: | ||
| build: | ||
| name: Check build variants | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,9 +6,11 @@ on: | |
| pull_request: | ||
| branches: [main] | ||
| types: [opened, synchronize, reopened] # trigger on PRs | ||
| paths-ignore: | ||
| - "docs/**" | ||
| - "*.md" | ||
| paths: | ||
| - "kernels/**" | ||
| - "kernels-data/bindings/python/**" | ||
| - "kernel-abi-check/bindings/python/**" | ||
| - ".github/workflows/test_kernels.yaml" | ||
|
Comment on lines
+9
to
+13
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't think there's any dependency on |
||
| workflow_dispatch: | ||
|
|
||
| concurrency: | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't see any other dependency.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kernelsPython package (to validate thatget_kernelworks correctly as a post-build hook).In other words, pretty much everything.