UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Update Ironbank

Merged RENOVATE_BOT requested to merge renovate/ironbank into main
All threads resolved!
2 files
+ 7
7
Compare changes
  • Side-by-side
  • Inline
Files
2
@@ -159,13 +159,13 @@ annotations:
repository: registry1.dso.mil/ironbank/opensource/grafana/tempo-query
```
- Ensure `tempoQuery.enabled` is true
- Ensure `tempoQuery.enabled` is __false__
Note: [this](https://github.com/grafana/helm-charts/commit/4c77fa7b3a54977d094071b446ff8b5b86982858) upstream commit disabled `tempo-query` by default in the chart. Evidently this is because `tempo-query` was always meant as a shim between Tempo and Grafana, but it hasn't been necessary [since 7.5.0](https://github.com/grafana/tempo/issues/456#issuecomment-815813684), as Grafana is capable of querying Tempo directly now.
Currently, Big Bang uses `tempo-query` for Cypress testing and users may expect a basic web interface for Tempo without Grafana (Tempo has non natively, only a HTTP API). This may be changed in an upcoming release, but we will keep utilizing `tempo-query` for the benefits of the interface.
Previously, Big Bang used `tempo-query` for Cypress testing and to provide users a basic web interface for Tempo without Grafana (Tempo offers only an HTTP API natively). This was changed after breaking changes to the `tempo-query` deployment model were [introduced in version 2.6.0](https://grafana.com/docs/tempo/next/release-notes/v2-6/#other-breaking-changes). Other `tempoQuery` configuration settings remain below, in the event users wish to deploy it with the associated Iron Bank container image.
```yaml
enabled: true
enabled: false
```
- Ensure `tempoQuery.resources` requests and limits are set
Loading