feat(pegboard): expose rivet server from within containers for docker compose

[NathanFlurry]

Changes

[NathanFlurry]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• fix: fix logs for new actors #2649
• feat: add actor queue #2644
• feat: add actor kv to runners #2638
• feat: get multi actors working e2e on docker compose #2637
• fix: add udp ports to docker compose #2635
• chore: fix merge conflicts #2634
• fix: get new actors working on docker compose #2623
• fix: figure out actor log shipping #2622
• fix(pegboard): convert ws manager socket to unix socket #2478
• fix(pb): get new actor ids working e2e #2471
• feat: implement new actor ids #2467
• feat(pb): get multi-actors working e2e, add resources for builds #2465
• feat(pb): add runner logs #2447
• feat: add runners to pb protocol #2433
• chore(pegboard): add exit code log from container runner #2477
• fix(workflows): fix branch version bug #2475
• feat(pegboard): expose rivet server from within containers for docker compose #2628 👈 (View in Graphite)
• feat(pegboard): add support for custom host entries #2627
• fix(pegboard): fix hosts file not getting written correctly #2626
• main

---

How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cloudflare-workers-and-pages]

Deploying rivet with    Cloudflare Pages

Latest commit:	eea0c38
Status:	✅  Deploy successful!
Preview URL:	https://a648c09f.rivet.pages.dev
Branch Preview URL:	https://06-17-feat-pegboard-expose-r.rivet.pages.dev

View logs

[graphite-app]

The DNS lookup for rivet-server lacks error handling. If the lookup fails, RIVET_SERVER_HOST will be empty, resulting in an invalid iptables rule (-j DNAT --to-destination ). Consider adding validation to ensure RIVET_SERVER_HOST contains a valid IP address before creating the rule:

RIVET_SERVER_HOST="$(dig +short rivet-server)"
if [ -z "$RIVET_SERVER_HOST" ]; then
    echo "ERROR: Failed to resolve rivet-server hostname"
    exit 1
fi
echo "Adding rule to forward $RIVET_SERVER_IPV4 to rivet-server ($RIVET_SERVER_HOST)"
add_ipt_rule "$ipt" "nat" "$NAT_CHAIN" "-d $RIVET_SERVER_IPV4 -j DNAT --to-destination $RIVET_SERVER_HOST"

This ensures the script fails explicitly rather than creating a broken network configuration.

Suggested change

	RIVET_SERVER_HOST="$(dig +short rivet-server)"
	echo "Adding rule to forward $RIVET_SERVER_IPV4 to rivet-server ($RIVET_SERVER_HOST)"
	add_ipt_rule "$ipt" "nat" "$NAT_CHAIN" "-d $RIVET_SERVER_IPV4 -j DNAT --to-destination $RIVET_SERVER_HOST"
	RIVET_SERVER_HOST="$(dig +short rivet-server)"
	if [ -z "$RIVET_SERVER_HOST" ]; then
	echo "ERROR: Failed to resolve rivet-server hostname"
	exit 1
	fi
	echo "Adding rule to forward $RIVET_SERVER_IPV4 to rivet-server ($RIVET_SERVER_HOST)"
	add_ipt_rule "$ipt" "nat" "$NAT_CHAIN" "-d $RIVET_SERVER_IPV4 -j DNAT --to-destination $RIVET_SERVER_HOST"

Spotted by Diamond

Is this helpful? React 👍 or 👎 to let us know.

[cloudflare-workers-and-pages]

Deploying rivet-hub with    Cloudflare Pages

Latest commit:	eea0c38
Status:	✅  Deploy successful!
Preview URL:	https://55ee5baf.rivet-hub-7jb.pages.dev
Branch Preview URL:	https://06-17-feat-pegboard-expose-r.rivet-hub-7jb.pages.dev

View logs

[cloudflare-workers-and-pages]

Deploying rivet-studio with    Cloudflare Pages

Latest commit:	eea0c38
Status:	⚡️  Build in progress...

View logs

[graphite-app]

Merge activity

• Jun 28, 1:49 AM UTC: NathanFlurry added this pull request to the Graphite merge queue.
• Jun 28, 1:50 AM UTC: CI is running for this pull request on a draft pull request (#2662) due to your merge queue CI optimization settings.
• Jun 28, 1:51 AM UTC: Merged by the Graphite merge queue via draft PR: #2662.