ci: fix v2 ota delta flash testing
What the developer wrote
Authored by Jon Griffiths
ci: fix v2 ota delta flash testing
What changed, and why it matters
This is a one-line change to a GitLab CI pipeline file. It adds a command to load the ESP-IDF build environment before running an over-the-air (OTA) delta firmware test for version 2 hardware. There is no change to device firmware, user-facing behavior, or security-sensitive code.
No security action required. Treat as a normal CI maintenance fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies gitlab/flash.yml, adding . $HOME/esp/esp-idf-5.4/export.sh to the flash_jade_ota_delta_v2_ci job. This script sets up environment variables required by the ESP-IDF toolchain so the subsequent ota_delta_ci.sh invocation can find the correct tools. The change is purely a CI/test environment fix and does not alter firmware, cryptographic, or communication logic.
Changed components
GitLab CI configuration (gitlab/flash.yml)flash_jade_ota_delta_v2_ci CI jobInspect captured patch +1 / −0
diff --git a/gitlab/flash.yml b/gitlab/flash.yml
index c48da3c..f632360 100644
--- a/gitlab/flash.yml
+++ b/gitlab/flash.yml
@@ -136,6 +136,7 @@ flash_jade_ota_delta_v2_ci:
- job: flash_jade_v2_noradio_ci
artifacts: false
script:
+ - . $HOME/esp/esp-idf-5.4/export.sh
- mv build_test_jade_v2_ci build
- mv build_test_jade_v2_noradio_ci build_noradio
- ./ota_delta_ci.sh esp32s3 --skipble
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.