Skip to content

Conversation

@BelSasha
Copy link
Contributor

No description provided.

@BelSasha BelSasha changed the title support multi tenant [WIP] support multi tenant Nov 25, 2025
Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@BelSasha BelSasha force-pushed the sb/multi-tenant-support branch from 2831a98 to 39005f6 Compare November 25, 2025 12:41
location ^~ /prometheus/ {
resolver {{ .Values.nginx.resolver }} valid=5s;
set $prom_host "kubetorch-metrics.{{ .Release.Namespace }}.svc.cluster.local:{{ .Values.metrics.prometheus.port }}";
set $prom_host "{{ .Values.metrics.metricsServiceName }}.{{ if eq .Values.metrics.metricsServiceName "kubetorch-metrics" }}{{ .Release.Namespace }}{{ else }}platform{{ end }}.svc.cluster.local:{{ .Values.metrics.prometheus.port }}";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can remove the if else logic here and just use --set metrics.metricsServiceName when installing?

location ^~ {{ .Values.nginxProxy.backends.logging.route }}/ {
resolver {{ .Values.nginx.resolver }} valid=5s;
set $loki_host "loki-gateway.{{ .Release.Namespace }}.svc.cluster.local:{{ .Values.logStreaming.port }}";
set $loki_host "loki-gateway.{{ if eq .Values.logStreaming.port 3100 }}{{ .Release.Namespace }}{{ else }}platform{{ end }}.svc.cluster.local:{{ .Values.logStreaming.port }}";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

similar note above

@BelSasha BelSasha force-pushed the sb/multi-tenant-support branch 3 times, most recently from 2375539 to a3fafe5 Compare December 1, 2025 20:24
@BelSasha BelSasha force-pushed the sb/multi-tenant-support branch from a3fafe5 to b28683d Compare December 2, 2025 07:42
@BelSasha BelSasha marked this pull request as ready for review December 2, 2025 07:42
@BelSasha BelSasha changed the title [WIP] support multi tenant Support multi tenant Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants