Skip to content

refactor baseline pubsub test out of private repo#6

Merged
yusefnapora merged 7 commits intolibp2p:masterfrom
yusefnapora:feat/pubsub-baseline
May 15, 2020
Merged

refactor baseline pubsub test out of private repo#6
yusefnapora merged 7 commits intolibp2p:masterfrom
yusefnapora:feat/pubsub-baseline

Conversation

@yusefnapora
Copy link
Copy Markdown
Contributor

This pulls in the baseline evaluation plan from the private repo we've been using to evaluate gossipsub under attack.

We're planning to publish that code soon also, but for now this just has the "honest" node type included.

cc @daviddias @vyzo @dirkmc @nonsense

BTW, if anyone wants to give me write access to this repo, that would be sweet 😄

@jacobheun
Copy link
Copy Markdown

@yusefnapora access should be fixed, I added the Repos - Go team.

Copy link
Copy Markdown
Member

@daviddias daviddias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks where we want to be. I recommend not overloading users with non relevant info (i.e. attack code or mentions of sybils)

@yusefnapora
Copy link
Copy Markdown
Contributor Author

yusefnapora commented May 13, 2020

I've tested this out on our jupyterhub instance using the k8s runner. Everything seems to work as expected, but it was taking a fairly long time to get all pods in the Ready state - about 2.5 minutes for 1k nodes.

I increased the setup time in the saved configs to 3 minutes, since if it's lower than the actual time needed to get into the running state, the test results get skewed.

Copy link
Copy Markdown

@vyzo vyzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's merge!

@yusefnapora yusefnapora merged commit f0762b1 into libp2p:master May 15, 2020
@yusefnapora yusefnapora deleted the feat/pubsub-baseline branch May 15, 2020 18:10
codemaestro64 pushed a commit to codemaestro64/test-plans that referenced this pull request Oct 28, 2025
* refactor baseline test out of private repo

* rm references to attackers

* fix default plan name & add widget to override

* add configs for local runners

* update runner notebook intro text

* fix build tags

* increase setup time in saved configs
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.

4 participants