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

TestPullInvalidIgnoredSR/SO is flaky #5796

Closed
calmh opened this issue Jun 14, 2019 · 2 comments

Comments

Projects
None yet
3 participants
@calmh
Copy link
Member

commented Jun 14, 2019

16:14:45 INFO: No stored folder metadata for "default": recalculating
16:14:46 INFO: Ready to synchronize "default" (default) (sendreceive)
16:14:46 INFO: Completed initial scan of sendreceive folder "default" (default)
16:14:46 INFO: No stored folder metadata for "default": recalculating
16:14:46 INFO: Ready to synchronize "default" (default) (sendreceive)
16:14:46 INFO: Device AIR6LPZ-7K4PTTV-UXQSMUU-CPQ5YWH-OEDFIIQ-JUG777G-2YQXXR5-YD6AWQR client is " " named "" at 
16:14:46 INFO: Completed initial scan of sendreceive folder "default" (default)
16:14:47 INFO: Folder "default" (default) isn't making sync progress - retrying in 1m0.000605946s.
16:14:47 INFO: Connection to AIR6LPZ-7K4PTTV-UXQSMUU-CPQ5YWH-OEDFIIQ-JUG777G-2YQXXR5-YD6AWQR at  closed: Syncthing is being stopped
    requests_test.go:391: File ignoredNonExisting wasn't updated in index

@imsodin imsodin changed the title TestPullInvalidIgnoredSR is flaky TestPullInvalidIgnoredSR/SO is flaky Jun 16, 2019

imsodin added a commit to imsodin/syncthing that referenced this issue Jun 16, 2019

@vext01

This comment has been minimized.

Copy link

commented Jun 16, 2019

Am I right in thinking that this is a bug in the test suite, and not in syncthing itself?

I'm looking to update the OpenBSD package to 1.1.4, but @imsodin's patch doesn't apply to the release -- seems a fair amount of stuff has changed there recently.

If it's a test suite bug, I'm not too worried about the failures.

@imsodin

This comment has been minimized.

Copy link
Member

commented Jun 16, 2019

Yes, correct, just test suit.

@calmh calmh closed this in cea5962 Jun 17, 2019

@calmh calmh added this to the v1.2.0 milestone Jun 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.