-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add ability to use custom basePath #511
Conversation
Hi @RustamGimadiev. Does it solves #364 (comment) ? |
hi @andormarkus yes, this PR will solve that issue, now need to do some frontend works to fix broken links in the sidebar, I guess the ALB ingress controller will also get working, now we test it with nginx without rewrite rules |
3d9f20c
to
91b5d7a
Compare
Kudos, SonarCloud Quality Gate passed! |
Kudos, SonarCloud Quality Gate passed!
|
…sePath is used than Helm chart livenessProbe and readinessProbe will fail because it will try to probe on the default path which does not exist anymore. With these changes it automatically picks up `SERVER_SERVLET_CONTEXT_PATH` from the myconfig.yaml and sets correct path for livenessProbe and readinessProbe
… used than Helm chart livenessProbe and readinessProbe will fail because it will try to probe on the default path which does not exist anymore. With these changes it automatically picks up `SERVER_SERVLET_CONTEXT_PATH` from the myconfig.yaml and sets correct path for livenessProbe and readinessProbe
* add ability to use relative pathes * Add basename prop (provectus#513) * add static controller * add docker-compose * Refactoring * Refactoring * fixed comparison bugs * dirty * clenaup * Update React app Co-authored-by: Alexander Krivonosov <31561808+GneyHabub@users.noreply.github.com> Co-authored-by: German Osin <german.osin@gmail.com> Co-authored-by: Oleg Shuralev <workshur@gmail.com>
…sePath is used than Helm chart livenessProbe and readinessProbe will fail because it will try to probe on the default path which does not exist anymore. With these changes it automatically picks up `SERVER_SERVLET_CONTEXT_PATH` from the myconfig.yaml and sets correct path for livenessProbe and readinessProbe
What changes did you make? (Give an overview)
Is there anything you'd like reviewers to focus on?
These changes will resolve #364, to test this deployment case need to execute
docker compose up -f docker/kafka-ui-reverse-proxy.yaml
, it simulates the same behavior as ingress with custom basePath in K8sHow Has This Been Tested? (put an "X" next to an item)
Checklist (put an "X" next to an item, otherwise PR will fail)
Check out Contributing and Code of Conduct