Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Documentation request: please add this solution for coder's embedded DERP server behind an istio ingress reverse proxy #11821

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.

Sign up for GitHub

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

Jump to bottom
Closed
valkenburg-prevue-ch opened this issue Jan 25, 2024 · 6 comments · Fixed by #15733
Closed

Documentation request: please add this solution for coder's embedded DERP server behind an istio ingress reverse proxy #11821

valkenburg-prevue-ch opened this issue Jan 25, 2024 · 6 comments · Fixed by #15733
Assignees
@ericpaulsen
Labels
docs Area: coder.com/docs

Comments

@valkenburg-prevue-ch
Copy link

valkenburg-prevue-ch commented Jan 25, 2024 •
edited
Loading

I am very happy with the 'health' tab in the coder ui, which revealed to me that the embedded derp server was not accessible. The help documentation which I was then pointed to, was helpful as well. If you could add the following solution somehow somewhere, this will save all people with the same problem the two days it took me to find this solution.

When coder is installed inside an istio mesh, and your ingress is managed by istio, you have to apply the following manifest:

apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: headscale-behind-istio-ingress
  namespace: istio-system
spec:
  configPatches:
    - applyTo: NETWORK_FILTER
      match:
        listener:
          filterChain:
            filter:
              name: envoy.filters.network.http_connection_manager
      patch:
        operation: MERGE
        value:
          typed_config:
            "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
            upgrade_configs:
              - upgrade_type: derp

which is directly derived from the solution found at https://github.com/juanfont/headscale/blob/main/docs/reverse-proxy.md#istio
@cdr-bot cdr-bot bot added the docs Area: coder.com/docs label Jan 25, 2024
@kylecarbs
Copy link
Member

cc @ericpaulsen maybe we just toss this under an Istio guide or something?

@kylecarbs
Copy link
Member

Thank you, @valkenburg-prevue-ch, for persisting in fixing this. You'll help a lot of people I'm sure! 🥳

@ericpaulsen
Copy link
Member

cc @ericpaulsen maybe we just toss this under an Istio guide or something?

can do.

@valkenburg-prevue-ch
Copy link
Author

Thank you, @valkenburg-prevue-ch, for persisting in fixing this. You'll help a lot of people I'm sure! 🥳

This is my way of paying for the free version I've been very happily and intensively using.

@ericpaulsen
Copy link
Member

@valkenburg-prevue-ch thank you for your service 🖖🏼

@ericpaulsen ericpaulsen self-assigned this Jan 25, 2024
@github-actions github-actions bot added the stale This issue is like stale bread. label Jul 24, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 1, 2024
@johnstcn johnstcn reopened this Nov 18, 2024
@johnstcn johnstcn removed the stale This issue is like stale bread. label Nov 18, 2024
@coder-labeler coder-labeler bot added the needs-triage Issue that require triage label Nov 18, 2024
@ericpaulsen
Copy link
Member

I should've gotten around to this - I now have cycles to work on this.

cc: @EdwardAngert

@matifali matifali removed the needs-triage Issue that require triage label Nov 21, 2024
@ericpaulsen ericpaulsen mentioned this issue Dec 3, 2024
Merged
aslilac pushed a commit that referenced this issue Dec 11, 2024
@ericpaulsen @aslilac
docs: add istio docs (#15733)
e389f13 
closes #11821
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ericpaulsen ericpaulsen

Labels
docs Area: coder.com/docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs: add istio docs coder/coder
5 participants
@johnstcn @kylecarbs @ericpaulsen @matifali @valkenburg-prevue-ch