Skip to content

feat(webui): add support for specific IPs for port binding#425

Merged
kvinwang merged 8 commits into
Dstack-TEE:masterfrom
barakeinav1:barak/webui_enable_dedicated_ips
Jan 9, 2026
Merged

feat(webui): add support for specific IPs for port binding#425
kvinwang merged 8 commits into
Dstack-TEE:masterfrom
barakeinav1:barak/webui_enable_dedicated_ips

Conversation

@barakeinav1

@barakeinav1 barakeinav1 commented Jan 4, 2026

Copy link
Copy Markdown
Contributor

Add support for custom external IP addresses in CVM port mappings, and fix UI display and alignment issues to correctly reflect dedicated IP bindings.

Changes by file:
vmm/ui/src/components/PortMappingEditor.ts

Added Custom IP option for port mappings and normalized existing ports when editing VMs.

vmm/ui/src/templates/app.html

Fixed port mapping display to show the actual dedicated IP instead of labeling it as “Public”.

vmm/ui/src/styles/main.css

Updated port row grid layout to keep buttons aligned when Custom IP is used.

@barakeinav1
barakeinav1 marked this pull request as draft January 4, 2026 10:55
@barakeinav1
barakeinav1 force-pushed the barak/webui_enable_dedicated_ips branch from 6f61cf2 to 09d791f Compare January 8, 2026 07:43
@barakeinav1
barakeinav1 marked this pull request as ready for review January 8, 2026 08:01
@barakeinav1

Copy link
Copy Markdown
Contributor Author
image image image

@kvinwang kvinwang left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Thanks!

@kvinwang
kvinwang enabled auto-merge January 9, 2026 00:22
@kvinwang
kvinwang merged commit 6ce16c2 into Dstack-TEE:master Jan 9, 2026
4 checks passed
kvinwang added a commit that referenced this pull request Jan 19, 2026
…_ips

feat(webui): add support for specific IPs for port binding
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