UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Setting recommended ingester settings from grafana Github, cleaning up values, adding queryUI templates

Merged Ryan Garcia requested to merge prod-tuning-recs into main
All threads resolved!

Noticing some slight inconsistencies between traceIDs showing up attached to Loki logs and actually being available to view in Tempo.

  • Addressing those reliability issues with an NP to allow ingress on tempo:9411 in namespace (to recieve traces from each apps istio-proxy sidecars).

  • Enabling zipkin endpoint listener on 0.0.0.0:9411

  • Enabling searching via tempoQuery (so it's actually used) which turns out to be JaegerUI itself.

  • Adding in values/templates/networkPolicies to facilitate external connections through istio to tempoQuery(Jaeger) UI via https://tracing.bigbang.dev

  • Cleaning up comments in values file that we control

Reviewing their documentation pages and recommended configuration on Github, making sure our helm chart makes use of their recommendations.

exporting of traces is a lot better with this configuration, but still seeing magnitudes more of traces in Loki compared to Tempo.

Edited by Ryan Garcia

Merge request reports

Merge request pipeline #719150 passed

Merge request pipeline passed for 7ba8c665

Merged by Ryan GarciaRyan Garcia 3 years ago (Mar 15, 2022 4:58pm UTC)

Merge details

  • Changes merged into main with 664fd20b (commits were squashed).
  • Deleted the source branch.

Pipeline #723434 passed

Pipeline passed for 664fd20b on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ryan Garcia added 1 commit

    added 1 commit

    • d3fd56f8 - Istio IGW to UI NP template and values

    Compare with previous version

  • Ryan Garcia added 1 commit

    added 1 commit

    Compare with previous version

  • Ryan Garcia added 1 commit

    added 1 commit

    Compare with previous version

  • Ryan Garcia resolved all threads

    resolved all threads

  • Ryan Garcia changed title from Setting recommended ingester settings from grafana Github to Setting recommended ingester settings from grafana Github, cleaning up values, adding queryUI templates

    changed title from Setting recommended ingester settings from grafana Github to Setting recommended ingester settings from grafana Github, cleaning up values, adding queryUI templates

  • Ryan Garcia changed the description

    changed the description

  • Ryan Garcia added kindbug statusreview labels and removed statusdoing label

    added kindbug statusreview labels and removed statusdoing label

  • Ryan Garcia requested review from @michaelmartin

    requested review from @michaelmartin

  • Ryan Garcia requested review from @echuang and @micah.nagel

    requested review from @echuang and @micah.nagel

  • looks good. installed ok for me.

  • Michael Martin approved this merge request

    approved this merge request

  • merged

  • Ryan Garcia mentioned in commit 664fd20b

    mentioned in commit 664fd20b

  • Please register or sign in to reply
    Loading