Skip to content

dist/testbed-support: add reset after flash for IoT-LAB nodes#22287

Open
baptleduc wants to merge 1 commit into
RIOT-OS:masterfrom
baptleduc:fix/iotlab-flash-recipe
Open

dist/testbed-support: add reset after flash for IoT-LAB nodes#22287
baptleduc wants to merge 1 commit into
RIOT-OS:masterfrom
baptleduc:fix/iotlab-flash-recipe

Conversation

@baptleduc
Copy link
Copy Markdown
Contributor

@baptleduc baptleduc commented May 12, 2026 •
edited
Loading

makefile.iotlab.single.inc.mk does not define flash-recipe, so it falls back to the default defined in Makefile.include:832:

define default-flash-recipe
  $(call check_cmd,$(FLASHER),Flash program)
  $(PROGRAMMER_FLASH)
endef
flash-recipe ?= $(default-flash-recipe)

This default doesn't reset the board after flashing. As a result, the node does not restart automatically and produces no output until a manual reset is triggered.

This was observed on b-l072z-lrwan1 nodes at the IoT-LAB Saclay site: examples/basic/hello-world produced no serial output after make flash term until the node was reset manually.

Fix: define an iotlab-flash-recipe that appends a reset step after flashing.

Testing procedure

Tested on b-l072z-lrwan1 at IoT-LAB Saclay. After flashing, the node now resets automatically and serial output appears immediately without manual intervention.

Issues/PRs references

Originally posted in #22271 (comment)

Declaration of AI Tools / LLMs usage

  • Claude Code for code review and guidance, with user authorship of the implementation

@baptleduc
dist/testbed-support: add flash-recipe to iotlab
def4576 
Without an explicit flash-recipe, the default was used which skips
the node reset. The iotlab-flash-recipe now runs reset after flash
to ensure the firmware starts correctly.

Signed-off-by: Baptiste Le Duc <baptiste.le-duc@vates.tech>
@github-actions github-actions Bot added Area: build system Area: Build system Area: tools Area: Supplementary tools labels May 12, 2026
@baptleduc baptleduc mentioned this pull request May 12, 2026
Open
@crasbe crasbe added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels May 12, 2026
@riot-ci
Copy link
Copy Markdown

riot-ci commented May 12, 2026

Murdock results

✔️ PASSED

def4576 dist/testbed-support: add flash-recipe to iotlab

Success Failures Total Runtime
1 0 1 01m:21s

Artifacts

@AnnsAnns
Copy link
Copy Markdown
Member

AnnsAnns commented May 13, 2026

🤔 I wonder whether this also fixes some release test weirdness with iotlab

@crasbe
Copy link
Copy Markdown
Contributor

crasbe commented May 13, 2026

🤔 I wonder whether this also fixes some release test weirdness with iotlab

There is only one way to find out... Do another release! 😈

@baptleduc
Copy link
Copy Markdown
Contributor Author

baptleduc commented May 14, 2026

🤔 I wonder whether this also fixes some release test weirdness with iotlab

There is only one way to find out... Do another release! 😈

Could I help you in any way ?

@crasbe
Copy link
Copy Markdown
Contributor

crasbe commented May 14, 2026

🤔 I wonder whether this also fixes some release test weirdness with iotlab

There is only one way to find out... Do another release! 😈

Could I help you in any way ?

That was just a joke. The release tests can be run without doing a release too :D

@baptleduc
Copy link
Copy Markdown
Contributor Author

baptleduc commented May 14, 2026 •
edited
Loading

🤔 I wonder whether this also fixes some release test weirdness with iotlab

There is only one way to find out... Do another release! 😈

Could I help you in any way ?

That was just a joke. The release tests can be run without doing a release too :D

Oops 😅😂

But do you want me to open a PR in te iot lab repo ?

@baptleduc
Copy link
Copy Markdown
Contributor Author

baptleduc commented May 24, 2026

Hello,
Is it possible to have an update please ?
Thanks in advance

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Area: build system Area: Build system Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@baptleduc @riot-ci @AnnsAnns @crasbe