-
Notifications
You must be signed in to change notification settings - Fork 295
Problem: recent bug fixes are not included #340
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
Changes from 9 commits
60090c5
3e64b96
ef612ee
0f4f800
ced583b
c0a9d02
086d5e1
7058de4
1798a93
d7e270a
bb6d9e6
08d637c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -119,6 +119,10 @@ import ( | |
|
|
||
| // unnamed import of statik for swagger UI support | ||
| _ "github.com/crypto-org-chain/cronos/client/docs/statik" | ||
|
|
||
| // Force-load the tracer engines to trigger registration | ||
| _ "github.com/ethereum/go-ethereum/eth/tracers/js" | ||
| _ "github.com/ethereum/go-ethereum/eth/tracers/native" | ||
| ) | ||
|
|
||
| const ( | ||
|
|
@@ -692,13 +696,18 @@ func New( | |
|
|
||
| app.SetEndBlocker(app.EndBlocker) | ||
|
|
||
| // upgrade handler | ||
| // upgrade handlers | ||
| plan0_7_0 := "v0.7.0" | ||
| app.UpgradeKeeper.SetUpgradeHandler(plan0_7_0, func(ctx sdk.Context, plan upgradetypes.Plan, fromVM module.VersionMap) (module.VersionMap, error) { | ||
| // The default genesis parameters of `feemarket` module are fine, because the `InitialBaseFee (1000000000)` is much lower than the current minimal gas price, | ||
| // so it don't have effect at the beginning, we can always adjust the parameters through the governance later. | ||
| return app.mm.RunMigrations(ctx, app.configurator, fromVM) | ||
| }) | ||
| // this upgrade is for breaking bug fixes on testnet | ||
| plan0_7_0HotfixTestnet := "v0.7.0-hotfix-testnet" | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. since we are planning to do a new binary release, should be increase the minor version?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. currently, it's still
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. maybe change the name of the plan to "v0.7.0-rc2" ? Because if we do another upgrade for testnet hotfix after this one, the naming will become really confusing
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. done |
||
| app.UpgradeKeeper.SetUpgradeHandler(plan0_7_0HotfixTestnet, func(ctx sdk.Context, plan upgradetypes.Plan, fromVM module.VersionMap) (module.VersionMap, error) { | ||
| return app.mm.RunMigrations(ctx, app.configurator, fromVM) | ||
| }) | ||
|
|
||
| upgradeInfo, err := app.UpgradeKeeper.ReadUpgradeInfoFromDisk() | ||
| if err != nil { | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,7 +6,7 @@ require ( | |
| github.com/armon/go-metrics v0.3.10 | ||
| github.com/cosmos/cosmos-sdk v0.45.1 | ||
| github.com/cosmos/ibc-go/v2 v2.0.2 | ||
| github.com/ethereum/go-ethereum v1.10.11 | ||
| github.com/ethereum/go-ethereum v1.10.15 | ||
| github.com/gogo/protobuf v1.3.3 | ||
| github.com/golang/protobuf v1.5.2 | ||
| github.com/gorilla/mux v1.8.0 | ||
|
|
@@ -20,8 +20,8 @@ require ( | |
| github.com/tendermint/tendermint v0.34.14 | ||
| github.com/tendermint/tm-db v0.6.4 | ||
| github.com/tharsis/ethermint v0.10.0-alpha1 | ||
| google.golang.org/genproto v0.0.0-20211223182754-3ac035c7e7cb | ||
| google.golang.org/grpc v1.43.0 | ||
| google.golang.org/genproto v0.0.0-20220211171837-173942840c17 | ||
| google.golang.org/grpc v1.44.0 | ||
| gopkg.in/yaml.v2 v2.4.0 | ||
| ) | ||
|
|
||
|
|
@@ -136,7 +136,7 @@ require ( | |
| github.com/tyler-smith/go-bip39 v1.1.0 // indirect | ||
| github.com/zondax/hid v0.9.0 // indirect | ||
| go.etcd.io/bbolt v1.3.6 // indirect | ||
| golang.org/x/crypto v0.0.0-20211215153901-e495a2d5b3d3 // indirect | ||
| golang.org/x/crypto v0.0.0-20220213190939-1e6e3497d506 // indirect | ||
| golang.org/x/net v0.0.0-20211112202133-69e39bad7dc2 // indirect | ||
| golang.org/x/sync v0.0.0-20210220032951-036812b2e83c // indirect | ||
| golang.org/x/sys v0.0.0-20211210111614-af8b64212486 // indirect | ||
|
|
@@ -153,7 +153,8 @@ require ( | |
| replace ( | ||
| // TODO: fix keyring upstream | ||
| github.com/99designs/keyring => github.com/crypto-org-chain/keyring v1.1.6-fixes | ||
| github.com/cosmos/cosmos-sdk => github.com/cosmos/cosmos-sdk v0.45.1 | ||
| // release/v0.45.x branch, TODO use tag when released | ||
| github.com/cosmos/cosmos-sdk => github.com/cosmos/cosmos-sdk v0.45.2-0.20220209090836-4161d2b6022a | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
|
|
||
| // TODO: remove when middleware will be implemented | ||
| github.com/cosmos/ibc-go/v2 => github.com/crypto-org-chain/ibc-go/v2 v2.0.2-hooks | ||
|
|
@@ -164,7 +165,7 @@ replace ( | |
| // See https://github.com/tecbot/gorocksdb/pull/216 | ||
| github.com/tecbot/gorocksdb => github.com/cosmos/gorocksdb v1.1.1 | ||
|
|
||
| github.com/tharsis/ethermint => github.com/crypto-org-chain/ethermint v0.10.0-alpha1-cronos-4 | ||
| github.com/tharsis/ethermint => github.com/crypto-org-chain/ethermint v0.10.0-alpha1-cronos-6 | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
|
|
||
| google.golang.org/grpc => google.golang.org/grpc v1.33.2 | ||
| ) | ||
Large diffs are not rendered by default.
Uh oh!
There was an error while loading. Please reload this page.