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

oonimkall/task_test.go:393: CI is broken #1549

Open
bassosimone opened this issue Oct 23, 2020 · 3 comments
Open

oonimkall/task_test.go:393: CI is broken #1549

bassosimone opened this issue Oct 23, 2020 · 3 comments
Assignees
Labels
bug Something isn't working priority/low

Comments

@bassosimone
Copy link
Contributor

See https://github.com/ooni/probe-engine/runs/1296480496. Excerpt of the issue:

2020-10-23T06:17:59.6039064Z ##[error]    task_test.go:393:   []string{
2020-10-23T06:17:59.6040325Z           	... // 9 identical elements
2020-10-23T06:17:59.6040732Z           	"status.progress",
2020-10-23T06:17:59.6041110Z           	"measurement",
2020-10-23T06:17:59.6041772Z         - 	"status.measurement_submission",
2020-10-23T06:17:59.6042357Z         + 	"failure.measurement_submission",
2020-10-23T06:17:59.6042881Z           	"status.measurement_done",
2020-10-23T06:17:59.6043289Z           	"status.end",
2020-10-23T06:17:59.6043639Z           	"task_terminated",
2020-10-23T06:17:59.6043921Z           }
2020-10-23T06:17:59.6044138Z         
2020-10-23T06:17:59.6045081Z --- FAIL: TestIntegrationInterruptExampleWithInput (7.39s)

This may depend on the flip between the old and the new infrastructure during Sprint 24.

@bassosimone bassosimone self-assigned this Oct 23, 2020
@bassosimone
Copy link
Contributor Author

bassosimone commented Oct 23, 2020

An easy way to reproduce is running ./miniooni -i http://www.x.org -vvv example_with_input_non_interruptible. This seems something that changed in the refactored backend. I will disable the failing integration test for the time being. Disabling the failing test is quite useful for the time being, because it enables me to merge other fixes with confidence.

@bassosimone
Copy link
Contributor Author

Merged into ooni/backend#461

@bassosimone
Copy link
Contributor Author

Reopening since it's convenient for me to separate this from ooni/backend#461

@bassosimone bassosimone reopened this Nov 9, 2020
@bassosimone bassosimone changed the title oonimkall/task_test.go:393: build is broken oonimkall/task_test.go:393: CI is broken Dec 2, 2020
@bassosimone bassosimone transferred this issue from ooni/probe-engine Jun 7, 2021
@bassosimone bassosimone added bug Something isn't working priority/low labels Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority/low
Projects
Status: No status
Development

No branches or pull requests

1 participant