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

make hard-coded identity work when snapshot is being imported #405

Merged
merged 3 commits into from
Apr 11, 2022

Conversation

nicolasochem
Copy link
Collaborator

Snapshot import wipes the data dir. But we may write an identity.json
file in it. This PR ensures that we are not deleting it after writing
it!

We also write it at every boot (because what if the user wants to change
it?)

We add the node id in the debug log for config-generator.

Snapshot import wipes the data dir. But we may write an identity.json
file in it. This PR ensures that we are not deleting it after writing
it!

We also write it at every boot (because what if the user wants to change
it?)

We add the node id in the debug log for config-generator.
@nicolasochem
Copy link
Collaborator Author

I verified it was working on oxheadinfra mainnet-dev by temporarily pointing a submod to this branch.

utils/config-generator.py Outdated Show resolved Hide resolved
utils/snapshot-downloader.sh Outdated Show resolved Hide resolved
@nicolasochem nicolasochem merged commit 293abe0 into master Apr 11, 2022
@harryttd harryttd deleted the fix_identity branch May 24, 2022 00:39
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.

3 participants