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

Expand TestPledgeBatching to test to test message propagation. #6352

Closed
Tracked by #6185
Stebalien opened this issue May 28, 2021 · 3 comments · Fixed by #6537
Closed
Tracked by #6185

Expand TestPledgeBatching to test to test message propagation. #6352

Stebalien opened this issue May 28, 2021 · 3 comments · Fixed by #6537
Assignees
Labels
P1 P1: Must be resolved

Comments

@Stebalien
Copy link
Member

We need to end-to-end test batch sector proving in a devnet to make sure we don't have any hidden issues anywhere in the pipeline (e.g., message propagation because we'll have larger messages).

This will help us catch bugs like #6338.

@Stebalien Stebalien added this to the Network Hyperdrive milestone May 28, 2021
@BigLep BigLep added the P1 P1: Must be resolved label May 28, 2021
@BigLep
Copy link
Member

BigLep commented May 28, 2021

2021-05-28 discussion: @magik6k says we can do this as an integration test. We can look at TestPledgeBatching for inspiration.

Need to test PubSub as well.

@Stebalien Stebalien changed the title Test batch sector proving in a devnet Expand TestPledgeBatching to test to test message propegation. May 28, 2021
@Stebalien Stebalien changed the title Expand TestPledgeBatching to test to test message propegation. Expand TestPledgeBatching to test to test message propagation. May 28, 2021
@BigLep BigLep mentioned this issue Jun 4, 2021
80 tasks
@BigLep BigLep assigned Stebalien and ZenGround0 and unassigned Stebalien Jun 17, 2021
@BigLep
Copy link
Member

BigLep commented Jun 17, 2021

2021-06-17 standup: base work off #6329

@BigLep BigLep linked a pull request Jun 21, 2021 that will close this issue
@arajasek
Copy link
Contributor

#6537 does this and is passing, just needs to land into the broader integ test work. This can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 P1: Must be resolved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants