Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Error message comes up upon shutting down the client. #11214

Closed
adilimroz opened this issue Oct 28, 2019 · 4 comments
Closed

Error message comes up upon shutting down the client. #11214

adilimroz opened this issue Oct 28, 2019 · 4 comments
Labels
hacktoberfest Join us for hacktoberfest - https://substrate.dev/hacktoberfest/ M4-core ⛓ Core client code / Rust. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow.
Milestone
2.7

Comments

@adilimroz
Copy link

adilimroz commented Oct 28, 2019 •
edited
Loading

  • Parity Ethereum version: 2.5.9 and other recent releases
  • Operating system: Windows / MacOS / Linux
  • Installation: built from source

Steps to reproduce

  1. Try shutting down a running parity-ethereum client with Ctrl+C keyboard combination.
    Observe that the client shuts down but with a message
    "unable to get mut ref for engine for shutdown"

Screenshot from 2019-10-28 23-56-45
@dvdplm dvdplm added hacktoberfest Join us for hacktoberfest - https://substrate.dev/hacktoberfest/ M4-core ⛓ Core client code / Rust. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow. B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. B1-patch-beta 🕷🕷 and removed B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. B1-patch-beta 🕷🕷 labels Oct 28, 2019
@dvdplm dvdplm added this to the 2.7 milestone Oct 28, 2019
@dvdplm
Copy link
Collaborator

dvdplm commented Oct 28, 2019

Probably over-zealous logging.

I suspect this is fixed on master or not shown in all modes of operation. @adilimroz do you see this even when just starting parity with default params? Or just when e.g. import/exporting blocks?

@adilimroz
Copy link
Author

@dvdplm It occurs while running the client in all possible configuration.

Screenshot from 2019-10-29 00-05-27

@niklasad1
Copy link
Collaborator

@adilimroz this should be fixed by #10691 and #10683

You can try https://github.com/paritytech/parity-ethereum/tree/v2.5.10-stable to verify

@adilimroz
Copy link
Author

Thanks @niklasad1 . Verified in master as well.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
hacktoberfest Join us for hacktoberfest - https://substrate.dev/hacktoberfest/ M4-core ⛓ Core client code / Rust. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow.
Projects
None yet
Milestone
2.7
Development

No branches or pull requests
3 participants
@dvdplm @adilimroz @niklasad1