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(miniooni): introduce the --repeat-every command line flag #819

Merged
merged 21 commits into from
Jul 8, 2022

Conversation

bassosimone
Copy link
Contributor

@bassosimone bassosimone commented Jun 28, 2022

Until OONI Run v2 has support for repeating the measurement with a schedule, introduce a command line flag requested by users to repeat a measurement every given number of seconds.

Part of ooni/probe#2184

So we don't need to write disgusting code such as

```JSON
{
  "HTTPEnabled": "true"
}
```

and we can instead write this:

```JSON
{
  "HTTPEnabled": true
}
}
```
Conflicts:
	internal/cmd/miniooni/libminiooni.go
	internal/engine/experimentbuilder.go
	internal/engine/experimentbuilder_test.go
Conflicts:
	internal/engine/experiment.go
	internal/engine/experimentbuilder.go
Conflicts:
	internal/cmd/miniooni/libminiooni.go
	internal/oonirun/experiment.go
While there, add v2 integration tests

Conflicts:
	internal/cmd/miniooni/libminiooni.go
	internal/oonirun/link.go
	internal/oonirun/v1.go
Conflicts:
	internal/cmd/miniooni/libminiooni.go
	internal/oonirun/v2.go
	internal/oonirun/v2_test.go
@bassosimone bassosimone changed the title feat(miniooni): sketch out oonirun v1 and v2 support feat(miniooni): introduce the --repeat-every command line flag Jul 8, 2022
@bassosimone bassosimone marked this pull request as ready for review July 8, 2022 15:01
@bassosimone bassosimone requested a review from hellais as a code owner July 8, 2022 15:01
@bassosimone bassosimone merged commit 8aad36a into master Jul 8, 2022
@bassosimone bassosimone deleted the oonirun branch July 8, 2022 15:04
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.

1 participant