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

Use DNS for localnet. #4835

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from
Draft

Use DNS for localnet. #4835

wants to merge 1 commit into from

Conversation

Frozen
Copy link
Contributor

@Frozen Frozen commented Jan 20, 2025

Use DNS for localnet instead of NewLocalSyncingPeerProvider

@Frozen Frozen self-assigned this Jan 20, 2025
@sophoah
Copy link
Contributor

sophoah commented Jan 21, 2025

With dns sync there is a txt dns request done to get the bootnode p2p information. I don't see how it will work here without a dns configured in this case.

Looking at localnet ops we are https://github.com/harmony-one/harmony/blob/main/test/deploy.sh#L75 passing the bootnode info already in the harmony command line, that would explain why the Travis Ci test didn't fail.

Was there any specific issue you were working on that led to that PR?

@Frozen
Copy link
Contributor Author

Frozen commented Jan 21, 2025

without this PR you have to hardcode node addresses. Each command (debug, mult-bls) potentially have it's own set of nodes. I already mentioned that, here you can see the problem https://github.com/harmony-one/harmony/pull/4820/files#diff-65c92fad7c09270b02e7cb7b0d66f7a7a4145645cdea36a9061bb45c62e9670fR209

@Frozen Frozen force-pushed the feature/localnet-use-dns branch from d160682 to 8584b70 Compare January 22, 2025 17:36
@Frozen Frozen marked this pull request as draft January 22, 2025 18:01
@Frozen
Copy link
Contributor Author

Frozen commented Jan 22, 2025

Found issue with shard peer discovering, converted to draft

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.

2 participants