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

stress test to use fault injection with d2c #1044

Merged
merged 2 commits into from
Oct 3, 2021

Conversation

BertKleewein
Copy link
Member

No description provided.

}

// Top-level task for limiting the length of the test. This task just sleeps
// until the time elapses and then raises a sting to actually end the task.
Copy link
Contributor

@anthonyvercolano anthonyvercolano Oct 1, 2021

Choose a reason for hiding this comment

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

sting? fancy dancy CS term. #Resolved

};
}

async run(): Promise<void> {
Copy link
Contributor

@anthonyvercolano anthonyvercolano Oct 1, 2021

Choose a reason for hiding this comment

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

Two with the same name? #Resolved

Copy link
Member Author

Choose a reason for hiding this comment

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

different class.

): Promise<void> {
while (!this.done) {
await func();
const interval = Math.floor(Math.random() * (maxInterval - minInterval + 1)) + minInterval;
Copy link
Contributor

@anthonyvercolano anthonyvercolano Oct 1, 2021

Choose a reason for hiding this comment

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

Is it possible to seed the random generator with a fixed constant so that we get the same random intervals on every run? #Resolved

Copy link
Member Author

Choose a reason for hiding this comment

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

The node Math.random generator is not seedable. Other RNGs exist that are seedable, but I don't want to add that right now. If we want repeatability later, we can add that.

@BertKleewein BertKleewein merged commit bfb6611 into Azure:master Oct 3, 2021
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.

2 participants