Skip to content

Allow passing a port when starting a new session#27

Merged
noorvir merged 1 commit intomasterfrom
zak/ingress-application-ports
Jun 21, 2023
Merged

Allow passing a port when starting a new session#27
noorvir merged 1 commit intomasterfrom
zak/ingress-application-ports

Conversation

@zknill
Copy link
Copy Markdown
Contributor

@zknill zknill commented Jun 21, 2023 •
edited
Loading

Produces an ls output like:

--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 Name                           vCPUs GPU           NumGPUs      HDD (GB)   Provider     Status            Connection String     Http Service                                                                      
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 horn-distant-older-pocket      4     rtx_4000      1            50         unweave      running           root@216.153.51.162   https://exc-uxvezhnxunf0wlo.tenant-unweave-zak.unweave.dev (internal port: 8080)

@zknill zknill requested a review from noorvir June 21, 2023 08:16
@zknill zknill force-pushed the zak/ingress-application-ports branch 2 times, most recently from 5d18fde to 3dc6962 Compare June 21, 2023 14:12
noorvir
noorvir reviewed Jun 21, 2023
View reviewed changes
@zknill zknill force-pushed the zak/ingress-application-ports branch from 3dc6962 to 963298c Compare June 21, 2023 15:07
@zknill
Allow passing a port when starting a new session
b44852d 
Expose that port as an https service.
@zknill zknill force-pushed the zak/ingress-application-ports branch from 963298c to b44852d Compare June 21, 2023 16:06
@noorvir noorvir merged commit 61c1e20 into master Jun 21, 2023
@noorvir noorvir deleted the zak/ingress-application-ports branch June 21, 2023 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@noorvir noorvir noorvir approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@zknill @noorvir