Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ui does not correctly display in dynamically allocated ports "0:[port]" #6247

Open
nicks opened this issue Oct 10, 2023 · 0 comments
Open

ui does not correctly display in dynamically allocated ports "0:[port]" #6247

nicks opened this issue Oct 10, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@nicks
Copy link
Member

nicks commented Oct 10, 2023

Steps to Reproduce

  1. Update tilt-example-html with
k8s_resource('example-html', port_forwards=["0:8000"])
  1. Run tilt up
  2. Visit the Tilt ui

Expected Behavior

the UI should link to the service we're port-fowarding to

Current Behavior

the UI links to localhost:0

Additional Info

The weirdest part of this bug is that the API server actually knows the correct port

$ tilt get pf -o yaml
apiVersion: v1
items:
- apiVersion: tilt.dev/v1alpha1
  kind: PortForward
  metadata:
    annotations:
      tilt.dev/log-span-id: kubernetesapply:example-html
      tilt.dev/resource: example-html
    creationTimestamp: "2023-10-10T21:53:23Z"
    name: example-html-example-html-bc598cb58-pxq7t
    ownerReferences:
    - apiVersion: tilt.dev/v1alpha1
      blockOwnerDeletion: true
      controller: true
      kind: KubernetesDiscovery
      name: example-html
      uid: 28f91202-4a98-4dd6-bf11-5e48f87c9fa4
    resourceVersion: "92"
    uid: 7b36ea85-6ab4-4cf6-a640-ab5ec31f86e7
  spec:
    cluster: default
    forwards:
    - containerPort: 8000
      host: localhost
    namespace: default
    podName: example-html-bc598cb58-pxq7t
  status:
    forwardStatuses:
    - addresses:
      - 127.0.0.1
      - ::1
      containerPort: 8000
      localPort: 46753
      startedAt: "2023-10-10T21:53:36.204691Z"
kind: List
metadata:
  resourceVersion: ""
@nicks nicks added the bug Something isn't working label Oct 10, 2023
@nicks nicks changed the title ui does not correctly display in dynamically allocated ports ui does not correctly display in dynamically allocated ports "0:[port]" Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant