Skip to content

chore: Update localnet TEE script for Rust launcher#2563

Open
barakeinav1 wants to merge 3 commits intomainfrom
add-localnet-toml-template-and-env
Open

chore: Update localnet TEE script for Rust launcher#2563
barakeinav1 wants to merge 3 commits intomainfrom
add-localnet-toml-template-and-env

Conversation

@barakeinav1
Copy link
Copy Markdown
Contributor

@barakeinav1 barakeinav1 commented Mar 23, 2026

closes #2523
This PR updates the existing localnet TEE script to the new RUST launcher and toml input it uses.

Summary

  • Add node.conf.localnet.toml.tpl — TOML config template for the Rust tee-launcher, matching the Config struct with [launcher_config], [mpc_node_config], and subsections
  • Add set-localnet-env.sh — convenience script to set all required env vars for a localnet deploy run

These are new files only — existing scripts are not modified.

Test plan

  • Verify template renders valid TOML via envsubst
  • Source set-localnet-env.sh and run deploy-tee-localnet.sh

@claude
Copy link
Copy Markdown

claude bot commented Mar 23, 2026

Review

No critical issues found. The PR adds two new localnet-only config/env files with no changes to existing code.

Minor note (non-blocking): set-localnet-env.sh:14 has MACHINE_IP=51.68.219.1 and set-localnet-env.sh:15 has BASE_PATH=/mnt/data/barak/dstack which appear to be developer-specific values. These won't cause bugs but may confuse other users of the template. Consider adding a comment noting these should be customized per-environment, or using placeholder values.

✅ Approved — straightforward addition of localnet deployment templates with no production code changes.

@barakeinav1 barakeinav1 changed the title chore: add TOML config template and localnet env script for Rust launcher chore: Update localnet TEE script for Rust launcher Mar 25, 2026
Copy link
Copy Markdown
Collaborator

@netrome netrome left a comment

Choose a reason for hiding this comment

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

Approving since these are standalone scripts not tied to production. I haven't reviewed them given the size of the PR and my limited time today.

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.

Update localnet TEE deploy script for Rust launcher

2 participants