Skip to content

chore: fix flags and parameters to allow local devnet test for buildernet #101

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

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

MoeMahhouk
Copy link
Contributor

This PR adds the following:

  • Local port binding for security purposes and disallow exposing any ports external, such as the attack attempt on the postgres container being exposed externally with poor authentication.
  • Switched from latest version for the builder-hub and its db containers to a stable version to avoid potential conflicts
  • Using own custom nginx mock proxy instead of the builder-hub-mock-proxy due to hardcoded configuration in the original container image
  • Changed the address of reth to 0.0.0.0 instead of 127.0.0.1 to expose it for the qemu VM image to allow peering
  • Changed the enr-address of lighthouse beacon node to 10.0.2.2 which is qemu's default gateway address for the host. This way, the host would broadcast itself to the peers over the same network that the buildernet qemu VM image accesses

@MoeMahhouk MoeMahhouk force-pushed the mm-builderhub-mock-proxy branch from 13ae8f1 to 6a4d269 Compare April 17, 2025 08:31
@MoeMahhouk MoeMahhouk changed the title Mm builderhub mock proxy chore: fix flags and parameters to allow local devnet test for buildernet Apr 17, 2025
WithPort("http", 8888).
DependsOnRunning(b.TargetService).
WithEntrypoint("/bin/sh").
WithArgs("-c", fmt.Sprintf(`cat > /etc/nginx/conf.d/default.conf << 'EOF'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need nginx?

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