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

Snapshot config - change to how this gets included #277

Closed
Nazgolze opened this issue May 28, 2018 · 6 comments
Closed

Snapshot config - change to how this gets included #277

Nazgolze opened this issue May 28, 2018 · 6 comments
Assignees

Comments

@Nazgolze
Copy link
Contributor

Will generate this based off of the default config instead of maintaining 2 config copies

Needs to be done for both 0.9.x and 1.0.x

@nazarhussain
Copy link

@Nazgolze Do you mean to use same config.json from core instead of having seperate snapshot.json file? Yes I believe it will be work, but we need to compare both files to see any significant config changes.

@fchavant
Copy link
Contributor

I believe the following need to be changed:

  • port
  • logFileName
  • db.database
  • loading.loadPerIteration

Additionally those options should be changed to prevent unexpected issues:

  • version
  • minVersion
  • peers.list (empty)

@diego-G
Copy link

diego-G commented May 28, 2018

Please, prioritise this issue. Thanks

@Nazgolze
Copy link
Contributor Author

Closed by #279

@fchavant fchavant reopened this May 29, 2018
@fchavant
Copy link
Contributor

We need to ensure .fileLogLevel is at least info (not the case with betanet for instance)

Nazgolze added a commit that referenced this issue May 30, 2018
Ensure ".fileLogLevel" is set to "info" in snapshot.json - Closes #277
@Nazgolze
Copy link
Contributor Author

Closed by #281

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants