Modify authservice prefixes to use FQDN
General MR
Summary
Currently when Tempo is on tempo.<domain>
the Authservice prefix is tempo
, which ends up having a side effect of matching the internal k8s service name of tempo-tempo...
.
To workaround this we set the prefix to include the domain. This is done for all chains to prevent potential future issues.
Merge request reports
Activity
added authservice label
assigned to @micah.nagel
requested review from @stas and @ryan.j.garcia
added 5 commits
-
626c5db2...216a3ac6 - 4 commits from branch
master
- e19c4b33 - Merge branch 'master' into tempo-sso-fix
-
626c5db2...216a3ac6 - 4 commits from branch
- Resolved by Micah Nagel
I think this makes sense anyways. That keeps us from matching
tempodemo.bigbang.dev
vs.tempo.bigbang.dev
too.
added 14 commits
-
e19c4b33...a5c51e42 - 12 commits from branch
master
- f2ea93b9 - Merge branch 'master' into tempo-sso-fix
- 9b1ccd0d - change match to include domain
-
e19c4b33...a5c51e42 - 12 commits from branch
added 15 commits
-
9b1ccd0d...9ab48e2e - 14 commits from branch
master
- 079b6a89 - Merge branch 'master' into tempo-sso-fix
-
9b1ccd0d...9ab48e2e - 14 commits from branch
requested review from @michaelmcleroy
requested review from @BrandenCobb
added statusreview label
changed milestone to %1.47.0
mentioned in commit a4456754