Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

Add global bencher config #652

Merged
merged 2 commits into from
Oct 8, 2021
Merged

Add global bencher config #652

merged 2 commits into from
Oct 8, 2021

Conversation

danburck
Copy link
Contributor

@danburck danburck commented Oct 8, 2021

Part of: ENG-157

Description


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@linear
Copy link

linear bot commented Oct 8, 2021

ENG-157 Add Bencher config

Add config file for Evmos and Ethermint repos

https://bencher.orijtech.com/configuration/

Copy link
Contributor

@fedekunze fedekunze left a comment

Choose a reason for hiding this comment

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

ACK, we can add package-specific config afterwards

@fedekunze fedekunze enabled auto-merge (squash) October 8, 2021 12:05
@fedekunze fedekunze merged commit 4116bfa into main Oct 8, 2021
@fedekunze fedekunze deleted the ENG-157-benchmark-config branch October 8, 2021 12:07
fedekunze added a commit that referenced this pull request Oct 8, 2021
Add bencher config with global +-10% threshold for improvements and regressions

Co-authored-by: Federico Kunze Küllmer <[email protected]>
fedekunze added a commit that referenced this pull request Oct 8, 2021
* ci: add bencher config (#652)

Add bencher config with global +-10% threshold for improvements and regressions

Co-authored-by: Federico Kunze Küllmer <[email protected]>

* fix conflicts

* evm: fix panic when transaction is reverted (#650)

* fix panic when transaction is reverted

* update changelog

* Update x/evm/keeper/context_stack.go

Co-authored-by: Federico Kunze Küllmer <[email protected]>

* rpc: test fix (#608)

* fix rpc tests with net namespace

* skip personal test

* skip rpc pending test

* fix endpoint

* fix rpc pending test

* fix missing gas param in some rpc tests

* fix eth_getproof when the block number is equal to pending or latest

* fix rpc tests filter subscribe failed

* lint

* remove unused linter

* fix PendingTransactionFilter and TestEth_GetFilterChanges_BlockFilter

* fix eth_estimateGas

* fix TestEth_EstimateGas_ContractDeployment

* skip TestEth_ExportAccount_WithStorage

* remove sleep in rpc test

* Update changelog

* add test-rpc in github action

* bump golangci-lint version to v1.42.1

* release: v0.7.1 cherry-picks

* changelog

* ci: fix solidity tests (#278)

* Fix CI

* Remove verbose-log to reduce size

* update timeout

* rm deploy contract action

* Update test-helper.js

* Update workflow

* Update workflow

* fix gas estimate amount

* Update test.yml

* fix error assert issue

* ignore bad test case

* remove estimate gas test

* Change fromBlock to 1 (TEMP, Reverse Required)

* bump timeout

Co-authored-by: Federico Kunze Küllmer <[email protected]>
Co-authored-by: yihuang <[email protected]>
Co-authored-by: Federico Kunze <[email protected]>

Co-authored-by: Daniel Burckhardt <[email protected]>
Co-authored-by: Thomas Nguy <[email protected]>
Co-authored-by: JayT106 <[email protected]>
Co-authored-by: Yijia Su <[email protected]>
Co-authored-by: yihuang <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants