Uploaded image for project: 'Distributed Tracing'
  1. Distributed Tracing
  2. TRACING-5750

Show warning for non-multitenancy TempoStack and TempoMonolithic instances

XMLWordPrintable

    • Icon: Epic Epic
    • Resolution: Done
    • Icon: Undefined Undefined
    • rhosdt-3.9
    • None
    • None
    • None
    • Support non-multitenancy TempoStack and TempoMonolithic instances in Tracing UI
    • 1
    • False
    • Hide

      None

      Show
      None
    • False
    • Not Selected
    • In Progress
    • 0% To Do, 0% In Progress, 100% Done

      The most common question for the Tracing UI is: "Why is my Tempo instance not showing up?"
      The reason why non-multitenancy instances are not supported is because they don't deploy a gateway, and therefore don't have any authentication/authorization.
      To support these instances, we can add a RBAC check on the UI plugin backend (check if user has access to the namespace of the Tempo instance).

      • Add a required field to Tempo CRs to indicate a non secure instance is going to be created (e.g. tenants.mode=insecure)
      • Add oauth proxy for Tempo query frontend
      • Show non secure instances in the UI

      The service account of the oauth-proxy pod requires TokenReview and SubjectAccessReview to verify the bearer token. However, granting these roles poses a security issue, as it can lead to privilege escalation (ownership of one namespace could lead to getting cluster-wide SubjectAccessReview access). Therefore, we won't support this and will show a warning in the docs and in the validating webhook.

              agerstma@redhat.com Andreas Gerstmayr
              agerstma@redhat.com Andreas Gerstmayr
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: