Skip to content

feat: add HAProxy and FlowProxy config to BuilderHub and forward HAProxy ports from buildernet VM#376

Merged
canercidam merged 2 commits intomainfrom
fryd/buildernet-vm-add-config-and-ports
Feb 18, 2026
Merged

feat: add HAProxy and FlowProxy config to BuilderHub and forward HAProxy ports from buildernet VM#376
canercidam merged 2 commits intomainfrom
fryd/buildernet-vm-add-config-and-ports

Conversation

@fkondej
Copy link
Contributor

@fkondej fkondej commented Feb 18, 2026

No description provided.

@fkondej fkondej self-assigned this Feb 18, 2026
@fkondej fkondej requested a review from canercidam as a code owner February 18, 2026 15:06
@claude
Copy link
Contributor

claude bot commented Feb 18, 2026

Claude finished @fkondej's task in 1m 21s —— View job


PR Review: #376

  • Gather context and read changed files
  • Analyze changes and identify issues
  • Post review feedback

Summary: This PR adds two things: (1) FlowProxy and HAProxy configuration sections to the BuilderHub config YAML, and (2) HAProxy port forwarding (HTTP 10080→80, HTTPS 10443→443) for the buildernet QEMU VM.

The changes are clean and follow existing patterns in both files. No significant issues found.

Minor observations:

  • The HAProxy rate limit values are quoted as strings (e.g., "1000") while the FlowProxy config uses unquoted values for addresses/ports. This is likely fine if the consumer expects strings for rate limits, but worth confirming consistency with the upstream config schema.
  • The rate limits are set very high (e.g., 10GB bytes-in rate for regular, 100GB for privileged), which makes sense for a local playground/dev environment to avoid interfering with testing.
  • The flashbots_orderflow_signing_address is set to the zero address — appropriate for a playground config where no real signing is needed.

No blocking issues. LGTM.

@canercidam canercidam merged commit db477c5 into main Feb 18, 2026
14 checks passed
@canercidam canercidam deleted the fryd/buildernet-vm-add-config-and-ports branch February 18, 2026 16:25
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