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

testing: add miners to harnesses #2150

Merged
merged 2 commits into from
Feb 23, 2023
Merged

Conversation

buck54321
Copy link
Member

Just a thought. We don't have to auto-mine, and I haven't considered how this might affect others tests, but it seems simple enough and I think we'll want something reasonable for bond maintenance.

@chappjc
Copy link
Member

chappjc commented Feb 20, 2023

Seems reasonable to me. The rotateBonds method is called every 20 seconds, so I found on simnet that I had to set me watch interval to be less than that (settled on 15) otherwise you occasionally rotate too late and get a tier drop.

@ukane-philemon
Copy link
Contributor

Nice. Makes sense and is a useful addition.

@buck54321
Copy link
Member Author

E2E tests seem to be working. Loadbot runs a miner that targets 1 block per roughly two epochs. If it's important to maintain that target (I don't think it is), we can always shut the miners down.

@buck54321 buck54321 marked this pull request as ready for review February 22, 2023 16:15
@chappjc chappjc added this to the 0.6 milestone Feb 22, 2023
Copy link
Member

@JoeGruffins JoeGruffins left a comment

Choose a reason for hiding this comment

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

Probably fine. Can always turn them off. Some tests mine though... I feel like dcr gets funky if you try to mine from two places. But again can just turn off and let the test mine.

The dcr harness goes to the mining session rather than the ctrl-harness after these changes.

@chappjc chappjc merged commit cb055d3 into decred:master Feb 23, 2023
@JoeGruffins
Copy link
Member

The dcr harness still defaults to a different session.

@chappjc chappjc added the bonds fidelity bonds label Mar 31, 2023
@dev-warrior777
Copy link
Contributor

The dcr harness still defaults to a different session.

fixed in

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bonds fidelity bonds
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants