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

task03: add stress test for native #95

Merged
merged 1 commit into from
Feb 4, 2019
Merged

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Jan 29, 2019

Intended to check if there is a regression to RIOT-OS/RIOT#6123.

Do not merge until RIOT-OS/RIOT#6123 is closed! (current fix proposal in RIOT-OS/RIOT#10891).

Intended to check if there is a regression to [1].

[1] RIOT-OS/RIOT#6123
@miri64 miri64 requested a review from aabadie January 29, 2019 08:34
@miri64
Copy link
Member Author

miri64 commented Feb 1, 2019

Do not merge until RIOT-OS/RIOT#6123 is closed! (current fix proposal in RIOT-OS/RIOT#10891).

done.

@miri64
Copy link
Member Author

miri64 commented Feb 4, 2019

@aabadie this should be included in the testings of RC2

Copy link
Contributor

@aabadie aabadie left a comment

Choose a reason for hiding this comment

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

ACK and go!

@aabadie aabadie merged commit 1af4687 into master Feb 4, 2019
@miri64 miri64 deleted the task03/fix/native-stress branch February 4, 2019 15:57
@@ -44,3 +44,21 @@ ICMPv6 echo request/reply exchange between two native nodes.
### Result

<1% packets lost on the pinging node.

Task #10 - ICMPv6 stress test on native (1 hour)
Copy link
Contributor

Choose a reason for hiding this comment

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

oups, I merged this a bit fast, this should be task #04.

Copy link
Member Author

Choose a reason for hiding this comment

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

You are right! However, I need to leave now. Can you fix this?

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

Successfully merging this pull request may close these issues.

2 participants