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

feat(dnsping): rewrite tests to use netem #1177

Merged
merged 41 commits into from
Jul 6, 2023
Merged

feat(dnsping): rewrite tests to use netem #1177

merged 41 commits into from
Jul 6, 2023

Conversation

bassosimone
Copy link
Contributor

@bassosimone bassosimone commented Jul 6, 2023

Reference issue: ooni/probe#2461

While there, fix a new (harmless but annoying) warning about some locks being copied in a netxlite test.

kelmenhorst and others added 30 commits April 5, 2023 13:44
@bassosimone bassosimone requested a review from hellais as a code owner July 6, 2023 08:40
@bassosimone bassosimone merged commit c357542 into master Jul 6, 2023
6 of 7 checks passed
@bassosimone bassosimone deleted the dnsping branch July 6, 2023 08:49
cyBerta pushed a commit to cyBerta/probe-cli that referenced this pull request Aug 4, 2023
Reference issue: ooni/probe#2461

While there, fix a new (harmless but annoying) warning about some locks
being copied in a netxlite test.

---------

Co-authored-by: kelmenhorst <k.elmenhorst@mailbox.org>
cyBerta pushed a commit to cyBerta/probe-cli that referenced this pull request Aug 4, 2023
Reference issue: ooni/probe#2461

While there, fix a new (harmless but annoying) warning about some locks
being copied in a netxlite test.

---------

Co-authored-by: kelmenhorst <k.elmenhorst@mailbox.org>
Murphy-OrangeMud pushed a commit to Murphy-OrangeMud/probe-cli that referenced this pull request Feb 13, 2024
Reference issue: ooni/probe#2461

While there, fix a new (harmless but annoying) warning about some locks
being copied in a netxlite test.

---------

Co-authored-by: kelmenhorst <k.elmenhorst@mailbox.org>
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.

2 participants