Skip to content

Commit eb7cac6

Browse files
authored
Set trigger: none in the CI tests (quokka-astro#1593)
### Description This way, the GPU CI tests will only be tiggered via a manual '/azp run' in a PR targeted at any branch. Commits into development branch won't trigger CI tests. Also added the submodules to the include list so that a pointer change (the commit SHA reference) at submodules like extern/amrex itself will trigger CI test. ### Checklist _Before this pull request can be reviewed, all of these tasks should be completed. Denote completed tasks with an `x` inside the square brackets `[ ]` in the Markdown source below:_ - [ ] I have added a description (see above). - [ ] I have added a link to any related issues (if applicable; see above). - [ ] I have read the [Contributing Guide](https://github.com/quokka-astro/quokka/blob/development/CONTRIBUTING.md). - [ ] I have added tests for any new physics that this PR adds to the code. - [ ] *(For quokka-astro org members)* I have manually triggered the GPU tests with the magic comment `/azp run`.
1 parent 1c4c7c0 commit eb7cac6

2 files changed

Lines changed: 20 additions & 49 deletions

File tree

.ci/azure-pipelines-amdgpu.yml

Lines changed: 10 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -3,31 +3,9 @@
33
# Add steps that build, run tests, deploy, and more:
44
# https://aka.ms/yaml
55

6-
trigger:
7-
branches:
8-
include:
9-
- development
10-
paths:
11-
include:
12-
- src/**
13-
- tests/**
14-
- inputs/**
15-
- CMakeLists.txt
16-
- .ci/azure-pipelines-amdgpu.yml
17-
- extern/cooling/*.h5
18-
- extern/amrex/**
19-
- extern/Microphysics/**
20-
- extern/fmt/**
21-
- extern/yaml-cpp/**
22-
- extern/openPMD-api/**
23-
- extern/AMReX-Hydro/**
24-
exclude:
25-
- docs/**
26-
- paper/**
27-
- scripts/**
28-
- '*.md'
29-
- '*.rst'
30-
- '*.txt'
6+
# Disable automatic CI trigger on push/merge to development branch.
7+
# Pipeline can still be triggered manually via `/azp run` or on PR validation.
8+
trigger: none
319

3210
pr:
3311
autoCancel: true
@@ -42,12 +20,19 @@ pr:
4220
- CMakeLists.txt
4321
- .ci/azure-pipelines-amdgpu.yml
4422
- extern/cooling/*.h5
23+
- extern/amrex
4524
- extern/amrex/**
25+
- extern/Microphysics
4626
- extern/Microphysics/**
27+
- extern/fmt
4728
- extern/fmt/**
29+
- extern/yaml-cpp
4830
- extern/yaml-cpp/**
31+
- extern/openPMD-api
4932
- extern/openPMD-api/**
33+
- extern/AMReX-Hydro
5034
- extern/AMReX-Hydro/**
35+
- extern/turbulence
5136
- extern/turbulence/**
5237
exclude:
5338
- docs/**

.ci/azure-pipelines.yml

Lines changed: 10 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -3,30 +3,9 @@
33
# Add steps that build, run tests, deploy, and more:
44
# https://aka.ms/yaml
55

6-
trigger:
7-
branches:
8-
include:
9-
- development
10-
paths:
11-
include:
12-
- src/**
13-
- inputs/**
14-
- CMakeLists.txt
15-
- .ci/azure-pipelines.yml
16-
- extern/cooling/*.h5
17-
- extern/amrex/**
18-
- extern/Microphysics/**
19-
- extern/fmt/**
20-
- extern/yaml-cpp/**
21-
- extern/openPMD-api/**
22-
- extern/AMReX-Hydro/**
23-
exclude:
24-
- docs/**
25-
- paper/**
26-
- scripts/**
27-
- '*.md'
28-
- '*.rst'
29-
- '*.txt'
6+
# Disable automatic CI trigger on push/merge to development branch.
7+
# Pipeline can still be triggered manually via `/azp run` or on PR validation.
8+
trigger: none
309

3110
pr:
3211
autoCancel: true
@@ -41,12 +20,19 @@ pr:
4120
- CMakeLists.txt
4221
- .ci/azure-pipelines.yml
4322
- extern/cooling/*.h5
23+
- extern/amrex
4424
- extern/amrex/**
25+
- extern/Microphysics
4526
- extern/Microphysics/**
27+
- extern/fmt
4628
- extern/fmt/**
29+
- extern/yaml-cpp
4730
- extern/yaml-cpp/**
31+
- extern/openPMD-api
4832
- extern/openPMD-api/**
33+
- extern/AMReX-Hydro
4934
- extern/AMReX-Hydro/**
35+
- extern/turbulence
5036
- extern/turbulence/**
5137
exclude:
5238
- docs/**

0 commit comments

Comments
 (0)