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

Add fine-grained control of time #88

Merged
merged 25 commits into from
Nov 27, 2023
Merged

Add fine-grained control of time #88

merged 25 commits into from
Nov 27, 2023

Conversation

p-offtermatt
Copy link
Member

@p-offtermatt p-offtermatt commented Nov 21, 2023

Closes: #87

  • Makes the testing infra more durable and faster
  • Pulls out time handling into a generic TimeHandler interface to support different ways of assigning timestamps
  • Adds the flags specified in the issue

@p-offtermatt p-offtermatt self-assigned this Nov 21, 2023
@p-offtermatt p-offtermatt changed the title Add fine-grained control of tiem Add fine-grained control of time Nov 21, 2023
@p-offtermatt p-offtermatt added A:backport/v0.34.x When a PR labelled with this label is merged, it will automatically be backported to v0.34.x A:backport/v0.37.x When a PR labelled with this label is merged, it will automatically be backported to v0.37.x A:backport/v0.38.x When a PR labelled with this label is merged, it will automatically be backported to v0.38.x labels Nov 21, 2023
@p-offtermatt p-offtermatt merged commit 7edb4c1 into main Nov 27, 2023
3 checks passed
mergify bot pushed a commit that referenced this pull request Nov 27, 2023
* Start implementing auto-include-tx flag

* Split testnet initialization and run

* Add script to restart testnet from beginning

* Start implementing auto-include-tx flag

* Delete backup directory before taking backup

* Uncomment killing processes with app binary

* Rewrite TxQueue

* Fix fire events method

* Implement tx filtering and rechecks

* Start implementing basic test for Tx effect

* Ensure the script is not blocking

* Rename Dockerfile-test to Dockerfile.test

* Add Contains method

* Remove initial setup from Dockerfile

* Remove sleep and add -p 1

* Use big.Int for community pool size

* Use testnet restart in tests instead of a new setup each time

* Add cometmock args to be taken by startup script

* Pull time handling into TimeHandler class

* Fix CheckTx and time

* Add test cases for AutoTx and block production

* Add time control to README

* Fix auto-tx flag

* Add test for starting timestamp

* Add test for starting time=system time

(cherry picked from commit 7edb4c1)

# Conflicts:
#	e2e-tests/local-testnet-singlechain.sh
#	e2e-tests/main_test.go
mergify bot pushed a commit that referenced this pull request Nov 27, 2023
* Start implementing auto-include-tx flag

* Split testnet initialization and run

* Add script to restart testnet from beginning

* Start implementing auto-include-tx flag

* Delete backup directory before taking backup

* Uncomment killing processes with app binary

* Rewrite TxQueue

* Fix fire events method

* Implement tx filtering and rechecks

* Start implementing basic test for Tx effect

* Ensure the script is not blocking

* Rename Dockerfile-test to Dockerfile.test

* Add Contains method

* Remove initial setup from Dockerfile

* Remove sleep and add -p 1

* Use big.Int for community pool size

* Use testnet restart in tests instead of a new setup each time

* Add cometmock args to be taken by startup script

* Pull time handling into TimeHandler class

* Fix CheckTx and time

* Add test cases for AutoTx and block production

* Add time control to README

* Fix auto-tx flag

* Add test for starting timestamp

* Add test for starting time=system time

(cherry picked from commit 7edb4c1)

# Conflicts:
#	Dockerfile-test
#	cometmock/abci_client/client.go
#	cometmock/main.go
#	cometmock/rpc_server/routes.go
#	e2e-tests/local-testnet-singlechain.sh
#	e2e-tests/main_test.go
mergify bot pushed a commit that referenced this pull request Nov 27, 2023
* Start implementing auto-include-tx flag

* Split testnet initialization and run

* Add script to restart testnet from beginning

* Start implementing auto-include-tx flag

* Delete backup directory before taking backup

* Uncomment killing processes with app binary

* Rewrite TxQueue

* Fix fire events method

* Implement tx filtering and rechecks

* Start implementing basic test for Tx effect

* Ensure the script is not blocking

* Rename Dockerfile-test to Dockerfile.test

* Add Contains method

* Remove initial setup from Dockerfile

* Remove sleep and add -p 1

* Use big.Int for community pool size

* Use testnet restart in tests instead of a new setup each time

* Add cometmock args to be taken by startup script

* Pull time handling into TimeHandler class

* Fix CheckTx and time

* Add test cases for AutoTx and block production

* Add time control to README

* Fix auto-tx flag

* Add test for starting timestamp

* Add test for starting time=system time

(cherry picked from commit 7edb4c1)

# Conflicts:
#	Dockerfile-test
#	cometmock/abci_client/client.go
#	cometmock/main.go
#	cometmock/rpc_server/routes.go
#	e2e-tests/local-testnet-singlechain.sh
#	e2e-tests/main_test.go
p-offtermatt added a commit that referenced this pull request Nov 27, 2023
* Update README.md

* Refactor: remove isConnected

* Add test for Abci query

* Enable tags only for semver and default branch (#82)

* Add fine-grained control of time (#88)

* Start implementing auto-include-tx flag

* Split testnet initialization and run

* Add script to restart testnet from beginning

* Start implementing auto-include-tx flag

* Delete backup directory before taking backup

* Uncomment killing processes with app binary

* Rewrite TxQueue

* Fix fire events method

* Implement tx filtering and rechecks

* Start implementing basic test for Tx effect

* Ensure the script is not blocking

* Rename Dockerfile-test to Dockerfile.test

* Add Contains method

* Remove initial setup from Dockerfile

* Remove sleep and add -p 1

* Use big.Int for community pool size

* Use testnet restart in tests instead of a new setup each time

* Add cometmock args to be taken by startup script

* Pull time handling into TimeHandler class

* Fix CheckTx and time

* Add test cases for AutoTx and block production

* Add time control to README

* Fix auto-tx flag

* Add test for starting timestamp

* Add test for starting time=system time
p-offtermatt added a commit that referenced this pull request Nov 27, 2023
* Add fine-grained control of time (#88)

* Start implementing auto-include-tx flag

* Split testnet initialization and run

* Add script to restart testnet from beginning

* Start implementing auto-include-tx flag

* Delete backup directory before taking backup

* Uncomment killing processes with app binary

* Rewrite TxQueue

* Fix fire events method

* Implement tx filtering and rechecks

* Start implementing basic test for Tx effect

* Ensure the script is not blocking

* Rename Dockerfile-test to Dockerfile.test

* Add Contains method

* Remove initial setup from Dockerfile

* Remove sleep and add -p 1

* Use big.Int for community pool size

* Use testnet restart in tests instead of a new setup each time

* Add cometmock args to be taken by startup script

* Pull time handling into TimeHandler class

* Fix CheckTx and time

* Add test cases for AutoTx and block production

* Add time control to README

* Fix auto-tx flag

* Add test for starting timestamp

* Add test for starting time=system time

(cherry picked from commit 7edb4c1)

# Conflicts:
#	Dockerfile-test
#	cometmock/abci_client/client.go
#	cometmock/main.go
#	cometmock/rpc_server/routes.go
#	e2e-tests/local-testnet-singlechain.sh
#	e2e-tests/main_test.go

* Fix merge

* Fix isConnected behaviour and start addressing test failures

* Adjust block queries to simd 47

* Modify Dockerfile to use old simd

---------

Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
Co-authored-by: Philip Offtermatt <p.offtermatt@gmail.com>
p-offtermatt added a commit that referenced this pull request Dec 1, 2023
* Add fine-grained control of time (#88)

* Start implementing auto-include-tx flag

* Split testnet initialization and run

* Add script to restart testnet from beginning

* Start implementing auto-include-tx flag

* Delete backup directory before taking backup

* Uncomment killing processes with app binary

* Rewrite TxQueue

* Fix fire events method

* Implement tx filtering and rechecks

* Start implementing basic test for Tx effect

* Ensure the script is not blocking

* Rename Dockerfile-test to Dockerfile.test

* Add Contains method

* Remove initial setup from Dockerfile

* Remove sleep and add -p 1

* Use big.Int for community pool size

* Use testnet restart in tests instead of a new setup each time

* Add cometmock args to be taken by startup script

* Pull time handling into TimeHandler class

* Fix CheckTx and time

* Add test cases for AutoTx and block production

* Add time control to README

* Fix auto-tx flag

* Add test for starting timestamp

* Add test for starting time=system time

(cherry picked from commit 7edb4c1)

* Fix merge

* Fix isConnected behaviour and start addressing test failures

* Adjust block queries to simd 47

* Modify Dockerfile to use old simd

---------

Co-authored-by: Philip Offtermatt <57488781+p-offtermatt@users.noreply.github.com>
Co-authored-by: Philip Offtermatt <p.offtermatt@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:backport/v0.34.x When a PR labelled with this label is merged, it will automatically be backported to v0.34.x A:backport/v0.37.x When a PR labelled with this label is merged, it will automatically be backported to v0.37.x A:backport/v0.38.x When a PR labelled with this label is merged, it will automatically be backported to v0.38.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement fine-grained control over time
1 participant