cortex-helm-chart icon indicating copy to clipboard operation
cortex-helm-chart copied to clipboard

Allow optional entry to /ruler/rule_groups endpoint

Open AlexDCraig opened this issue 1 year ago • 3 comments

What this PR does: Allow optional entry into /ruler/rule_groups endpoint via nginx proxy

Which issue(s) this PR fixes: Fixes #475

Checklist

  • [ x] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

AlexDCraig avatar Aug 07 '23 22:08 AlexDCraig

The custom route injection for this is kind of hideous:

nginx:
  config:
    serverSnippet:
      location ~ /ruler/rule_groups {
          proxy_pass      http://{{ template "cortex.fullname" . }}-ruler.{{ printf "%s.svc.%s:%d" .Release.Namespace .Values.clusterDomain (.Values.config.server.http_listen_port | int) }}$request_uri;
      }

Or is there a way to call that $rootDomain function the other routes use from the values file?

AlexDCraig avatar Aug 08 '23 15:08 AlexDCraig

The custom route injection for this is kind of hideous:

nginx:
  config:
    serverSnippet:
      location ~ /ruler/rule_groups {
          proxy_pass      http://{{ template "cortex.fullname" . }}-ruler.{{ printf "%s.svc.%s:%d" .Release.Namespace .Values.clusterDomain (.Values.config.server.http_listen_port | int) }}$request_uri;
      }

Or is there a way to call that $rootDomain function the other routes use from the values file?

I don't know, have you tried setting $rootDomain? I assume the context might be different

nschad avatar Aug 09 '23 06:08 nschad

This issue has been automatically marked as stale because it has not had any activity in the past 30 days. It will be closed in 15 days if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Sep 17 '23 01:09 stale[bot]