Implementing proper bootstrap procedure #7

Open

lee.miller wants to merge 6 commits from lee.miller/MiNode:bootstrap into v0.3

pull from: lee.miller/MiNode:bootstrap

merge into: Bitmessage:v0.3

Bitmessage:v0.3

Bitmessage:tor

Bitmessage:testing

Bitmessage:proofofwork

Bitmessage:network

Bitmessage:queue

Bitmessage:protocol

Bitmessage:master

Bitmessage:ssl

Bitmessage:lint

Bitmessage:doc

Bitmessage:network-test

Bitmessage:bootstrap

Bitmessage:validation

Bitmessage:test-i2p

Bitmessage:ci-buildbot

Bitmessage:test

Bitmessage:log

Bitmessage:setup

Bitmessage:experimental

Conversation 1 Commits 6 Files Changed 4 +119 -22

lee.miller commented 2024-06-25 22:06:25 +00:00

Contributor

Copy Link

Hi!

I added a class Bootstrapper, inherited from ConnectionBase, like it's done in PyBitmessage. Manager thread starts its instances if has a few connections.

Hi! I added a class `Bootstrapper`, inherited from `ConnectionBase`, like it's done in PyBitmessage. Manager thread starts its instances if has a few connections.

lee.miller added 5 commits 2024-06-25 22:06:26 +00:00

Define a base class for connection to subclass for special purposes 70f887956c

A rough implementation of proper bootstrapping cf7c03c9f5

Try not to add core nodes to pool 0330346e16

Reduce number of simultaneous bootstrappers, refill the bootstrap pool bb65e7f0b9

Started a test case for network with a test for bootstrapping 0513e6f84e

lee.miller added 1 commit 2024-07-14 01:51:56 +00:00

Use socket.inet_pton in main.bootstrap_from_dns() to support IPv6 addrs 1776cf958d

lee.miller added 1 commit 2024-07-14 01:58:01 +00:00

Warn on socket.gaierror in main.bootstrap_from_dns() c411d2f28d

lee.miller added 1 commit 2024-07-14 02:06:50 +00:00

More debug logging in main.bootstrap_from_dns() 6e68d9b81a

lee.miller added 1 commit 2024-07-14 02:22:12 +00:00

More debug logging in main.bootstrap_from_dns() dad99a51e0

lee.miller added 1 commit 2024-07-14 02:28:59 +00:00

Even more debug logging 3e32f0e025

lee.miller commented 2024-07-14 02:53:04 +00:00

Author

Contributor

Copy Link

The action is failing because your act-runner replaces the result of socket.getaddrinfo by the IPs of node3.sysdeploy.org.

The action is failing because your act-runner replaces the result of `socket.getaddrinfo` by the IPs of node3.sysdeploy.org.

PeterSurda approved these changes 2024-07-14 03:51:09 +00:00

Dismissed

PeterSurda left a comment

Owner

Copy Link

I haven't analysed all the internal details, but looks ok.

I haven't analysed all the internal details, but looks ok.

lee.miller requested review from PeterSurda 2024-07-14 03:57:23 +00:00

lee.miller removed review request for PeterSurda 2024-07-14 03:57:37 +00:00

lee.miller requested review from PeterSurda 2024-07-14 03:57:44 +00:00

lee.miller force-pushed bootstrap from 3e32f0e025 to 1776cf958d 2024-07-15 00:37:36 +00:00 Compare

lee.miller force-pushed bootstrap from 1776cf958d to 95219b0f25 2024-07-15 19:13:05 +00:00 Compare

lee.miller force-pushed bootstrap from 95219b0f25 to b005b77d15 2024-07-18 17:15:38 +00:00 Compare

lee.miller added 1 commit 2024-07-18 21:02:41 +00:00

Test bootstrapping with a minode process b385e6f9c4

lee.miller changed title from WIP: Implementing proper bootstrap procedure to Implementing proper bootstrap procedure 2024-07-18 21:41:43 +00:00

lee.miller force-pushed bootstrap from b385e6f9c4 to ab54b87bdf 2024-07-18 21:57:49 +00:00 Compare

lee.miller changed title from Implementing proper bootstrap procedure to WIP: Implementing proper bootstrap procedure 2024-07-19 01:21:10 +00:00

lee.miller force-pushed bootstrap from ab54b87bdf to b335496011 2024-07-23 16:01:18 +00:00 Compare

lee.miller referenced this pull request 2024-07-29 23:32:06 +00:00

Make I2P tests not fail for now #10

PeterSurda approved these changes 2024-08-14 08:09:41 +00:00

PeterSurda left a comment

Owner

Copy Link

Still looks ok.

Still looks ok.

lee.miller force-pushed bootstrap from b335496011 to 08916c0c9b 2024-08-14 13:58:47 +00:00 Compare

lee.miller changed title from WIP: Implementing proper bootstrap procedure to Implementing proper bootstrap procedure 2024-08-14 14:17:13 +00:00

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u bootstrap:lee.miller-bootstrap

git checkout lee.miller-bootstrap

Merge

Merge the changes and update on Gitea.

git checkout v0.3

git merge --no-ff lee.miller-bootstrap

git checkout v0.3

git merge --ff-only lee.miller-bootstrap

git checkout lee.miller-bootstrap

git rebase v0.3

git checkout v0.3

git merge --no-ff lee.miller-bootstrap

git checkout v0.3

git merge --squash lee.miller-bootstrap

git checkout v0.3

git merge --ff-only lee.miller-bootstrap

git checkout v0.3

git merge lee.miller-bootstrap

git push origin v0.3

Sign in to join this conversation.

Reviewers

No reviewers

PeterSurda

Labels

Clear labels

No items

No Label

Milestone

Clear milestone

No items

No Milestone

Assignees

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Bitmessage/MiNode#7

No description provided.