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

Store the database in a role specific subdirectory #2923

Merged
7 commits merged into from
Sep 7, 2021

Conversation

hirschenberger
Copy link
Contributor

@hirschenberger hirschenberger commented Apr 22, 2021

companion PR for paritytech/substrate#9645

@ordian ordian added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Apr 22, 2021
@cla-bot-2021
Copy link

cla-bot-2021 bot commented Apr 26, 2021

User @hirschenberger, please sign the CLA here.

hirschenberger added a commit to hirschenberger/substrate that referenced this pull request Aug 28, 2021
ghost pushed a commit to paritytech/substrate that referenced this pull request Sep 7, 2021
* Store the database in a role specific subdirectory

This is a cleaned up version of #8658 fixing #6880

polkadot companion: paritytech/polkadot#2923

* Disable prometheus in tests

* Also change p2p port

* Fix migration logic

* Use different identification file for rocks and parity db

Add tests for paritydb migration
@ghost
Copy link

ghost commented Sep 7, 2021

Waiting for commit status.

@ghost ghost merged commit fb6c9b3 into paritytech:master Sep 7, 2021
This pull request was closed.
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). A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants