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

fix error when export chain state #446

Merged
merged 2 commits into from
Aug 17, 2021
Merged

fix error when export chain state #446

merged 2 commits into from
Aug 17, 2021

Conversation

yihuang
Copy link
Contributor

@yihuang yihuang commented Aug 17, 2021

Description

To reproduce:

$ ethermintd export --height -1 --for-zero-height --home /chain/.ethermintd --chain-id cronostestnet-338
...
Error: error exporting state: expected validator ethvaloper1znla...fff7d7 not found

Related cosmos-sdk change: https://github.com/cosmos/cosmos-sdk/pull/8363/files#diff-42390a0f12b0fc40eb466b1f1b8de3b34022f112e216ac26a177435ffffba529R160


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)

@codecov
Copy link

codecov bot commented Aug 17, 2021

Codecov Report

Merging #446 (3d1f063) into main (2edcbf4) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #446   +/-   ##
=======================================
  Coverage   50.58%   50.58%           
=======================================
  Files          50       50           
  Lines        4928     4928           
=======================================
  Hits         2493     2493           
  Misses       2328     2328           
  Partials      107      107           
Impacted Files Coverage Δ
app/export.go 13.88% <0.00%> (ø)

@fedekunze fedekunze enabled auto-merge (squash) August 17, 2021 11:57
@fedekunze fedekunze merged commit d93205b into evmos:main Aug 17, 2021
@yihuang yihuang deleted the fix-export branch August 17, 2021 12:19
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