Skip to content

Conversation

@peter-marcisovsky
Copy link
Collaborator

@peter-marcisovsky peter-marcisovsky commented Nov 26, 2025

Description

Running esp-idf usb host and usb device examples with overridden local components on target runners.

Related

Changes

Target runners

  • usb_device target runners in esp-usb run esp-idf device examples
  • usb_host_flash_disk target runners in esp-usb run esp-idf host examples

Conditional workflow run

  • the build_and_run_idf_examples.yml workflow run conditionally specified by the PR author
  • to save some CI build time, as this test don't have to be build for every PR
  • the workflow runs if :
    • The PR title contains release (case-sensitive) - as we usually name the release PR as esp_tinyusb release v2.2.2 for example, the workflow would run automatically on release PRs
    • The PR label tests esp-idf is present - user can add this label to trigger the run

Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@peter-marcisovsky peter-marcisovsky self-assigned this Nov 26, 2025
@peter-marcisovsky peter-marcisovsky added Status: In Progress Issue is being worked on CI CI related PR labels Nov 26, 2025
@peter-marcisovsky peter-marcisovsky force-pushed the ci/target_run_idf_examples branch 2 times, most recently from a1d7fe5 to 73b88d9 Compare November 27, 2025 11:44
@peter-marcisovsky
Copy link
Collaborator Author

peter-marcisovsky commented Nov 27, 2025

The CI workflow correctly accessed esp32p4 usb_host_flash_disk runners, but there is the error about esp32p4 ECO versions mismatch on the esp-idf Latest :

esptool.util.FatalError: '/opt/esp/idf/examples/peripherals/usb/host/usb_host_lib/build_esp32p4_default/bootloader/bootloader.bin' requires chip revision in range [v3.0 - v3.99] (this chip is revision v1.0). Use the force argument to flash anyway.

Build is for esp32p4 ECO5, the runners are not ECO5.
Disabling the esp32p4 usb_host_flash_disk run for esp-idf latest until the new runners are avaliable.

@peter-marcisovsky peter-marcisovsky force-pushed the ci/target_run_idf_examples branch 2 times, most recently from 44dbe6c to 2821fb2 Compare November 28, 2025 09:46
@peter-marcisovsky peter-marcisovsky added tests esp-idf Run esp-idf example tests in PR and removed tests esp-idf Run esp-idf example tests in PR labels Nov 28, 2025
@peter-marcisovsky peter-marcisovsky changed the title feat(ci): Run esp-idf examples on targets feat(ci): Run esp-idf examples on targets release Nov 28, 2025
@peter-marcisovsky peter-marcisovsky changed the title feat(ci): Run esp-idf examples on targets release feat(ci): Run esp-idf examples on targets Nov 28, 2025
@peter-marcisovsky peter-marcisovsky force-pushed the ci/target_run_idf_examples branch from a0c721e to 8f7a8ec Compare November 28, 2025 10:11
@peter-marcisovsky peter-marcisovsky added the tests esp-idf Run esp-idf example tests in PR label Nov 28, 2025
@peter-marcisovsky peter-marcisovsky force-pushed the ci/target_run_idf_examples branch 3 times, most recently from d77c06b to 69a797c Compare November 28, 2025 10:26
examples/peripherals/usb/device:
enable:
- if: (IDF_VERSION >= "6.0.0")
reason: Device examples have been updated to use esp_tinyusb 2.x only on esp-idf latest for now, TODO IDF-14282
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This limitation is partially removed and moved to the build_and_run_idf_examples.yml

As the esp_tinyusb 2.x support is being backported to all the releases, but not all the MR are merged. We will override the local esp_tinyusb component for:

  • IDF 6.1
  • IDF 6.0
  • IDF 5.5
  • IDF 5.3

For the rest of the releases, the device examples in esp-idf will be run with managed components.

@peter-marcisovsky peter-marcisovsky marked this pull request as ready for review November 28, 2025 10:58
@peter-marcisovsky peter-marcisovsky force-pushed the ci/target_run_idf_examples branch 2 times, most recently from 95ec11b to 3303766 Compare November 28, 2025 12:40
@peter-marcisovsky peter-marcisovsky added Status: Reviewing Issue is being reviewed and removed Status: In Progress Issue is being worked on labels Nov 28, 2025
    - build and run esp-idf device and host examples
    - conditional run of the workflow
@peter-marcisovsky peter-marcisovsky force-pushed the ci/target_run_idf_examples branch from 3303766 to 5c73605 Compare December 4, 2025 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI CI related PR Status: Reviewing Issue is being reviewed tests esp-idf Run esp-idf example tests in PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants