Skip to content

Add an ingress-controller to the stack - #17

Merged
bussyjd merged 8 commits into
mainfrom
oisin/ingress
Jun 23, 2025
Merged

Add an ingress-controller to the stack#17
bussyjd merged 8 commits into
mainfrom
oisin/ingress

Conversation

@OisinKyne

@OisinKyne OisinKyne commented Jun 18, 2025

Copy link
Copy Markdown
Contributor

Tackles [#3].

This PR introduces an ingress-controller to the stack, which allows Obol Apps to speak to the host machine. This allows a host's browser to connect to Obol Apps via the intranet.

Screenshot 2025-06-18 at 18 45 40

Obol Stack Local RPC

ToDo:

  • Put this extra code behind a flag like --host or similar. Takes a long time to start up, not everyone might want it.
  • Delay the kubectl apply command until ingress-nginx can receive traffic, otherwise it will always fail
  • Put the RPC on a different path to a bare minimum hello world webpage. abstract away the evm/network bit ideally.
  • Figure out why metamask isn't happy with the RPC currently
  • Document how to add the RPC to your local metamask.
  • Make a hello world home page with instructions on adding the RPC url to metamask
  • Replace the temp home page with something more proper. Ultimately a full on website.
  • Should we consider making this feature opt-out, behind a flag like --headless ?
  • Re-running ./obolup --host after ./obolup doesn't work without --clean, because we've already created a kind cluster without the needed port mappings. Add warning logging requiring a clean and/or auto recreate
  • Fix race condition of launching the browser before nginx is ready to serve the new ingress

@OisinKyne OisinKyne changed the title Progress towards a localhost ingress for obol stack apps Add an ingress-controller to the stack Jun 18, 2025
@bussyjd
bussyjd changed the base branch from main to feature/adk June 18, 2025 18:55
@OisinKyne
OisinKyne changed the base branch from feature/adk to main June 19, 2025 12:39
@OisinKyne OisinKyne mentioned this pull request Jun 19, 2025
1 task
bussyjd and others added 3 commits June 20, 2025 12:38
Added "Add to MetaMask" buttons for both Mainnet and Hoodi RPC endpoints
to make it easier for users to configure their wallets.
When running ./obolup --host after ./obolup without --clean, the existing
cluster lacks required port mappings (80/443). Added detection logic that:
- Checks if existing cluster has required port mappings for host mode
- Shows clear warning message when mappings are missing
- Instructs user to run with --clean flag
- Prevents execution to avoid confusing errors
@bussyjd

bussyjd commented Jun 21, 2025

Copy link
Copy Markdown
Contributor

Replace the temp home page with something more proper. Ultimately a full on website.

We'll eventually switch to our own UI hosted in the obol-adk agent
https://google.github.io/adk-docs/tools/mcp-tools/#step-3-test-your-custom-mcp-server-with-an-adk-agent

@bussyjd bussyjd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@bussyjd
bussyjd merged commit 9647997 into main Jun 23, 2025
1 check passed
@OisinKyne
OisinKyne deleted the oisin/ingress branch August 5, 2025 18:26
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