Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Repair GH pipeline: QA integration tests for samples is broken #250

Merged
merged 3 commits into from
Jul 13, 2024
Merged

Conversation

rexut
Copy link
Member

@rexut rexut commented Jul 13, 2024

Restructuring QA integration tests for samples

In addition to Bridle's own examples, it should also be possible to build specific Zephyr upstream examples with an alternative configuration base with the Bridle-owned specifications. For this purpose, the new tags zephyr and bridle must be introduced and respected. For the GitHub workflows, this makes the explicit naming of individual test roots unnecessary. The selection is made exclusively via the newly introduced tags.

Avoid RAM overflow on qemu_cortex_m0

Since Zephyr upstream PR 74435, the sensor shell commands rely on a complete integration of the sensor ASYNC API incl. RTIO subsystem with a thread pool for a dedicated RTIO work queue. This exceeds the 16KB RAM of the Cortex M0 emulator by far. The only option available at the moment is to disable the sensor shell commands.

Protect supported display shields

In preparation to PR #238, #243 and #245 the GH pipeline have to catch all currently display shields by simple builds as integration tests.

rexut added 3 commits July 2, 2024 17:21
In addition to Bridle's own examples, it should also be possible
to build specific Zephyr upstream examples with an alternative
configuration base with the Bridle-owned specifications. For this
purpose, the new tags `zephyr` and `bridle` must be introduced and
respected. For the GitHub workflows, this makes the explicit naming
of individual test roots unnecessary. The selection is made
exclusively via the newly introduced tags.

Signed-off-by: Stephan Linz <[email protected]>
Since Zephyr upstream PR 74435, the sensor shell commands rely on
a complete integration of the sensor ASYNC API incl. RTIO subsystem
with a thread pool for a dedicated RTIO work queue. This exceeds
the 16KB RAM of the Cortex M0 emulator by far. The only option
available at the moment is to disable the sensor shell commands.

Signed-off-by: Stephan Linz <[email protected]>
- waveshare_pico_lcd_1_14: Waveshare Pico LCD 1.14
- waveshare_pico_lcd_2: Waveshare Pico LCD 2
- waveshare_pico_restouch_lcd_3_5: Waveshare Pico ResTouch LCD 3.5
- waveshare_2_4_lcd: Waveshare 2.4 LCD

Signed-off-by: Stephan Linz <[email protected]>
@rexut rexut marked this pull request as ready for review July 13, 2024 06:52
@rexut rexut self-assigned this Jul 13, 2024
@rexut rexut added this to the v3.6.99 milestone Jul 13, 2024
@rexut rexut requested a review from tobiaskaestner July 13, 2024 07:04
Copy link
Contributor

@tobiaskaestner tobiaskaestner left a comment

Choose a reason for hiding this comment

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

Lgtm

@rexut rexut merged commit 3568c9d into main Jul 13, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants