Skip to content
This repository was archived by the owner on Aug 2, 2021. It is now read-only.

cmd/swarm: disable flaky snapshot test#1376

Merged
nonsense merged 1 commit intoswarm-rather-stablefrom
snapshot-test-disable
May 7, 2019
Merged

cmd/swarm: disable flaky snapshot test#1376
nonsense merged 1 commit intoswarm-rather-stablefrom
snapshot-test-disable

Conversation

@acud
Copy link
Copy Markdown
Contributor

@acud acud commented May 7, 2019

No description provided.

@acud acud requested a review from nonsense May 7, 2019 18:35
@acud acud self-assigned this May 7, 2019
@nonsense
Copy link
Copy Markdown
Contributor

nonsense commented May 7, 2019

This is flaky due to the Waiting for healthy Kademlia functionality AFAIK.

@acud
Copy link
Copy Markdown
Contributor Author

acud commented May 7, 2019

Correct

@nonsense nonsense merged commit 441c913 into swarm-rather-stable May 7, 2019
@nonsense
Copy link
Copy Markdown
Contributor

nonsense commented May 7, 2019

Actually maybe a note inside the test, rather than todo: fix this would be helpful when we come across it.

nonsense added a commit that referenced this pull request May 10, 2019
swarm/storage/localstore: fix broken metric (#1373)

p2p/protocols: count different messages (#1374)

cmd/swarm: disable snapshot create test due to constant flakes (#1376)
nonsense added a commit that referenced this pull request May 10, 2019
swarm/storage/localstore: fix broken metric (#1373)

p2p/protocols: count different messages (#1374)

cmd/swarm: disable snapshot create test due to constant flakes (#1376)

swarm/network: remove redundant goroutine (#1377)
nonsense added a commit that referenced this pull request May 10, 2019
swarm/storage/localstore: fix broken metric (#1373)

p2p/protocols: count different messages (#1374)

cmd/swarm: disable snapshot create test due to constant flakes (#1376)

swarm/network: remove redundant goroutine (#1377)
@acud acud deleted the snapshot-test-disable branch June 3, 2019 14:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants