Skip to content

Commit c720572

Browse files
Damian-Nordickkasperczyk-no
authored andcommitted
[nrf noup] Use "continue-on-error" for examples in CI
Make building nRF Connect examples optional in github workflows in a sense that they do not fail the workflow if they do not compile. The reason is that the CI is often run for downstream patches, that are required to proceed with changes in NCS or Zephyr and we do not want to force developers to update examples in sdk-connectedhomeip at that moment - they will be updated in the upstream later. Signed-off-by: Damian Krolik <[email protected]>
1 parent 6ba95f1 commit c720572

File tree

1 file changed

+39
-0
lines changed

1 file changed

+39
-0
lines changed

.github/workflows/examples-nrfconnect.yaml

+39
Original file line numberDiff line numberDiff line change
@@ -83,20 +83,33 @@ jobs:
8383
- name: Update nRF Connect SDK revision to the currently recommended.
8484
timeout-minutes: 15
8585
run: scripts/run_in_build_env.sh "python3 scripts/setup/nrfconnect/update_ncs.py --update --shallow"
86+
continue-on-error: true
8687
- name: Run unit tests of factory data generation script
8788
timeout-minutes: 15
8889
run: scripts/run_in_build_env.sh "./scripts/tools/nrfconnect/tests/test_generate_factory_data.py"
8990
- name: Build example nRF Connect SDK Lock App on nRF52840 DK
9091
if: github.event_name == 'push' || steps.changed_paths.outputs.nrfconnect == 'true'
92+
continue-on-error: true
9193
timeout-minutes: 15
9294
run: |
9395
scripts/examples/nrfconnect_example.sh lock-app nrf52840dk_nrf52840
9496
.environment/pigweed-venv/bin/python3 scripts/tools/memory/gh_sizes.py \
9597
nrfconnect nrf52840dk_nrf52840 lock-app \
9698
examples/lock-app/nrfconnect/build/zephyr/zephyr.elf \
9799
/tmp/bloat_reports/
100+
- name: Build example nRF Connect SDK Lighting App on nRF52840 DK
101+
if: github.event_name == 'push' || steps.changed_paths.outputs.nrfconnect == 'true'
102+
continue-on-error: true
103+
timeout-minutes: 15
104+
run: |
105+
scripts/examples/nrfconnect_example.sh lighting-app nrf52840dk_nrf52840
106+
.environment/pigweed-venv/bin/python3 scripts/tools/memory/gh_sizes.py \
107+
nrfconnect nrf52840dk_nrf52840 lighting-app \
108+
examples/lighting-app/nrfconnect/build/zephyr/zephyr.elf \
109+
/tmp/bloat_reports/
98110
- name: Build example nRF Connect SDK Lighting App on nRF52840 Dongle
99111
if: github.event_name == 'push' || steps.changed_paths.outputs.nrfconnect == 'true'
112+
continue-on-error: true
100113
timeout-minutes: 15
101114
run: |
102115
scripts/examples/nrfconnect_example.sh lighting-app nrf52840dongle_nrf52840 -DCONF_FILE=prj_no_dfu.conf -DCONFIG_CHIP_ROTATING_DEVICE_ID=y
@@ -106,6 +119,7 @@ jobs:
106119
/tmp/bloat_reports/
107120
- name: Build example nRF Connect SDK Lighting App on nRF52840 DK with RPC
108121
if: github.event_name == 'push' || steps.changed_paths.outputs.nrfconnect == 'true'
122+
continue-on-error: true
109123
timeout-minutes: 20
110124
run: |
111125
scripts/examples/nrfconnect_example.sh lighting-app nrf52840dk_nrf52840 -DOVERLAY_CONFIG=rpc.overlay
@@ -124,6 +138,7 @@ jobs:
124138
/tmp/bloat_reports/
125139
- name: Build example nRF Connect SDK Shell on nRF52840 DK
126140
if: github.event_name == 'push' || steps.changed_paths.outputs.shell == 'true'
141+
continue-on-error: true
127142
timeout-minutes: 15
128143
run: |
129144
scripts/examples/nrfconnect_example.sh shell nrf52840dk_nrf52840
@@ -133,15 +148,37 @@ jobs:
133148
/tmp/bloat_reports/
134149
- name: Build example nRF Connect SDK Pigweed on nRF52840 DK
135150
if: github.event_name == 'push' || steps.changed_paths.outputs.pigweed-app == 'true'
151+
continue-on-error: true
136152
timeout-minutes: 15
137153
run: |
138154
scripts/examples/nrfconnect_example.sh pigweed-app nrf52840dk_nrf52840
139155
.environment/pigweed-venv/bin/python3 scripts/tools/memory/gh_sizes.py \
140156
nrfconnect nrf52840dk_nrf52840 pigweed-app \
141157
examples/pigweed-app/nrfconnect/build/zephyr/zephyr.elf \
142158
/tmp/bloat_reports/
159+
- name: Build example nRF Connect SDK Lock App on nRF5340 DK
160+
if: github.event_name == 'push' || steps.changed_paths.outputs.nrfconnect == 'true'
161+
continue-on-error: true
162+
timeout-minutes: 15
163+
run: |
164+
scripts/examples/nrfconnect_example.sh lock-app nrf5340dk_nrf5340_cpuapp
165+
.environment/pigweed-venv/bin/python3 scripts/tools/memory/gh_sizes.py \
166+
nrfconnect nrf5340dk_nrf5340_cpuapp lock-app \
167+
examples/lock-app/nrfconnect/build/zephyr/zephyr.elf \
168+
/tmp/bloat_reports/
169+
- name: Build example nRF Connect SDK Lighting App on nRF5340 DK
170+
if: github.event_name == 'push' || steps.changed_paths.outputs.nrfconnect == 'true'
171+
continue-on-error: true
172+
timeout-minutes: 15
173+
run: |
174+
scripts/examples/nrfconnect_example.sh lighting-app nrf5340dk_nrf5340_cpuapp
175+
.environment/pigweed-venv/bin/python3 scripts/tools/memory/gh_sizes.py \
176+
nrfconnect nrf5340dk_nrf5340_cpuapp lighting-app \
177+
examples/lighting-app/nrfconnect/build/zephyr/zephyr.elf \
178+
/tmp/bloat_reports/
143179
- name: Build example nRF Connect SDK Pump App on nRF52840 DK
144180
if: github.event_name == 'push' || steps.changed_paths.outputs.nrfconnect == 'true'
181+
continue-on-error: true
145182
timeout-minutes: 15
146183
run: |
147184
scripts/examples/nrfconnect_example.sh pump-app nrf52840dk_nrf52840
@@ -151,6 +188,7 @@ jobs:
151188
/tmp/bloat_reports/
152189
- name: Build example nRF Connect SDK Pump Controller App on nRF52840 DK
153190
if: github.event_name == 'push' || steps.changed_paths.outputs.nrfconnect == 'true'
191+
continue-on-error: true
154192
timeout-minutes: 15
155193
run: |
156194
scripts/examples/nrfconnect_example.sh pump-controller-app nrf52840dk_nrf52840
@@ -159,6 +197,7 @@ jobs:
159197
examples/pump-controller-app/nrfconnect/build/zephyr/zephyr.elf \
160198
/tmp/bloat_reports/
161199
- name: Build example nRF Connect SDK All Clusters App on nRF52840 DK
200+
continue-on-error: true
162201
timeout-minutes: 20
163202
run: |
164203
scripts/examples/nrfconnect_example.sh all-clusters-app nrf52840dk_nrf52840 -DCONF_FILE=prj_dfu.conf

0 commit comments

Comments
 (0)