seriohub / velero-helm

Helm charts for velero-ui, velero-api, and velero-watchdog
14 stars 6 forks source link

Need to consolidate ingress rules into one file, otherwise api path does not work #17

Closed nickveldrin closed 4 months ago

nickveldrin commented 4 months ago

Hello, i am trying to see if i can get this project to work, and while i'm still not exactly there, i had noticed that the helm chart deploys ingresses for both the UI as well as API. When i try to hit the /api, it does not work, and the logon page of the UI is not able to reach it.

However, if i append the api pieces of the backend service to the UI ingress, the api is then reachable.

Sorry, i'm not sure how to fix the comment to show the code properly.

spec:
  ingressClassName: nginx
  rules:
  - host: velero.domain
    http:
      paths:
      - backend:
          service:
            name: velero-ui-vui-ui
            port:
              number: 80
        path: /
        pathType: Prefix
      - backend:
          service:
            name: velero-ui-vui-api
            port:
              number: 80
        path: /api
        pathType: Prefix
      - backend:
          service:
            name: velero-ui-vui-api
            port:
              number: 80
        path: /ws
        pathType: Prefix
  tls:
  - hosts:
    - velero.domain
davideserio commented 4 months ago

Hi, after your correction, does the UI communicate correctly with the API? If not, what does the diagnostic information on the login page show?

nickveldrin commented 4 months ago

Hi, after your correction, does the UI communicate correctly with the API? If not, what does the diagnostic information on the login page show?

Yes, after consolidating the ingress paths everything is good on the front page.

davideserio commented 4 months ago

Thanks for the report. In the next version, we'll improve the ingress.

nickveldrin commented 4 months ago

I figured out the code markdown above, and I've attached a screenshot with the single ingress with those edits. velero 2024-05-15 051449