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

URL Malformed on Backfill with ingress "my.server/mycodeserver" #16690

Closed
airton-neto opened this issue Sep 21, 2023 · 0 comments · Fixed by #16954
Closed

URL Malformed on Backfill with ingress "my.server/mycodeserver" #16690

airton-neto opened this issue Sep 21, 2023 · 0 comments · Fixed by #16954
Assignees
Labels
type: bug Something isn't working

Comments

@airton-neto
Copy link

airton-neto commented Sep 21, 2023

Dagster version

1.4.15

What's the issue?

URL composition problem when doing bakfills.

Ingress has the structure "my.server/mycodeserver" (my.server was wrapped in the pics).
After adding backfill and clicking "View" tooltip, codeserver is taken off of the URL and the page crashes.
_1
_2

My ingress.yml file is like

apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  annotations:
    cert-manager.io/cluster-issuer: letsencrypt
    ingress.kubernetes.io/ssl-redirect: 'true'
    nginx.org/ssl-services: dagsters
  name: dagsters
  namespace: dagsters
spec:
  ingressClassName: nginx
  rules:
    - host: my.dagster.server
      http:
        paths:
          - backend:
              service:
                name: eolica-2w-webserver
                port:
                  name: http
            path: /eolica-2w
            pathType: ImplementationSpecific
  tls:
    - hosts:
        - my.dagster.server
      secretName: dagsters-tls-cert

What did you expect to happen?

See backfill page (I still can, just by fixing the url after pressing "view").

How to reproduce?

Deploy a code server using this kind of ingress
Access via browser (Chrome in my case)
Create a backfill
Press "View".

Deployment type

Dagster Helm chart

Deployment details

Used the latest helm chart version.

Additional information

No response

Message from the maintainers

Impacted by this issue? Give it a 👍! We factor engagement into prioritization.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants