Skip to content
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

add notes regarding exporting/importing market data with lotus-shed #972

Merged
merged 8 commits into from
Jul 27, 2021

Conversation

nonsense
Copy link
Member

@nonsense nonsense commented Jul 26, 2021

This PR is adding steps on how to export/import a backup with all markets-related metadata, so that users who upgraded to a multi-service architecture and accepted storage deals, can continue to retrieve those deals correctly in case they rollback to a monolith architecture.

@nonsense nonsense force-pushed the nonsense/add-docs-for-export-import-markets branch from 806a4c7 to 4cf960e Compare July 26, 2021 10:39
@filecorgi
Copy link
Contributor

  • Image optimization came back clean!
  • Vuepress build was successful!

docs/mine/lotus/split-markets-miners.md:

  • A comma may be missing after the conjunctive/linking adverb ‘Currently’.
    ... subsystems in the lotus-miner process. Currently there are 4 subsystems, th...
                                               ^^^^^^^^^
    
  • An apostrophe may be missing.
    ...istinct types of lotus-miner nodes: The markets node - a lotus-miner process...
                                               ^^^^^^^
    
  • The preposition ‘for’ seems more likely in this position.
    ...ode - a lotus-miner process responsible to handling the storage market subsy...
                                               ^^
    
  • An apostrophe may be missing.
    ...ode is started with configuration for a markets node, it includes a libp2p n...
                                               ^^^^^^^
    
  • The word “backup” is a noun. The verb is spelled with a space.
    ... the monolith miner process, you should backup your miner's metadata reposit...
                                               ^^^^^^
    
  • Consider a shorter alternative to avoid wordiness.
    ...the LOTUS_BACKUP_BASE_PATH env variable in order to do that. # Splitting the...
                                               ^^^^^^^^^^^
    
  • An apostrophe may be missing.
    ...ou need to create a config.toml for the markets node, and have it ready for ...
                                               ^^^^^^^
    
  • Possibly missing comma found.
    ...ve it ready for the next step. For more information see configuration usage ...
                                               ^^^^^^^^^^^
    
  • An apostrophe may be missing.
    ...proving node on the same machine as the markets node, make sure that their l...
                                               ^^^^^^^
    
  • Did you mean: “By default,”?
    ...sure that their listeners do not clash. By default the lotus-miner API serve...
                                               ^^^^^^^^^^
    
  • An apostrophe may be missing.
    ... you adjust the [Libp2p] section on the markets node accordingly - it needs ...
                                               ^^^^^^^
    
  • An apostrophe may be missing.
    .... This performs a one-time setup of the markets node. Part of that setup inc...
                                               ^^^^^^^
    
  • This abbreviation for “identification” is spelled all-uppercase.
    ...f that setup includes updating the peer id in the miner actor by submitting ...
                                               ^^
    
  • An apostrophe may be missing.
    ...able! This command should be run on the markets miner instance host, as it i...
                                               ^^^^^^^
    
  • An apostrophe may be missing.
    ...er instance host, as it is creating the markets miner instance repository, a...
                                               ^^^^^^^
    
  • An apostrophe may be missing.
    ...aling/proving miner process without the markets subsystem Update your config...
                                               ^^^^^^^
    
  • Possibly missing comma found.
    ...oving miner process without the markets subsystem Update your config.toml an...
                                               ^^^^^^^^^
    
  • An apostrophe may be missing.
    ...TH environment variable! # 6. Start the markets miner process with the marke...
                                               ^^^^^^^
    
  • An apostrophe may be missing.
    ...tart the markets miner process with the markets subsystem # Interacting with...
                                               ^^^^^^^
    
  • If the ‘because’ clause is essential to the meaning, do not use a comma before the clause.
    ... interacting with only one miner process, because it has precedence over LOT...
                                               ^^^^^^^^^
    
  • An apostrophe may be missing.
    ...US_MINER_PATH. Now that you have both a markets miner process and a mining/s...
                                               ^^^^^^^
    
  • An apostrophe may be missing.
    ...following: Get the peer identity of the markets miner process Get a list of ...
                                               ^^^^^^^
    
  • Only proper nouns start with an uppercase character (there are exceptions for headlines).
    ...r identity of the markets miner process Get a list of storage deals from the...
                                               ^^^
    
  • An apostrophe may be missing.
    ...ss Get a list of storage deals from the markets miner process Get a list of ...
                                               ^^^^^^^
    
  • Only proper nouns start with an uppercase character (there are exceptions for headlines).
    ...ge deals from the markets miner process Get a list of sectors from the minin...
                                               ^^^
    
  • An apostrophe may be missing.
    ...cly exposed, as we will be enabling the markets subsystem on it. In the mini...
                                               ^^^^^^^
    
  • Only proper nouns start with an uppercase character (there are exceptions for headlines).
    ...ml and set EnableMarkets option to true Move back the DAG store directory to...
                                               ^^^^
    
  • An apostrophe may be missing.
    ...s it was changed to the identity of the markets node during the initialising...
                                               ^^^^^^^
    
  • An apostrophe may be missing.
    ...g/proving node, which now also runs the markets subsystem, i.e. currently al...
                                               ^^^^^^^
    

@jennijuju jennijuju merged commit 5b306fb into master Jul 27, 2021
@jennijuju jennijuju deleted the nonsense/add-docs-for-export-import-markets branch July 27, 2021 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants