Skip to content

github: workflows: test: build only on windows, test on cm3 - #91

Merged
fabiobaltieri merged 1 commit into
zephyrproject-rtos:mainfrom
fabiobaltieri:test
Aug 21, 2026
Merged

fabiobaltieri merged 1 commit into
zephyrproject-rtos:mainfrom
fabiobaltieri:test

Conversation

@fabiobaltieri

Copy link
Copy Markdown
Member

As of recent discussion it seems like qemu on windows is unreliable run those tests as build only and test on qemu_cortex_m3 which should be more reliable overall.

Link: zephyrproject-rtos/zephyr#113543 (review)

@nashif

nashif commented Jul 21, 2026

Copy link
Copy Markdown
Member

qemu on windows is unreliable

that is too general, only the cortex_m0 is a problem right now, so you can also just skip that and allow the others to continue building/running. Whatever makes sense....

@fabiobaltieri

Copy link
Copy Markdown
Member Author

that is too general, only the cortex_m0 is a problem right now, so you can also just skip that and allow the others to continue building/running. Whatever makes sense....

zephyrproject-rtos/zephyr#112124 qemu_cortex_r5 seems broken as well, I can drop m0 but then we'd just be playing whackamole.

@henrikbrixandersen

Copy link
Copy Markdown
Member

Another option could be to limit the number of concurrent Twister jobs involving these QEMU targets on MS Windows to 1 (suggested by @nashif). This works for zephyrproject-rtos/example-application#105

@fabiobaltieri

Copy link
Copy Markdown
Member Author

Another option could be to limit the number of concurrent Twister jobs involving these QEMU targets on MS Windows to 1 (suggested by @nashif). This works for zephyrproject-rtos/example-application#105

Feels like a hack to be honest but we need to come up with a coherent story between the repository, right now it's just a big mess. Maybe we can run with concurrency one on main (which tests on every push) and build only on the other repo?

@fabiobaltieri

Copy link
Copy Markdown
Member Author

My main fear with the concurrency trick is that we end up in a situation where it works sporadically depending on how loaded the github runner is.

@henrikbrixandersen

Copy link
Copy Markdown
Member

My main fear with the concurrency trick is that we end up in a situation where it works sporadically depending on how loaded the github runner is.

That's a fair concern.

PLATFORMS_FLAGS+="-p native_sim "
fi
PLATFORMS_FLAGS+="-p qemu_cortex_m0 "
PLATFORMS_FLAGS+="-p qemu_cortex_m3 "

@henrikbrixandersen henrikbrixandersen Jul 21, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not sure why the m3 platform was added? Did you mean to remove the m0?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

My understanding is that it's more reliable, undecided on whether to drop m0 or not.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Let's drop it, whatever.

As of recent discussion it seems like qemu on windows is unreliable run
those tests as build only and test on qemu_cortex_m3 which should be
more reliable overall.

Link: zephyrproject-rtos/zephyr#113543 (review)
Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
@fabiobaltieri

Copy link
Copy Markdown
Member Author

Not that it's particularly reliable even with build only https://github.com/fabiobaltieri/action-zephyr-setup/actions/runs/29823824223/attempts/1

@fabiobaltieri
fabiobaltieri requested review from kartben and nashif July 21, 2026 11:18
@henrikbrixandersen

Copy link
Copy Markdown
Member

@kartben, @nashif Please take a look.

@fabiobaltieri
fabiobaltieri merged commit fe224e0 into zephyrproject-rtos:main Aug 21, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants