Skip to content
New issue

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

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

Already on GitHub? # to your account

08-interop: deprecate Contiki tests in favor of Contiki-NG #280

Closed
wants to merge 1 commit into from

Conversation

jia200x
Copy link
Member

@jia200x jia200x commented Apr 14, 2023

Contribution description

This PR deprecates the Contiki test in favor of Contiki-NG.
The original test is hard to run and requires the CC2538DK evaluation board, which costs ~500 USD and only a few maintainers have access to it.

The new test consist of ICMPv6 exchange between a RIOT node and a Contiki-NG node using link-local addresses.
Contiki-NG is easy to compile and flash, as the toolchain and flashers run inside Docker.

Testing procedure

  • Follow the instructions of the new Contiki-NG test (08-interop: Task 03).
  • Make sure there aren't any format issues in the new description.

Issues/PRs references

#279

@jia200x jia200x requested review from miri64 and MrKevinWeiss April 14, 2023 14:00
@jia200x jia200x force-pushed the pr/deprecate_contiki branch from 9972482 to 88747cf Compare April 14, 2023 14:01
@jia200x jia200x mentioned this pull request Apr 14, 2023
86 tasks
Copy link
Contributor

@MrKevinWeiss MrKevinWeiss left a comment

Choose a reason for hiding this comment

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

It seems like there is a PAN ID set step missing

Copy link
Contributor

@MrKevinWeiss MrKevinWeiss left a comment

Choose a reason for hiding this comment

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

After setting pan ID

contiki-ng

#f4ce.36c6.d8d1.e340> ping fe80::204:2519:1801:86ee
Pinging fe80::204:2519:1801:86ee
Received ping reply from fe80::204:2519:1801:86ee, len 4, ttl 255, delay 31 ms

riot

> ping fe80::f6ce:36c6:d8d1:e340
ping fe80::f6ce:36c6:d8d1:e340
12 bytes from fe80::f6ce:36c6:d8d1:e340%6: icmp_seq=0 ttl=64 rssi=-64 dBm time=6.954 ms
12 bytes from fe80::f6ce:36c6:d8d1:e340%6: icmp_seq=1 ttl=64 rssi=-48 dBm time=7.900 ms
12 bytes from fe80::f6ce:36c6:d8d1:e340%6: icmp_seq=2 ttl=64 rssi=-49 dBm time=38.231 ms

--- fe80::f6ce:36c6:d8d1:e340 PING statistics ---
3 packets transmitted, 3 packets received, 0% packet loss
round-trip min/avg/max = 6.954/17.695/38.231 ms

Pretty easy to setup and much more applicable. I would say with the PAN fix it should be fine!

@jia200x jia200x force-pushed the pr/deprecate_contiki branch from 88747cf to 4adb7f2 Compare April 18, 2023 09:07
@jia200x
Copy link
Member Author

jia200x commented Apr 18, 2023

amended directly!

@MrKevinWeiss
Copy link
Contributor

superseded by #292

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants