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

Develop #2

Merged
merged 20 commits into from
Jul 29, 2024
Merged

Develop #2

merged 20 commits into from
Jul 29, 2024

Conversation

alienc0der
Copy link
Owner

No description provided.

yihuang and others added 20 commits July 5, 2024 03:01
* Problem: no simpler benchmark setup

Solution:
- most of the features of testground are not really used by us yet, so
  for a simple benchmark to get a TPS result, it's possible to do an
  alternative setup.
  the alternative implementation setup all the node data files in
  advance, then run the nodes in containers in a stateless manner, they
  don't need any coordinations at runtime, so we don't need those
  infrastructures to support the coordinations, just deploy the testplan
  image to the cluster like a stateless service.

* match test_group_id

* lint

* Update testground/benchmark/benchmark/peer.py

Co-authored-by: mmsqe <[email protected]>
Signed-off-by: yihuang <[email protected]>

* change docker-composer

* cleanup

* speficy name && disable strict

* run load generator and jsonnet config

quit

wait for grpc service

* fix lint

* readme and flake

* fix total

* print tps

* Update testground/README.md

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Signed-off-by: yihuang <[email protected]>

* Revert "print tps"

This reverts commit 8ecd342.

* kill directly

---------

Signed-off-by: yihuang <[email protected]>
Co-authored-by: mmsqe <[email protected]>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
…org-chain#1508)

Bumps [certifi](https://github.com/certifi/python-certifi) from 2024.2.2 to 2024.7.4.
- [Commits](certifi/python-certifi@2024.02.02...2024.07.04)

---
updated-dependencies:
- dependency-name: certifi
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump github.com/rs/cors from 1.10.1 to 1.11.0

Bumps [github.com/rs/cors](https://github.com/rs/cors) from 1.10.1 to 1.11.0.
- [Commits](rs/cors@v1.10.1...v1.11.0)

---
updated-dependencies:
- dependency-name: github.com/rs/cors
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

* gomod2nix

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: huangyi <[email protected]>
…to-org-chain#1509)

Bumps [certifi](https://github.com/certifi/python-certifi) from 2024.6.2 to 2024.7.4.
- [Commits](certifi/python-certifi@2024.06.02...2024.07.04)

---
updated-dependencies:
- dependency-name: certifi
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: mmsqe <[email protected]>
* Problem: rocksdb 9.2.1 is not used

* Update CHANGELOG.md

Co-authored-by: mmsqe <[email protected]>
Signed-off-by: yihuang <[email protected]>

---------

Signed-off-by: yihuang <[email protected]>
Co-authored-by: mmsqe <[email protected]>
…rg-chain#1516)

Bumps [setuptools](https://github.com/pypa/setuptools) from 69.1.0 to 70.0.0.
- [Release notes](https://github.com/pypa/setuptools/releases)
- [Changelog](https://github.com/pypa/setuptools/blob/main/NEWS.rst)
- [Commits](pypa/setuptools@v69.1.0...v70.0.0)

---
updated-dependencies:
- dependency-name: setuptools
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…rypto-org-chain#1515)

* Problem: sig verification result is not cached between incarnations

* Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

* update ethermint

* update deps

* fix gas

* update deps

---------

Signed-off-by: yihuang <[email protected]>
Co-authored-by: mmsqe <[email protected]>
…chain#1518)

* Problem: memiavl/versiondb don't build with upstream sdk

Solution:
- keep it api compatible with both

* deps

* remove support of streaming service, versiondb will override

* changelog

* fix test

* align deps

* fix lint

* fix lint

---------

Co-authored-by: mmsqe <[email protected]>
…1521)

* Problem: testground image don't have tmp directory

Solution:
- create the directory inside the image, it's required by cronosd

* fix image
…chain#1522)

* Problem: testground test case requires persistent volume

Solution:
- persistent volume is a headache in k8s cluster, but we can embed the data directory into image to avoid that.

* cleanup

* lint

* nix develop -c black .

---------

Co-authored-by: mmsqe <[email protected]>
…-org-chain#1524)

* Problem: testground stateless benchmark don't collect outputs

Solution:
- tar the home directory into outputs volume

* fix readme

* fix readme

* Update testground/benchmark/benchmark/stateless.py

Signed-off-by: yihuang <[email protected]>

* fix

* log balance change after test

---------

Signed-off-by: yihuang <[email protected]>
* Problem: memory leak in archive node

- Suspect large `application.db`
- try cache index/filters

* Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

---------

Signed-off-by: yihuang <[email protected]>
…chain#1529)

* Problem: pinL0FilterAndIndexBlocksInCache is not enabled

* more space
…crypto-org-chain#1533)

* Problem: testground stateless test validator don't quit automatically

Solution:
- validator quit when detected the chain is in idle

* improve log

* more stable benchmark
@alienc0der alienc0der merged commit 0d34e93 into develop Jul 29, 2024
1 of 2 checks passed
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.

5 participants