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

Port-forward directly to k3s #13314

Merged
merged 1 commit into from
Sep 26, 2022
Merged

Port-forward directly to k3s #13314

merged 1 commit into from
Sep 26, 2022

Conversation

mads-hartmann
Copy link
Contributor

Description

Port forwarding to Grafana/Prometheus in preview environments is broken at the moment. We get an error when we try to visit the page. See below:

gitpod /workspace/gitpod (ak/supervisor_observability) $ ./dev/preview/portforward-monitoring-satellite.sh -c harvester

Starting port-forwarding:

Prometheus:
https://9090-gitpodio-gitpod-ijwocmzi0o3.ws-eu67.gitpod.io

Grafana:
https://3000-gitpodio-gitpod-ijwocmzi0o3.ws-eu67.gitpod.io


Forwarding from 127.0.0.1:9090 -> 32001
Forwarding from [::1]:9090 -> 32001
Forwarding from 127.0.0.1:3000 -> 32000
Forwarding from [::1]:3000 -> 32000
Handling connection for 3000
E0926 13:52:32.090059   23719 portforward.go:406] an error occurred forwarding 3000 -> 32000: error forwarding port 32000 to pod 351aac1da93beb827a09a5a69754f125480d8216fde7517febfa68f7e5e64330, uid : failed to execute portforward in network namespace "/var/run/netns/cni-07716292-fde6-283c-aad7-8f84b947f878": failed to dial 32000: dial tcp4 127.0.0.1:32000: connect: connection refused
E0926 13:52:32.090520   23719 portforward.go:234] lost connection to pod

This PR changes it so that we don't go through svc/proxy in the Harvester cluster, but instead go directly to the k3s cluster.

Related Issue(s)

No issue. Reported here.

How to test

I used this change on Antons branch and it worked.

Release Notes

NONE

Documentation

N/A

Werft options:

  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-integration-tests=all
    Valid options are all, workspace, webapp, ide

@roboquat roboquat merged commit 6c9f601 into main Sep 26, 2022
@roboquat roboquat deleted the mads/fix-port-forward branch September 26, 2022 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants