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

parity: print correct keys path on startup #9501

Merged
merged 2 commits into from
Sep 8, 2018

Conversation

5chdn
Copy link
Contributor

@5chdn 5chdn commented Sep 8, 2018

fix #9500

It prints the wrong key path using the spec name but uses the correct key path using the spec data dir. this fixes the startup notification.

hello from ethberlin <3

@5chdn 5chdn added A0-pleasereview 🤓 Pull request needs code review. A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). M4-core ⛓ Core client code / Rust. M2-config 📂 Chain specifications and node configurations. labels Sep 8, 2018
@5chdn 5chdn added this to the 2.1 milestone Sep 8, 2018
@5chdn
Copy link
Contributor Author

5chdn commented Sep 8, 2018

❯ ./target/debug/parity --chain ../testnet/parity/goerli.json 
Migrating database from version 12 to 13
Migrating blooms to blooms-db...
Migration finished
2018-09-08 14:07:06  Starting Parity-Ethereum/v2.1.0-unstable-61bd47ccc-20180908/x86_64-linux-gnu/rustc1.27.0
2018-09-08 14:07:06  Keys path /home/user/.local/share/io.parity.ethereum/keys/goerli
2018-09-08 14:07:06  DB path /home/user/.local/share/io.parity.ethereum/chains/goerli/db/6a900bab113371a2
2018-09-08 14:07:06  State DB configuration: fast
2018-09-08 14:07:06  Operating mode: active
2018-09-08 14:07:06  Configured for Görli using AuthorityRound engine
2018-09-08 14:07:12  Public node URL: enode://ac1977d8753fc12dd801e87139c0fd56fed3bac136c8a9cafe4f82e147612f3232266e7f4c74d869722fd26ecefff42496ea7b2225d624d432c123f5f4ed44b8@10.0.1.220:30303
2018-09-08 14:07:16  Imported #810 0xbc5e…e865 (0 txs, 0.00 Mgas, 16 ms, 0.57 KiB) + another 3 block(s) containing 0 tx(s)

@tomusdrw tomusdrw added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Sep 8, 2018
@ordian
Copy link
Collaborator

ordian commented Sep 8, 2018

@5chdn rebase on top of master to fix CI check

@5chdn 5chdn merged commit 2177a01 into openethereum:master Sep 8, 2018
@5chdn 5chdn deleted the a5-config-dbpath branch September 8, 2018 22:43
This was referenced Sep 8, 2018
5chdn added a commit that referenced this pull request Sep 8, 2018
5chdn added a commit that referenced this pull request Sep 8, 2018
ChainSafeSystems pushed a commit to goerli/parity-goerli that referenced this pull request Sep 9, 2018
* rpc(debug_getBadBlocks): fix test (openethereum#9502)

* Generalized blockReward and difficultyBombDelays config (openethereum#9480)

* Implement multi blockReward

* Implement difficultyBombDelays

* Fix json crate compile

* json keys can only be string

* parity: print correct keys path on startup (openethereum#9501)
5chdn added a commit that referenced this pull request Sep 10, 2018
* parity-version: bump stable to 1.11.11

* Update tobalaba.json (#9419)

* Update hardcoded sync (#9421)

- Update foundation hardcoded header to block 6219777 
- Update ropsten hardcoded header to block 3917825 
- Update kovan hardcoded header to block 8511489

* parity: print correct keys path on startup (#9501)

* Only check warp syncing for eth_getWorks (#9484)

* Only check warp syncing for eth_getWorks

* Use SyncStatus::is_snapshot_syncing
5chdn added a commit that referenced this pull request Sep 10, 2018
* parity-version: bump beta to 2.0.4

* [light/jsonrpc] Provide the actual account for `eth_coinbase` RPC and unify error handeling for light and full client (#9383)

* Provide the actual `account` for eth_coinbase

The previous implementation always provided the `zero address` on
`eth_coinbase` RPC. Now, instead the actual address is returned on
success or an error when no account(s) is found!

* full client `eth_coinbase` return err

In the full-client return an error when no account is found instead of
returning the `zero address`

* Remove needless blocks on single import

* Remove needless `static` lifetime on const

* Fix `rpc_eth_author` test

* parity: print correct keys path on startup (#9501)

* aura: don't report skipped primaries when empty steps are enabled (#9435)

* Only check warp syncing for eth_getWorks (#9484)

* Only check warp syncing for eth_getWorks

* Use SyncStatus::is_snapshot_syncing

* Fix Snapshot restoration failure on Windows (#9491)

* Close Blooms DB files before DB restoration

* PR Grumbles I

* PR Grumble

* Grumble
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). A8-looksgood 🦄 Pull request is reviewed well. M2-config 📂 Chain specifications and node configurations. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

keys path should use datadir from chain spec
3 participants