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

hardcode openvpn endpoints under ooni infra #1638

Merged

Conversation

ainghazal
Copy link
Contributor

@ainghazal ainghazal commented Jul 28, 2024

changing the experiment as per request, to satisfy reqs for shipping the experiment in time for Aug 2024 release.

Initially I had some concerns about this approach, but it seems to work as long it's ensured that the server has duplicate-cn key in its config.

For what we saw from real-data, the order of magnitude of expected probes executing the experimental card is in the range of 100 per period of 5 minutes.

Checklist

@ainghazal ainghazal requested review from hellais and DecFox as code owners July 28, 2024 18:39
@ainghazal ainghazal force-pushed the feat/hardcode-default-openvpn-endpoint branch 5 times, most recently from 209a238 to 4e79fde Compare July 31, 2024 18:09
changing the experiment as per request, to satisfy reqs for shipping the
experiment in time for Aug 2024 release.

Initially I had some concerns about this approach, but it seems to work
as long it's ensured that the server has `duplicate-cn` key in its
config.

For what we saw from real-data, the order of magnitude of expected
probes executing the experimental card is in the range of 100 per period
of 5 minutes.
@ainghazal ainghazal force-pushed the feat/hardcode-default-openvpn-endpoint branch from 74448c8 to a05eeac Compare July 31, 2024 19:24
Copy link
Contributor

@DecFox DecFox left a comment

Choose a reason for hiding this comment

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

LGTM!

@DecFox DecFox merged commit 6d615d2 into ooni:master Aug 7, 2024
13 of 16 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.

2 participants