Skip to content

Commit

Permalink
Fix chef CI no op and prevent in future (#20183)
Browse files Browse the repository at this point in the history
  • Loading branch information
aBozowski authored and pull[bot] committed Jul 19, 2023
1 parent 2f96365 commit 2788289
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
5 changes: 4 additions & 1 deletion examples/chef/chef.py
Original file line number Diff line number Diff line change
Expand Up @@ -360,7 +360,10 @@ def main(argv: Sequence[str]) -> None:
#

if options.ci:
for device_name in [d for d in _DEVICE_LIST if d in cicd_config["ci_allow_list"]]:
for device_name in cicd_config["ci_allow_list"]:
if device_name not in _DEVICE_LIST:
flush_print(f"{device_name} in CICD config but not {_DEVICE_FOLDER}!")
exit(1)
if options.build_target == "nrfconnect":
shell.run_cmd("export GNUARMEMB_TOOLCHAIN_PATH=\"$PW_ARM_CIPD_INSTALL_DIR\"")
shell.run_cmd(f"cd {_CHEF_SCRIPT_PATH}")
Expand Down
2 changes: 1 addition & 1 deletion examples/chef/cicd_config.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"ci_allow_list": ["rootnode_dimmablelight_gY80DaqEUL"],
"ci_allow_list": ["rootnode_dimmablelight_bCwGYSDpoe"],
"cd_platforms": {
"linux": "linux_x86",
"esp32": "m5stack",
Expand Down

0 comments on commit 2788289

Please sign in to comment.